Have tests pass eslint

JakeChampion · May 17, 2018 · 5d80368 · 5d80368
1 parent 3ccb1ce
commit 5d80368
Show file tree

Hide file tree

Showing 4 changed files with 22 additions and 4 deletions.
diff --git a/.eslintrc.json b/.eslintrc.json
@@ -14,5 +14,25 @@
   ],
   "rules": {
     "object-shorthand": "off"
-  }
+  },
+  "overrides": [
+    {
+      "files": ["test/*.js"],
+      "env": {
+        "browser": true,
+        "mocha": true
+      },
+      "globals": {
+        "assert": true,
+        "chai": true,
+        "FileReaderSync": true
+      }
+    },
+    {
+      "files": ["test/worker.js"],
+      "env": {
+        "worker": true
+      }
+    }
+  ]
 }
diff --git a/test/cjs.js b/test/cjs.js
@@ -1,5 +1,3 @@
-/* global describe, it, require */
-
 var isFunction = require('chai').assert.isFunction
 var WHATWGFetch = require('../')
 

diff --git a/test/mocha-phantomjs-hooks.js b/test/mocha-phantomjs-hooks.js
@@ -1,4 +1,3 @@
-/* globals exports */
 exports.beforeStart = function(context) {
   var originalResourceError = context.page.onResourceError
   context.page.onResourceError = function(resErr) {

diff --git a/test/test.js b/test/test.js
@@ -97,6 +97,7 @@ if (self.fetch) {
 var slice = Array.prototype.slice
 
 function featureDependent(testOrSuite, condition) {
+  // eslint-disable-next-line no-invalid-this
   (condition ? testOrSuite : testOrSuite.skip).apply(this, slice.call(arguments, 2))
 }