test: use 'node' rather than 'jsdom' as the jest testEnvironment

Author: dpopp07

package.json

@@ -118,10 +118,10 @@
     "prepublishOnly": "npm run build",
     "test-integration": "mocha test/integration",
     "test-unit": "jest test/unit/",
-    "test-unit-travis": "jest --silent --runInBand test/unit/",
+    "test-unit-travis": "jest --silent --runInBand test/unit/ && codecov",
     "test": "npm run test-unit && npm run test-integration && nyc report --reporter=html",
     "test-travis": "npm run test-unit-travis && nyc mocha test/integration/ --grep @slow --invert && nyc report --reporter=html",
-    "report-coverage": "nyc report --reporter=text-lcov > coverage.lcov && ./node_modules/.bin/codecov",
+    "report-coverage": "nyc report --reporter=text-lcov > coverage.lcov && codecov",
     "watch-doc": "nodemon --watch ./ --ext js,tmpl,json --ignore dist/ --ignore doc/ --ignore test/ --ignore examples/ --exec npm run doc",
     "watch": "npm run test-unit -- --watch",
     "watchify": "watchify index.js --standalone Watson --outfile dist/watson.js --debug --verbose"
@@ -146,6 +146,7 @@
   },
   "jest": {
     "collectCoverage": true,
-    "coverageDirectory": "./coverage/"
+    "coverageDirectory": "./coverage/",
+    "testEnvironment": "node"
   }
 }

test/unit/requestWrapper.test.js

@@ -1,11 +1,3 @@
-/**
- * @jest-environment node
- */
-
-// the above comment instructs jest to use a node, rather than browser, environment
-// the sendRequest method uses the environment to determine if running in the browser
-// and uses special handling if it is. the tests will fail without the above comment
-
 'use strict';
 
 const sendRequest = require('../../lib/requestwrapper').sendRequest;