Add eslint-plugin-ie11 (#1666)

This would have prevented the failing build in master that occurred
after merging #1661, which would have allowed the author to fix it.

package.json

@@ -50,6 +50,7 @@
     "dependency-check": "^2.9.1",
     "eslint": "^4.6.1",
     "eslint-config-sinon": "^1.0.0",
+    "eslint-plugin-ie11": "^1.0.0",
     "eslint-plugin-mocha": "^4.2.0",
     "husky": "^0.14.2",
     "lint-staged": "^6.0.0",

test/.eslintrc

@@ -2,6 +2,7 @@ env:
   mocha: true
 
 plugins:
+  - ie11
   - mocha
 
 rules:
@@ -20,3 +21,8 @@ rules:
   mocha/no-sibling-hooks: error
   mocha/no-skipped-tests: warn
   mocha/no-top-level-hooks: error
+
+  ie11/no-collection-args: error
+  ie11/no-for-in-const: error
+  ie11/no-loop-func: warn
+  ie11/no-weak-collections: error

test/collection-test.js

@@ -102,7 +102,8 @@ describe("collection", function () {
 
             for (var i = 0; i < types.length; i++) {
                 // yes, it's silly to create functions in a loop, it's also a test
-                assert.exception(function () { // eslint-disable-line no-loop-func
+                // eslint-disable-next-line no-loop-func ie11/no-loop-func
+                assert.exception(function () {
                     this.collection.createStubInstance(types[i]);
                 });
             }

test/stub-test.js

@@ -2451,7 +2451,8 @@ describe("stub", function () {
 
             for (var i = 0; i < types.length; i++) {
                 // yes, it's silly to create functions in a loop, it's also a test
-                assert.exception(function () { // eslint-disable-line no-loop-func
+                // eslint-disable-next-line no-loop-func ie11/no-loop-func
+                assert.exception(function () {
                     createStubInstance(types[i]);
                 });
             }