fix firestore node tests

.changeset/plenty-trains-switch.md

@@ -0,0 +1,6 @@
+---
+"@firebase/firestore-compat": patch
+"@firebase/firestore": patch
+---
+
+fix firestore node tests

config/mocharc.node.js

@@ -34,4 +34,9 @@ if (process.env.CI) {
   config.reporter = 'min';
 }
 
+// Firestore uses babel to compile tests in Nodejs
+if (process.env.NO_TS_NODE) {
+  delete config.require;
+}
+
 module.exports = config;

package.json

@@ -62,9 +62,12 @@
     "repo-scripts/*"
   ],
   "devDependencies": {
-    "@babel/core": "7.15.5",
-    "@babel/plugin-transform-modules-commonjs": "7.15.4",
-    "@babel/preset-env": "7.15.6",
+    "@babel/core": "7.16.0",
+    "@babel/plugin-transform-modules-commonjs": "7.16.0",
+    "@babel/preset-env": "7.16.0",
+    "@babel/register": "7.16.0",
+    "@babel/preset-typescript": "7.16.0",
+    "babel-plugin-transform-import-meta": "2.1.0",
     "@changesets/changelog-github": "0.4.1",
     "@changesets/cli": "2.17.0",
     "@types/chai": "4.2.22",

packages/firestore-compat/src/config.ts

@@ -84,7 +84,7 @@ export function configureForFirebase(
         const firestoreExp = container.getProvider('firestore').getImmediate()!;
         return firestoreFactory(app, firestoreExp);
       },
-      ComponentType.PUBLIC
+      'PUBLIC' as ComponentType.PUBLIC
     ).setServiceProps({ ...firestoreNamespace })
   );
 }

packages/firestore-compat/src/index.ts

@@ -17,7 +17,7 @@
 
 // eslint-disable-next-line import/no-extraneous-dependencies
 import firebase from '@firebase/app-compat';
-import { FirebaseNamespace } from '@firebase/app-types';
+import type { FirebaseNamespace } from '@firebase/app-types';
 import * as types from '@firebase/firestore-types';
 
 import { name, version } from '../package.json';

packages/firestore/babel-register.js

@@ -0,0 +1 @@
+require("@babel/register")({ extensions: ['.js', '.ts'] });

packages/firestore/babel.config.json

@@ -0,0 +1,7 @@
+{
+    "presets": [
+        "@babel/preset-typescript",
+        ["@babel/preset-env", {"targets": {"node": "10"}, "modules": "cjs"}]
+    ],
+    "plugins": ["babel-plugin-transform-import-meta"]
+}

packages/firestore/lite/register.ts

@@ -50,7 +50,7 @@ export function registerFirestore(): void {
         }
         return firestoreInstance;
       },
-      ComponentType.PUBLIC
+      'PUBLIC' as ComponentType.PUBLIC
     )
   );
   // RUNTIME_ENV and BUILD_TARGET are replaced by real values during the compilation

packages/firestore/package.json

@@ -34,7 +34,7 @@
     "test:browser": "karma start --single-run",
     "test:browser:debug": "karma start --browsers=Chrome --auto-watch",
     "test:node": "node ./scripts/run-tests.js  --main=test/register.ts  --emulator 'test/{,!(browser|lite)/**/}*.test.ts'",
-    "test:node:prod": "ts-node-script ./scripts/run-tests.ts --main=test/register.ts 'test/{,!(browser|lite)/**/}*.test.ts'",
+    "test:node:prod": "node ./scripts/run-tests.js --main=test/register.ts 'test/{,!(browser|lite)/**/}*.test.ts'",
     "test:node:persistence": "node ./scripts/run-tests.js  --main=test/register.ts --persistence --emulator 'test/{,!(browser|lite)/**/}*.test.ts'",
     "test:node:persistence:prod": "node ./scripts/run-tests.js --main=test/register.ts --persistence 'test/{,!(browser|lite)/**/}*.test.ts'",
     "test:travis": "ts-node --compiler-options='{\"module\":\"commonjs\"}' ../../scripts/emulator-testing/firestore-test-runner.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 path_1=require("path");var child_process_promise_1=require("child-process-promise");var yargs=require("yargs");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=["--reporter","lcovonly",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 path_1=require("path");var child_process_promise_1=require("child-process-promise");var yargs=require("yargs");var argv=yargs.options({main:{type:"string",demandOption:true},platform:{type:"string",default:"node"},emulator:{type:"boolean"},persistence:{type:"boolean"}}).parseSync();var nyc=path_1.resolve(__dirname,"../../../node_modules/.bin/nyc");var mocha=path_1.resolve(__dirname,"../../../node_modules/.bin/mocha");var babel=path_1.resolve(__dirname,"../babel-register.js");process.env.NO_TS_NODE="true";process.env.TEST_PLATFORM=argv.platform;var args=["--reporter","lcovonly",mocha,"--require",babel,"--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

