CI

Author: schmidt-sebastian

packages/firestore/exp/test/bootstrap.ts

@@ -24,6 +24,8 @@ import '../register';
  * https://github.com/webpack-contrib/karma-webpack#alternative-usage
  */
 
+process.env.TEST_PLATFORM = 'browser';
+
 // 'context()' definition requires additional dependency on webpack-env package.
 // eslint-disable-next-line @typescript-eslint/no-explicit-any
 const testsContext = (require as any).context(

packages/firestore/lite/test/bootstrap.ts

@@ -24,6 +24,8 @@ import '../register';
  * https://github.com/webpack-contrib/karma-webpack#alternative-usage
  */
 
+process.env.TEST_PLATFORM = 'node-lite';
+
 // 'context()' definition requires additional dependency on webpack-env package.
 // eslint-disable-next-line @typescript-eslint/no-explicit-any
 const testsContext = (require as any).context('.', true, /^.*\.test.*$/);

packages/firestore/package.json

@@ -24,7 +24,7 @@
     "gendeps:exp": "../../scripts/exp/extract-deps.sh --types ./exp-types/index.d.ts --bundle ./dist/exp/tmp.js --output ./exp/dependencies.json",
     "pregendeps:lite": "node scripts/build-bundle.js --input ./lite/index.ts --output ./dist/lite/tmp.js",
     "gendeps:lite": "../../scripts/exp/extract-deps.sh --types ./lite-types/index.d.ts --bundle ./dist/lite/tmp.js --output ./lite/dependencies.json",
-    "test:lite": "node ./scripts/run-tests.js --emulator --main=lite/index.ts  'lite/test/**/*.test.ts'",
+    "test:lite": "node ./scripts/run-tests.js --emulator  --platform node_lite --main=lite/index.ts 'lite/test/**/*.test.ts'",
     "test:lite:browser": "karma start --single-run --lite",
     "test:lite:browser:debug": "karma start --single-run --lite --auto-watch",
     "test:exp": "node ./scripts/run-tests.js --emulator --main=exp/index.ts test/integration/api/*.test.ts",

packages/firestore/scripts/run-tests.js

@@ -14,4 +14,4 @@
  * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
  * See the License for the specific language governing permissions and
  * limitations under the License.
- */exports.__esModule=true;var yargs=require("yargs");var path_1=require("path");var child_process_promise_1=require("child-process-promise");var argv=yargs.options({main:{type:"string",demandOption:true},emulator:{type:"boolean"},persistence:{type:"boolean"}}).argv;var nyc=path_1.resolve(__dirname,"../../../node_modules/.bin/nyc");var mocha=path_1.resolve(__dirname,"../../../node_modules/.bin/mocha");process.env.TS_NODE_CACHE="NO",process.env.TS_NODE_COMPILER_OPTIONS='{"module":"commonjs"}';var args=[mocha,"--require","ts-node/register","--require",argv.main,"--config","../../config/mocharc.node.js"];if(argv.emulator){process.env.FIRESTORE_EMULATOR_PORT="8080";process.env.FIRESTORE_EMULATOR_PROJECT_ID="test-emulator"}if(argv.persistence){process.env.USE_MOCK_PERSISTENCE="YES";args.push("--require","test/util/node_persistence.ts")}args=args.concat(argv._);var childProcess=child_process_promise_1.spawn(nyc,args,{stdio:"inherit",cwd:process.cwd()}).childProcess;process.once("exit",(function(){return childProcess.kill()}));process.once("SIGINT",(function(){return childProcess.kill("SIGINT")}));process.once("SIGTERM",(function(){return childProcess.kill("SIGTERM")}));
+ */exports.__esModule=true;var yargs=require("yargs");var path_1=require("path");var child_process_promise_1=require("child-process-promise");var argv=yargs.options({main:{type:"string",demandOption:true},platform:{type:"string",default:"node"},emulator:{type:"boolean"},persistence:{type:"boolean"}}).argv;var nyc=path_1.resolve(__dirname,"../../../node_modules/.bin/nyc");var mocha=path_1.resolve(__dirname,"../../../node_modules/.bin/mocha");process.env.TS_NODE_CACHE="NO";process.env.TS_NODE_COMPILER_OPTIONS='{"module":"commonjs"}';process.env.TEST_PLATFORM=argv.platform;var args=[mocha,"--require","ts-node/register","--require",argv.main,"--config","../../config/mocharc.node.js"];if(argv.emulator){process.env.FIRESTORE_EMULATOR_PORT="8080";process.env.FIRESTORE_EMULATOR_PROJECT_ID="test-emulator"}if(argv.persistence){process.env.USE_MOCK_PERSISTENCE="YES";args.push("--require","test/util/node_persistence.ts")}args=args.concat(argv._);var childProcess=child_process_promise_1.spawn(nyc,args,{stdio:"inherit",cwd:process.cwd()}).childProcess;process.once("exit",(function(){return childProcess.kill()}));process.once("SIGINT",(function(){return childProcess.kill("SIGINT")}));process.once("SIGTERM",(function(){return childProcess.kill("SIGTERM")}));

packages/firestore/scripts/run-tests.ts

@@ -24,6 +24,10 @@ const argv = yargs.options({
     type: 'string',
     demandOption: true
   },
+  platform: {
+    type: 'string',
+    default: 'node'
+  },
   emulator: {
     type: 'boolean'
   },
@@ -37,6 +41,7 @@ const mocha = resolve(__dirname, '../../../node_modules/.bin/mocha');
 
 process.env.TS_NODE_CACHE = 'NO';
 process.env.TS_NODE_COMPILER_OPTIONS = '{"module":"commonjs"}';
+process.env.TEST_PLATFORM = argv.platform;
 
 let args = [
   mocha,

packages/firestore/src/platform/base64.ts

@@ -15,41 +15,21 @@
  * limitations under the License.
  */
 
-import { isNode, isReactNative } from '@firebase/util';
-
-import * as node from './node/base64';
-import * as rn from './rn/base64';
-import * as browser from './browser/base64';
+// This file is only used under ts-node.
+// eslint-disable-next-line @typescript-eslint/no-require-imports
+const platform = require(`${process.env.TEST_PLATFORM}/base64`);
 
 /** Converts a Base64 encoded string to a binary string. */
 export function decodeBase64(encoded: string): string {
-  if (isNode()) {
-    return node.decodeBase64(encoded);
-  } else if (isReactNative()) {
-    return rn.decodeBase64(encoded);
-  } else {
-    return browser.decodeBase64(encoded);
-  }
+  return platform.decodeBase64(encoded);
 }
 
 /** Converts a binary string to a Base64 encoded string. */
 export function encodeBase64(raw: string): string {
-  if (isNode()) {
-    return node.encodeBase64(raw);
-  } else if (isReactNative()) {
-    return rn.encodeBase64(raw);
-  } else {
-    return browser.encodeBase64(raw);
-  }
+  return platform.encodeBase64(raw);
 }
 
 /** True if and only if the Base64 conversion functions are available. */
 export function isBase64Available(): boolean {
-  if (isNode()) {
-    return node.isBase64Available();
-  } else if (isReactNative()) {
-    return rn.isBase64Available();
-  } else {
-    return browser.isBase64Available();
-  }
+  return platform.isBase64Available();
 }

packages/firestore/src/platform/connection.ts

@@ -14,45 +14,18 @@
  * See the License for the specific language governing permissions and
  * limitations under the License.
  */
+import {ConnectivityMonitor} from "../remote/connectivity_monitor";
+import {DatabaseInfo} from "../core/database_info";
+import {Connection} from "../remote/connection";
 
-// This file is only used in when the client is run under ts-node
-// eslint-disable-next-line import/no-extraneous-dependencies
-import { _components } from '@firebase/app-exp';
-
-import { isNode, isReactNative } from '@firebase/util';
-import { ConnectivityMonitor } from '../remote/connectivity_monitor';
-import { DatabaseInfo } from '../core/database_info';
-import { Connection } from '../remote/connection';
-import * as node from './node/connection';
-import * as nodeLite from './node_lite/connection';
-import * as rn from './rn/connection';
-import * as browser from './browser/connection';
-import * as browserLite from './browser_lite/connection';
-
-function isLite(): boolean {
-  return _components.has('firestore/lite');
-}
+// This file is only used under ts-node.
+// eslint-disable-next-line @typescript-eslint/no-require-imports
+const platform = require(`${process.env.TEST_PLATFORM}/base64`);
 
 export function newConnectivityMonitor(): ConnectivityMonitor {
-  if (isNode()) {
-    return node.newConnectivityMonitor();
-  } else if (isReactNative()) {
-    return rn.newConnectivityMonitor();
-  } else {
-    return browser.newConnectivityMonitor();
-  }
+  return platform.newConnectivityMonitor();
 }
 
 export function newConnection(databaseInfo: DatabaseInfo): Promise<Connection> {
-  if (isNode()) {
-    return isLite()
-      ? nodeLite.newConnection(databaseInfo)
-      : node.newConnection(databaseInfo);
-  } else if (isReactNative()) {
-    return rn.newConnection(databaseInfo);
-  } else {
-    return isLite()
-      ? browserLite.newConnection(databaseInfo)
-      : browser.newConnection(databaseInfo);
-  }
+  return platform.newConnection(databaseInfo);
 }

packages/firestore/src/platform/dom.ts

@@ -15,29 +15,16 @@
  * limitations under the License.
  */
 
-import { isNode, isReactNative } from '@firebase/util';
-import * as node from './node/dom';
-import * as rn from './rn/dom';
-import * as browser from './browser/dom';
+// This file is only used under ts-node.
+// eslint-disable-next-line @typescript-eslint/no-require-imports
+const platform = require(`${process.env.TEST_PLATFORM}/base64`);
 
 /** The Platform's 'window' implementation or null if not available. */
 export function getWindow(): Window | null {
-  if (isNode()) {
-    return node.getWindow();
-  } else if (isReactNative()) {
-    return rn.getWindow();
-  } else {
-    return browser.getWindow();
-  }
+  return platform.getWindow();
 }
 
 /** The Platform's 'document' implementation or null if not available. */
 export function getDocument(): Document | null {
-  if (isNode()) {
-    return node.getDocument();
-  } else if (isReactNative()) {
-    return rn.getDocument();
-  } else {
-    return browser.getDocument();
-  }
+  return platform.getDocument();
 }

packages/firestore/src/platform/format_json.ts

@@ -15,18 +15,11 @@
  * limitations under the License.
  */
 
-import { isNode, isReactNative } from '@firebase/util';
-import * as node from './node/format_json';
-import * as rn from './rn/format_json';
-import * as browser from './browser/format_json';
+// This file is only used under ts-node.
+// eslint-disable-next-line @typescript-eslint/no-require-imports
+const platform = require(`${process.env.TEST_PLATFORM}/base64`);
 
 /** Formats an object as a JSON string, suitable for logging. */
 export function formatJSON(value: unknown): string {
-  if (isNode()) {
-    return node.formatJSON(value);
-  } else if (isReactNative()) {
-    return rn.formatJSON(value);
-  } else {
-    return browser.formatJSON(value);
-  }
+  return platform.formatJSON(value);
 }

packages/firestore/src/platform/random_bytes.ts

@@ -15,22 +15,15 @@
  * limitations under the License.
  */
 
-import { isNode, isReactNative } from '@firebase/util';
-import * as node from './node/random_bytes';
-import * as rn from './rn/random_bytes';
-import * as browser from './browser/random_bytes';
+// This file is only used under ts-node.
+// eslint-disable-next-line @typescript-eslint/no-require-imports
+const platform = require(`${process.env.TEST_PLATFORM}/base64`);
 
 /**
  * Generates `nBytes` of random bytes.
  *
  * If `nBytes < 0` , an error will be thrown.
  */
 export function randomBytes(nBytes: number): Uint8Array {
-  if (isNode()) {
-    return node.randomBytes(nBytes);
-  } else if (isReactNative()) {
-    return rn.randomBytes(nBytes);
-  } else {
-    return browser.randomBytes(nBytes);
-  }
+    return platform.randomBytes(nBytes);
 }

packages/firestore/src/platform/serializer.ts

@@ -14,34 +14,13 @@
  * See the License for the specific language governing permissions and
  * limitations under the License.
  */
+import {DatabaseId} from "../core/database_info";
+import {JsonProtoSerializer} from "../remote/serializer";
 
-// This file is only used in when the client is run under ts-node
-// eslint-disable-next-line import/no-extraneous-dependencies
-import { _components } from '@firebase/app-exp';
-
-import { isNode, isReactNative } from '@firebase/util';
-import * as node from './node/serializer';
-import * as nodeLite from './node_lite/serializer';
-import * as rn from './rn/serializer';
-import * as browser from './browser/serializer';
-import * as browserLite from './browser_lite/serializer';
-import { DatabaseId } from '../core/database_info';
-import { JsonProtoSerializer } from '../remote/serializer';
-
-function isLite(): boolean {
-  return _components.has('firestore/lite');
-}
+// This file is only used under ts-node.
+// eslint-disable-next-line @typescript-eslint/no-require-imports
+const platform = require(`${process.env.TEST_PLATFORM}/serializer`);
 
 export function newSerializer(databaseId: DatabaseId): JsonProtoSerializer {
-  if (isNode()) {
-    return isLite()
-      ? nodeLite.newSerializer(databaseId)
-      : node.newSerializer(databaseId);
-  } else if (isReactNative()) {
-    return rn.newSerializer(databaseId);
-  } else {
-    return isLite()
-      ? browserLite.newSerializer(databaseId)
-      : browser.newSerializer(databaseId);
-  }
+    return platform.newSerializer(databaseId);
 }

packages/firestore/test/integration/bootstrap.ts

@@ -24,6 +24,8 @@ import '../../index';
  * https://github.com/webpack-contrib/karma-webpack#alternative-usage
  */
 
+process.env.TEST_PLATFORM = 'browser';
+
 // 'context()' definition requires additional dependency on webpack-env package.
 // eslint-disable-next-line @typescript-eslint/no-explicit-any
 const testsContext = (require as any).context(

packages/firestore/test/unit/bootstrap.ts

@@ -22,6 +22,8 @@
  * https://github.com/webpack-contrib/karma-webpack#alternative-usage
  */
 
+process.env.TEST_PLATFORM = 'browser';
+
 // 'context()' definition requires additional dependency on webpack-env package.
 // eslint-disable-next-line @typescript-eslint/no-explicit-any
 const testsContext = (require as any).context(