test: add filter, seed and better diff checking to test script

hansl · alexeagle · commit 49ba744f2356 · 2019-03-22T07:15:48.000-07:00
It wasnt checking the staging area. It also now shows the packages that will be
tested.

Filter and Seed are arguments that are directly passed to Jasmine. They can be
used to reproduce tests.
diff --git a/scripts/test.ts b/scripts/test.ts
@@ -17,6 +17,7 @@ import { join, relative } from 'path';
 import { Position, SourceMapConsumer } from 'source-map';
 import * as ts from 'typescript';
 import { packages } from '../lib/packages';
+import { diff } from 'semver';
 
 const codeMap = require('../lib/istanbul-local').codeMap;
 const Jasmine = require('jasmine');
@@ -225,12 +226,15 @@ export default function (args: ParsedArgs, logger: logging.Logger) {
     const sha = branchRevList.find(s => masterRevList.includes(s));
 
     if (sha) {
-      const diffFiles = _exec(
-        'git',
-        ['diff', sha, 'HEAD', '--name-only'],
-        {},
-        logger,
-      ).trim().split('\n');
+      const diffFiles = [
+        // Get diff between $SHA and HEAD.
+        ..._exec('git', ['diff', sha, 'HEAD', '--name-only'], {}, logger)
+          .trim().split('\n'),
+        // And add the current status to it (so it takes the non-committed changes).
+        ..._exec('git', ['status', '--short', '--show-stash'], {}, logger)
+          .split('\n').map(x => x.slice(2).trim()),
+      ].filter(x => x !== '');
+
       const diffPackages = new Set();
       for (const pkgName of Object.keys(packages)) {
         const relativeRoot = relative(projectBaseDir, packages[pkgName].root);
@@ -241,6 +245,10 @@ export default function (args: ParsedArgs, logger: logging.Logger) {
         }
       }
 
+      // Show the packages that we will test.
+      logger.info(`Found ${diffPackages.size} packages:`);
+      logger.info(JSON.stringify([...diffPackages], null, 2));
+
       // Remove the tests from packages that haven't changed.
       tests = tests
         .filter(p => Object.keys(packages).some(name => {
@@ -267,6 +275,10 @@ export default function (args: ParsedArgs, logger: logging.Logger) {
 
   return new Promise(resolve => {
     runner.onComplete((passed: boolean) => resolve(passed ? 0 : 1));
-    runner.execute(tests);
+    if (args.seed != undefined) {
+      runner.seed(args.seed);
+    }
+
+    runner.execute(tests, args.filter);
   });
 }
