Fix default HTTP client tests (Azure#321)

* Enable CORS

* Add empty line

* Add mock http proxy

* Fix few tests

* Remove axios workaround that was fixed in current release

* Fix all tests except download/upload progress

* Fix stream HTTP tests

* Delete testserver

* Remove reporting skipped tests as warning

* Remove .only

* Add nodeIt

* Fix Constants.ts

* Remove it.skip()

.scripts/checkEverything.ts

@@ -2,11 +2,8 @@ import { checkEverything } from "@ts-common/azure-js-dev-tools";
 import { checkConstantsVersion } from "./checkConstantsVersion";
 
 checkEverything({
-  checkForSkipCallsOptions: {
-    skipIsWarning: true
-  },
   additionalChecks: {
     name: "Constants.ts Version",
     check: checkConstantsVersion
   }
-});
+});

karma.conf.ts

@@ -49,9 +49,13 @@ module.exports = function (config: any) {
     concurrency: Infinity,
 
     customLaunchers: {
+      ChromeNoSecurity: {
+        base: "ChromeHeadless",
+        flags: ["--disable-web-security"]
+      },
       ChromeDebugging: {
         base: "Chrome",
-        flags: [`http://localhost:${defaults.port}/debug.html`, "--auto-open-devtools-for-tabs"]
+        flags: [`http://localhost:${defaults.port}/debug.html`, "--auto-open-devtools-for-tabs", "--disable-web-security"]
       }
     },
   });

lib/axiosHttpClient.ts

@@ -12,9 +12,7 @@ import { RestError } from "./restError";
 import { WebResource, HttpRequestBody } from "./webResource";
 import { ProxySettings } from "./serviceClient";
 
-const axiosClient = axios.create();
-// Workaround for https://github.com/axios/axios/issues/1158
-axiosClient.interceptors.request.use(config => ({ ...config, method: config.method && config.method.toUpperCase() as any }));
+export const axiosClient = axios.create();
 
 /**
  * A HttpClient implementation that uses axios to send HTTP requests.

lib/util/constants.ts

@@ -8,7 +8,7 @@ export const Constants = {
    * @const
    * @type {string}
    */
-  msRestVersion: "1.5.0",
+  msRestVersion: "1.5.1",
 
   /**
    * Specifies HTTP.

mocha.config.json

@@ -1,3 +1,3 @@
 {
   "reporterEnabled": "list, mocha-junit-reporter"
-}
+}

package.json

@@ -5,7 +5,7 @@
     "email": "azsdkteam@microsoft.com",
     "url": "https://github.com/Azure/ms-rest-js"
   },
-  "version": "1.5.0",
+  "version": "1.5.1",
   "description": "Isomorphic client Runtime for Typescript/node.js/browser javascript client libraries generated using AutoRest",
   "tags": [
     "isomorphic",
@@ -73,6 +73,7 @@
     "@types/xhr-mock": "^2.0.0",
     "@types/xml2js": "^0.4.3",
     "abortcontroller-polyfill": "^1.1.9",
+    "axios-mock-adapter": "^1.16.0",
     "chai": "^4.2.0",
     "express": "^4.16.3",
     "glob": "^7.1.2",
@@ -111,6 +112,7 @@
     "webpack": "^4.27.1",
     "webpack-cli": "^3.1.2",
     "webpack-dev-middleware": "^3.1.2",
+    "xhr-mock": "^2.4.1",
     "yarn": "^1.6.0"
   },
   "homepage": "https://github.com/Azure/ms-rest-js",
@@ -132,7 +134,7 @@
     "test": "run-p test:tslint test:unit test:karma",
     "test:tslint": "tslint -p . -c tslint.json --exclude \"./test/**/*.ts\"",
     "test:unit": "nyc mocha",
-    "test:karma": "npm run build:test-browser && node ./node_modules/karma/bin/karma start karma.conf.ts --single-run",
+    "test:karma": "npm run build:test-browser && node ./node_modules/karma/bin/karma start karma.conf.ts --browsers ChromeNoSecurity --single-run ",
     "test:karma:debug": "npm run build:test-browser && node ./node_modules/karma/bin/karma start karma.conf.ts  --log-level debug --browsers ChromeDebugging --debug --auto-watch",
     "publish-preview": "mocha --no-colors && shx rm -rf dist/test && node ./.scripts/publish",
     "local": "ts-node ./.scripts/local.ts",