@@ -39,17 +39,18 @@ const argv = yargs.options({
 
 const nyc = resolve(__dirname, '../../../node_modules/.bin/nyc');
 const mocha = resolve(__dirname, '../../../node_modules/.bin/mocha');
+const babel = resolve(__dirname, '../babel-register.js');
 
-process.env.TS_NODE_CACHE = 'NO';
-process.env.TS_NODE_COMPILER_OPTIONS = '{"module":"commonjs"}';
+// used in '../../config/mocharc.node.js' to disable ts-node
+process.env.NO_TS_NODE = "true";
 process.env.TEST_PLATFORM = argv.platform;
 
 let args = [
   '--reporter',
   'lcovonly',
   mocha,
   '--require',
-  'ts-node/register',
+  babel,
   '--require',
   argv.main,
   '--config',

packages/firestore/src/platform/node/grpc_connection.ts

@@ -40,9 +40,9 @@ import { Deferred } from '../../util/promise';
 
 // This is a hack fix for Node ES modules to use `require`.
 // @ts-ignore To avoid using `--module es2020` flag.
-const require = module.createRequire(import.meta.url);
+const requireInESM = module.createRequire(import.meta.url);
 // eslint-disable-next-line @typescript-eslint/no-require-imports
-const { version: grpcVersion } = require('@grpc/grpc-js/package.json');
+const { version: grpcVersion } = requireInESM('@grpc/grpc-js/package.json');
 
 const LOG_TAG = 'Connection';
 const X_GOOG_API_CLIENT_VALUE = `gl-node/${process.versions.node} fire/${SDK_VERSION} grpc/${grpcVersion}`;

packages/firestore/src/platform/node/load_protos.ts

@@ -20,8 +20,8 @@ import { fileURLToPath } from 'url';
 
 // __filename and __dirname globals are unavailable in ES modules
 // @ts-ignore To avoid using `--module es2020` flag.
-const __filename = fileURLToPath(import.meta.url);
-const __dirname = dirname(__filename);
+const __filenameInESM = fileURLToPath(import.meta.url);
+const __dirnameInESM = dirname(__filenameInESM);
 
 import { loadPackageDefinition, GrpcObject } from '@grpc/grpc-js';
 import { loadSync } from '@grpc/proto-loader';
@@ -44,7 +44,7 @@ export const protoLoaderOptions: IConversionOptions = {
  */
 export function loadProtos(): GrpcObject {
   const root = resolve(
-    __dirname,
+    __dirnameInESM,
     process.env.FIRESTORE_PROTO_ROOT || '../../protos'
   );
   const firestoreProtoFile = join(root, 'google/firestore/v1/firestore.proto');

packages/firestore/src/register.ts

@@ -49,7 +49,7 @@ export function registerFirestore(
         firestoreInstance._setSettings(settings);
         return firestoreInstance;
       },
-      ComponentType.PUBLIC
+      'PUBLIC' as ComponentType.PUBLIC
     )
   );
   registerVersion(name, version, variant);

packages/firestore/test/unit/specs/describe_spec.ts

@@ -15,7 +15,7 @@
  * limitations under the License.
  */
 
-import * as stringify from 'json-stable-stringify';
+import stringify from 'json-stable-stringify';
 import { ExclusiveTestFunction, PendingTestFunction } from 'mocha';
 
 import { queryEquals, QueryImpl } from '../../../src/core/query';

packages/firestore/test/util/node_persistence.ts

@@ -19,7 +19,7 @@ import * as fs from 'fs';
 import * as os from 'os';
 
 // @ts-ignore: There are no types for indexeddbshim.
-import * as registerIndexedDBShim from 'indexeddbshim';
+import registerIndexedDBShim from 'indexeddbshim';
 
 import { FakeWindow, SharedFakeWebStorage } from './test_platform';