fix(polymorphic): Split libs into node and browser env, update webpac…

…k config

* fix(polymorphic): Split libs into node and browser env + update webpack config

* test(polymorphic): run units on bundled modules instead of on clean typescript - support for node an

* refactor(cleanup): cleanup console logs, package

* fix(cloud): Remove sessionToken in inapp browser is enabled

jest.config.js

@@ -2,35 +2,24 @@ module.exports = {
   name: 'filestack-js',
   collectCoverage: true,
   clearMocks: true,
-  projects: [{
+  projects: [
+  {
     displayName: 'Common',
     clearMocks: true,
-    testMatch: ['<rootDir>/src/**/*.spec.ts'],
+    testMatch: ['<rootDir>/build/main/**/*.spec.js', '<rootDir>/build/main/**/*.spec.node.js'],
     testEnvironment: 'node',
-    moduleFileExtensions: ['ts', 'tsx', 'js', 'jsx', 'json', 'node'],
-    transform: {
-      '^.+\\.tsx?$': 'ts-jest'
-    },
-  }, {
-    displayName: 'Node',
-    clearMocks: true,
-    testMatch: ['<rootDir>/src/**/*.spec.ts'],
-    moduleFileExtensions: ['ts', 'tsx', 'js', 'jsx', 'json', 'node'],
-    testMatch: ['<rootDir>/src/**/*.spec.node.ts'],
-    testEnvironment: 'node',
-    transform: {
-      '^.+\\.tsx?$': 'ts-jest'
-    },
+    moduleFileExtensions: ['js'],
   }, {
     displayName: 'Browser',
     testMatch: ['<rootDir>/src/**/*.browser.spec.ts'],
     clearMocks: true,
     testEnvironment: 'jsdom',
     setupFiles: ['jest-localstorage-mock'],
-    testMatch: ['<rootDir>/src/**/*.spec.browser.ts'],
-    moduleFileExtensions: ['ts', 'tsx', 'js', 'jsx', 'json', 'node'],
-    transform: {
-      '^.+\\.tsx?$': 'ts-jest'
-    },
-  }]
+    testMatch: [ '<rootDir>/build/main/**/*.spec.browser.js'],
+    moduleFileExtensions: ['js'],
+    moduleNameMapper: {
+      "\(.*)\\.node": "$1.browser",
+    }
+  }
+]
 };

package.json

@@ -22,7 +22,7 @@
     "webpack:profile": "npx webpack --profile --json > .stats.json",
     "webpack:analyze": "npm run webpack:profile && npx webpack-bundle-analyzer .stats.json build/browser/",
     "lint": "tslint -p tsconfig.tslint.json",
-    "test": "npm run lint && jest",
+    "test": "npm run lint && npm run build && jest",
     "test:watch": "npm t -- --watch",
     "test:codecov": "npx codecov",
     "docs": "trash build/docs && typedoc src",
@@ -37,7 +37,7 @@
     "node": ">=8"
   },
   "dependencies": {
-    "@sentry/minimal": "^5.5.0",
+    "@sentry/minimal": "^5.11.1",
     "abab": "^2.0.0",
     "axios": "0.18.1",
     "debug": "^4.1.1",
@@ -88,10 +88,9 @@
     "prettier-tslint": "^0.4.2",
     "standard-version": "^6.0.1",
     "trash-cli": "^3.0.0",
-    "ts-jest": "^24.0.2",
-    "tslint": "^5.18.0",
+    "tslint": "^6.0.0",
     "tslint-config-semistandard": "^8.0.1",
-    "typedoc": "^0.15.1",
+    "typedoc": "^0.16.8",
     "typescript": "^3.5.3",
     "validate-commit-msg": "^2.14.0",
     "webpack": "^4.35.3",

src/lib/api/cloud.ts

@@ -38,8 +38,32 @@ export const CALLBACK_URL_KEY = 'fs-tab';
 export class CloudClient {
   session: Session;
   cloudApiUrl: string;
+
+  /**
+   * Returns flag if token should be cached in local storage
+   *
+   * @private
+   * @type {boolean}
+   * @memberof CloudClient
+   */
   private cache: boolean = false;
+
+  /**
+   * Token returned from api for accessing clouds
+   *
+   * @private
+   * @type {string}
+   * @memberof CloudClient
+   */
   private _token: string;
+
+  /**
+   * Flag for in-app browser setup
+   * (in-app browsers are not supporting new window so we need to save token to session cache)
+   *
+   * @private
+   * @memberof CloudClient
+   */
   private _isInAppBrowser = false;
 
   constructor(session: Session, options?: ClientOptions) {
@@ -193,9 +217,15 @@ export class CloudClient {
 
     if (name) {
       payload.clouds = { [name]: {} };
-    } else if (this.cache) {
-      // No name means logout of ALL clouds. Clear local session.
-      localStorage.removeItem(PICKER_KEY);
+    } else {
+      if (this.cache) {
+        // No name means logout of ALL clouds. Clear local session.
+        localStorage.removeItem(PICKER_KEY);
+      }
+
+      if (this._isInAppBrowser) {
+        sessionStorage.removeItem(PICKER_KEY);
+      }
     }
 
     return requestWithSource()

src/lib/api/request.spec.ts

@@ -19,7 +19,7 @@ import axios from 'axios';
 import { requestWithSource, postWithRetry } from './request';
 import * as nock from 'nock';
 
-const v = require('../../../package.json').version;
+const v = require('../../../../package.json').version;
 
 const testHost = 'https://test.com';