improvement: start work on cleaning up build process

also, revert devDependency version on axios to 0.18.0 instead of 0.19.0

Author: Weffe

.gitignore

@@ -1,3 +1,8 @@
 node_modules/
 *.log
-dist/
+dist/
+.DS_Store
+.rts2_cache_cjs
+.rts2_cache_es
+.rts2_cache_esm
+.rts2_cache_umd

package.json

@@ -3,23 +3,18 @@
   "author": "Rogelio Negrete (Weffe)",
   "version": "2.1.0",
   "description": "Adds api versioning for axios",
-  "main": "dist/lib/index.js",
-  "module": "dist/esm/index.js",
-  "jsnext:main": "dist/esm/index.js",
-  "types": "dist/lib/index.d.ts",
+  "main": "dist/index.js",
+  "module": "dist/axios-api-versioning.js",
+  "typings": "dist/index.d.ts",
   "sideEffects": false,
   "scripts": {
-    "prestart": "run-s clean",
-    "start": "tsc --watch",
-    "build": "run-s clean build:cjs build:esm build:types",
-    "build:cjs": "tsc",
-    "build:esm": "tsc -p tsconfig.esm.json",
-    "build:types": "tsc -p tsconfig.types.json",
+    "start": "tsdx watch",
+    "build": "tsdx build",
+    "test": "tsdx test",
     "sandbox": "tsc -p sandbox/tsconfig.sandbox.json",
     "clean": "rimraf dist/",
     "release": "standard-version",
-    "docs": "docsify serve docs -o --port 2065",
-    "test": "jest"
+    "docs": "docsify serve docs -o --port 2065"
   },
   "keywords": [
     "axios",
@@ -38,41 +33,27 @@
     "axios": "^0.18.0"
   },
   "devDependencies": {
-    "@types/jest": "^24.0.11",
-    "axios": "^0.19.0",
+    "@types/jest": "^24.0.15",
+    "axios": "0.18.0",
     "axios-mock-adapter": "^1.16.0",
     "docsify-cli": "^4.3.0",
     "http-status-codes": "^1.3.2",
-    "jest": "^24.1.0",
-    "npm-run-all": "^4.1.5",
-    "rimraf": "^2.6.3",
+    "husky": "^2.4.1",
+    "prettier": "^1.18.2",
+    "pretty-quick": "^1.11.1",
     "standard-version": "^6.0.1",
-    "ts-jest": "^24.0.2",
-    "typescript": "^3.3.3333"
-  },
-  "dependencies": {
-    "tslib": "^1.9.3"
+    "tsdx": "^0.7.1",
+    "tslib": "^1.10.0",
+    "typescript": "^3.5.2"
   },
   "files": [
     "dist",
     "README.md",
     "LICENSE"
   ],
-  "jest": {
-    "roots": [
-      "<rootDir>/src"
-    ],
-    "moduleFileExtensions": [
-      "ts",
-      "tsx",
-      "js",
-      "jsx",
-      "json",
-      "node"
-    ],
-    "transform": {
-      "^.+\\.ts$": "ts-jest"
-    },
-    "testRegex": "(/__tests__/.*|(\\.|/)(test|spec))\\.ts$"
+  "husky": {
+    "hooks": {
+      "pre-commit": "pretty-quick --staged"
+    }
   }
 }

src/__tests__/axios-api-versioning.mock.ts

@@ -0,0 +1,79 @@
+import axios from "axios";
+import MockAdapter from "axios-mock-adapter";
+import * as status from "http-status-codes";
+import { withVersioning } from "../axios-api-versioning";
+import { IWithVersioningConfig, VersioningStrategy } from "../types";
+import { AxiosInstanceWithVersioning } from "../types/axios";
+
+const test_url = "http://localhost:3000";
+const MOCK_RES = "hello_world";
+let mock: MockAdapter;
+let instance: AxiosInstanceWithVersioning;
+
+describe('Testing correct response config of "QueryString" strategy', () => {
+  let versioningConfig: IWithVersioningConfig = {
+    apiVersion: "1.0",
+    versioningStrategy: VersioningStrategy.QueryString
+  };
+
+  beforeAll(() => {
+    instance = withVersioning(axios, versioningConfig);
+    mock = new MockAdapter(instance);
+  });
+
+  test('it should have the "apiVersion" as a query param in the response config', async () => {
+    mock.onGet(test_url).reply(status.OK, MOCK_RES);
+
+    const res = await instance.get(test_url);
+    const { params } = res.config;
+
+    expect(params).toHaveProperty("api-version");
+    expect(params["api-version"]).toBe(versioningConfig.apiVersion);
+  });
+});
+
+describe('Testing correct response config of "MediaType" strategy', () => {
+  let versioningConfig: IWithVersioningConfig = {
+    apiVersion: "1.0",
+    versioningStrategy: VersioningStrategy.MediaType
+  };
+
+  beforeAll(() => {
+    instance = withVersioning(axios, versioningConfig);
+    mock = new MockAdapter(instance);
+  });
+
+  test('it should have the "apiVersion" as an accept-param in the Accept header', async () => {
+    mock.onGet(test_url).reply(status.OK, MOCK_RES);
+
+    const res = await instance.get(test_url);
+    const { headers } = res.config;
+
+    expect(headers).toHaveProperty("Accept");
+    expect(headers["Accept"]).toMatch(`v=${versioningConfig.apiVersion}`);
+  });
+});
+
+describe('Testing correct response config of "UrlPath" strategy', () => {
+  let versioningConfig: IWithVersioningConfig = {
+    apiVersion: "1",
+    versioningStrategy: VersioningStrategy.UrlPath
+  };
+
+  const blank_test_url = test_url + "/v{apiVersion}";
+  const versioned_test_url = test_url + `/v${versioningConfig.apiVersion}`;
+
+  beforeAll(() => {
+    instance = withVersioning(axios, versioningConfig);
+    mock = new MockAdapter(instance);
+  });
+
+  test('it should have the "apiVersion" as a url param in the url', async () => {
+    mock.onGet(versioned_test_url).reply(status.OK, MOCK_RES);
+
+    const res = await instance.get(blank_test_url);
+    const { url } = res.config;
+
+    expect(url).toBe(versioned_test_url);
+  });
+});