fix: drop browser global whitelisting from test harnesses (#1383)

Author: shockey

.eslintrc

@@ -1,7 +1,6 @@
 {
   "extends": "eslint-config-airbnb-base",
   "env": {
-    "browser": true,
     "node": true
   },
   "rules": {

package.json

@@ -97,9 +97,6 @@
     "utfstring": "^2.0.0"
   },
   "jest": {
-    "globals": {
-      "navigator": true
-    },
     "testEnvironment": "node",
     "testMatch": [
       "**/test/*.js?(x)",

src/http.js

@@ -134,14 +134,19 @@ export function serializeHeaders(headers = {}) {
   return obj
 }
 
-export function isFile(obj, navigatorObj = navigator) {
+export function isFile(obj, navigatorObj) {
+  if (!navigatorObj && typeof navigator !== 'undefined') {
+    // eslint-disable-next-line no-undef
+    navigatorObj = navigator
+  }
   if (navigatorObj && navigatorObj.product === 'ReactNative') {
     if (obj && typeof obj === 'object' && typeof obj.uri === 'string') {
       return true
     }
     return false
   }
   if (typeof File !== 'undefined') {
+    // eslint-disable-next-line no-undef
     return obj instanceof File
   }
   return obj !== null && typeof obj === 'object' && typeof obj.pipe === 'function'

test/http.js

@@ -436,7 +436,8 @@ describe('http', () => {
     const mockReactNativeNavigator = {
       product: 'ReactNative'
     }
-    const browserFile = new File()
+
+    const browserFile = new global.File()
     const reactNativeFileObject = {
       uri: '/mock/path'
     }