Setting up firestore integration tests

Author: hsubox76

config/karma.saucelabs.js

@@ -31,10 +31,10 @@ if (!testConfigFile) {
  */
 const browserMap = {
   // Desktop
-  Chrome_Windows: seleniumLauncher('chrome', 'Windows 10', 'latest'),
-  Firefox_Windows: seleniumLauncher('firefox', 'Windows 10', 'latest'),
-  Safari_macOS: seleniumLauncher('safari', 'macOS 10.13', 'latest'),
-  Edge_Windows: seleniumLauncher('MicrosoftEdge', 'Windows 10', 'latest'),
+  // Chrome_Windows: seleniumLauncher('chrome', 'Windows 10', 'latest'),
+  // Firefox_Windows: seleniumLauncher('firefox', 'Windows 10', 'latest'),
+  // Safari_macOS: seleniumLauncher('safari', 'macOS 10.13', 'latest'),
+  // Edge_Windows: seleniumLauncher('MicrosoftEdge', 'Windows 10', 'latest'),
   IE_Windows: seleniumLauncher('internet explorer', 'Windows 10', 'latest')
 
   // Mobile

integration/firestore/karma.conf.js

@@ -20,8 +20,8 @@ const path = require('path');
 const karmaBase = require('../../config/karma.base');
 
 const files = [
-  `${path.dirname(require.resolve('firebase'))}/firebase.js`,
-  `${path.dirname(require.resolve('firebase'))}/firebase-firestore.js`,
+  // `${path.dirname(require.resolve('firebase'))}/firebase.js`,
+  // `${path.dirname(require.resolve('firebase'))}/firebase-firestore.js`,
   './dist/test-harness.js'
 ];
 

package.json

@@ -40,7 +40,6 @@
     "test:changed": "node scripts/run_changed.js",
     "test:setup": "node tools/config.js",
     "test:saucelabs": "node scripts/run_saucelabs.js",
-    "test:saucelabs:single": "karma start config/karma.saucelabs.js --single-run",
     "docgen:js": "node scripts/docgen/generate-docs.js --api js",
     "docgen:node": "node scripts/docgen/generate-docs.js --api node",
     "docgen": "yarn docgen:js; yarn docgen:node",

packages/firestore/karma.conf.js

@@ -42,6 +42,7 @@ module.exports = function(config) {
  * --unit and --integration command-line arguments.
  */
 function getTestFiles(argv) {
+  console.log(argv);
   const unitTests = 'test/unit/bootstrap.ts';
   const integrationTests = 'test/integration/bootstrap.ts';
   if (argv.unit) {

scripts/run_saucelabs.js

@@ -39,13 +39,12 @@ const testFiles = configFiles.length
   : glob
       .sync(`{packages,integration}/*/karma.conf.js`)
       // Automated tests in integration/firestore are currently disabled.
-      .filter(name => !name.includes('integration/firestore'));
+      // .filter(name => !name.includes('integration/firestore'));
 
 // Get CI build number or generate one if running locally.
 const buildNumber =
   process.env.TRAVIS_BUILD_NUMBER ||
-  // GitHub Actions does not have a build number, but the feature has been requested.
-  process.env.GITHUB_SHA ||
+  process.env.GITHUB_RUN_ID ||
   `local_${process.env.USER}_${new Date().getTime()}`;
 
 /**
@@ -67,26 +66,36 @@ async function runTest(testFile) {
       });
     }
   }
+  if (testFile.includes('integration/firestore')) {
+    let exitCode = 0;
+    await spawn('yarn', ['--cwd', 'integration/firestore', 'build:memory'], { stdio: 'inherit' });
+    await runKarma(testFile);
+    await spawn('yarn', ['--cwd', 'integration/firestore', 'build:persistence'], { stdio: 'inherit' });
+    return runKarma(testFile);
+  }
+  return runKarma(testFile);
+}
 
-  const promise = spawn('yarn', [
-    'test:saucelabs:single',
+async function runKarma(testFile) {
+  const karmaArgs = [
+    'karma',
+    'start',
+    'config/karma.saucelabs.js',
+    '--single-run',
     '--testConfigFile',
     testFile,
     '--buildNumber',
     buildNumber
-  ]);
+  ];
+
+  if (testFile.includes('packages/firestore')) {
+    // Firestore requires this flag to run unit tests only.
+    karmaArgs.push('--unit');
+  }
+  const promise = spawn('npx', karmaArgs, { stdio: 'inherit' });
   const childProcess = promise.childProcess;
   let exitCode = 0;
 
-  childProcess.stdout.on('data', data => {
-    console.log(`[${testFile}]:`, data.toString());
-  });
-
-  // Lerna's normal output goes to stderr for some reason.
-  childProcess.stderr.on('data', data => {
-    console.log(`[${testFile}]:`, data.toString());
-  });
-
   // Capture exit code of this single package test run
   childProcess.on('exit', code => {
     exitCode = code;