@travetto/test 3.4.0-rc.0 → 3.4.0-rc.1

package/README.md

@@ -79,6 +79,7 @@ would translate to:
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
 const tslib_1 = require("tslib");
+const Ⲑ_util_1 = tslib_1.__importStar(require("@travetto/test/src/execute/util.js"));
 const Ⲑ_check_1 = tslib_1.__importStar(require("@travetto/test/src/assert/check.js"));
 const Ⲑ_root_index_1 = tslib_1.__importStar(require("@travetto/manifest/src/root-index.js"));
 const Ⲑ_decorator_1 = tslib_1.__importStar(require("@travetto/registry/src/decorator.js"));
@@ -88,6 +89,8 @@ const test_1 = require("@travetto/test");
 let SimpleTest = class SimpleTest {
     static Ⲑinit = Ⲑ_root_index_1.RootIndex.registerFunction(SimpleTest, ᚕf, 1887908328, { test: { hash: 102834457 } }, false, false);
     async test() {
+        if (Ⲑ_util_1.RunnerUtil.tryDebugger)
+            debugger;
         Ⲑ_check_1.AssertCheck.check({ file: ᚕf, line: 10, text: "{ size: 20, address: { state: 'VA' } }", operator: "deepStrictEqual" }, true, { size: 20, address: { state: 'VA' } }, {});
     }
 };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@travetto/test",
-  "version": "3.4.0-rc.0",
+  "version": "3.4.0-rc.1",
   "description": "Declarative test framework",
   "keywords": [
     "unit-testing",
@@ -27,15 +27,15 @@
     "directory": "module/test"
   },
   "dependencies": {
-    "@travetto/base": "^3.4.0-rc.0",
-    "@travetto/registry": "^3.4.0-rc.0",
+    "@travetto/base": "^3.4.0-rc.1",
+    "@travetto/registry": "^3.4.0-rc.1",
     "@travetto/terminal": "^3.4.0-rc.0",
-    "@travetto/worker": "^3.4.0-rc.0",
-    "@travetto/yaml": "^3.4.0-rc.0"
+    "@travetto/worker": "^3.4.0-rc.1",
+    "@travetto/yaml": "^3.4.0-rc.1"
   },
   "peerDependencies": {
-    "@travetto/cli": "^3.4.0-rc.0",
-    "@travetto/transformer": "^3.4.0-rc.0"
+    "@travetto/cli": "^3.4.0-rc.2",
+    "@travetto/transformer": "^3.4.0-rc.1"
   },
   "peerDependenciesMeta": {
     "@travetto/transformer": {
@@ -47,7 +47,7 @@
   },
   "travetto": {
     "displayName": "Testing",
-    "profiles": [
+    "roles": [
       "test"
     ]
   },

package/src/execute/runner.ts

@@ -30,7 +30,7 @@ export class Runner {
   async runFiles(): Promise<boolean> {
     const consumer = await RunnableTestConsumer.get(this.#state.consumer ?? this.#state.format);
 
-    const files = (await RunnerUtil.getTestFiles(this.patterns));
+    const files = (await RunnerUtil.getTestFiles(this.patterns)).map(f => f.sourceFile);
 
     console.debug('Running', { files, patterns: this.patterns });
 

package/src/execute/util.ts

@@ -2,7 +2,7 @@ import { createReadStream } from 'fs';
 import readline from 'readline';
 
 import { ExecUtil, ShutdownManager, TimeUtil } from '@travetto/base';
-import { RootIndex } from '@travetto/manifest';
+import { IndexedFile, RootIndex } from '@travetto/manifest';
 
 /**
  * Simple Test Utilities
@@ -36,12 +36,16 @@ export class RunnerUtil {
   /**
    * Find all valid test files given the globs
    */
-  static async getTestFiles(globs: RegExp[]): Promise<string[]> {
-    const files = RootIndex.findTest({})
-      .filter(f => globs.some(g => g.test(f.import)));
+  static async getTestFiles(globs?: RegExp[]): Promise<IndexedFile[]> {
+    const files = RootIndex.find({
+      module: m => m.roles.includes('test') || m.roles.includes('std'),
+      folder: f => f === 'test',
+      file: f => f.role === 'test'
+    })
+      .filter(f => globs?.some(g => g.test(f.import)) ?? true);
 
     const validFiles = files
-      .map(f => this.isTestFile(f.sourceFile).then(valid => ({ file: f.sourceFile, valid })));
+      .map(f => this.isTestFile(f.sourceFile).then(valid => ({ file: f, valid })));
 
     return (await Promise.all(validFiles))
       .filter(x => x.valid)
@@ -61,4 +65,14 @@ export class RunnerUtil {
     }
     return countRes.valid ? +countRes.stdout : 0;
   }
+
+  /**
+   * Determine if we should invoke the debugger
+   */
+  static get tryDebugger(): boolean {
+    if (process.env.TRV_TEST_BREAK_ENTRY === '1') {
+      return true;
+    }
+    return false;
+  }
 }

package/src/execute/watcher.ts

@@ -8,6 +8,7 @@ import { buildStandardTestManager } from '../worker/standard';
 import { TestConsumerRegistry } from '../consumer/registry';
 import { CumulativeSummaryConsumer } from '../consumer/types/cumulative';
 import { RunEvent } from '../worker/types';
+import { RunnerUtil } from './util';
 
 function isRunEvent(ev: unknown): ev is RunEvent {
   return ObjectUtil.isPlainObject(ev) && 'type' in ev && typeof ev.type === 'string' && ev.type === 'run-test';
@@ -78,7 +79,7 @@ export class TestWatcher {
     process.send?.('ready');
 
     if (runAllOnStart) {
-      for (const test of await RootIndex.findTest({})) {
+      for (const test of await RunnerUtil.getTestFiles()) {
         await import(test.import);
         itr.add(test.sourceFile);
       }

package/support/cli.test_count.ts

@@ -19,9 +19,9 @@ export class TestCountCommand {
     // Load all tests
     for (const file of files) {
       try {
-        await import(RootIndex.getFromSource(file)!.import);
+        await import(file.import);
       } catch (err) {
-        console.error('Failed to import', file, err);
+        console.error('Failed to import', file.sourceFile, err);
       }
     }
 

package/support/transformer.annotate.ts

@@ -4,18 +4,40 @@ import {
   TransformerState, DecoratorMeta, OnMethod, OnClass, CoreUtil, DecoratorUtil
 } from '@travetto/transformer';
 
+const RUN_UTIL = 'RunnerUtil';
+
+const RunUtilⲐ = Symbol.for('@travetto/test:runner');
+
+/**
+ * Annotate transformation state
+ */
+interface AnnotateState {
+  [RunUtilⲐ]?: ts.Expression;
+}
+
 /**
  * Annotate tests and suites for better diagnostics
  */
 export class AnnotationTransformer {
 
+
+  /**
+   * Initialize transformer state
+   */
+  static initState(state: TransformerState & AnnotateState): void {
+    if (!state[RunUtilⲐ]) {
+      const runUtil = state.importFile('@travetto/test/src/execute/util').ident;
+      state[RunUtilⲐ] = CoreUtil.createAccess(state.factory, runUtil, RUN_UTIL, 'tryDebugger');
+    }
+  }
+
   /**
    * Build source annotation, indicating line ranges
    * @param state
    * @param node
    * @param dec
   */
-  static buildAnnotation(state: TransformerState, node: ts.Node, dec: ts.Decorator, expression: ts.CallExpression): ts.Decorator {
+  static buildAnnotation(state: TransformerState & AnnotateState, node: ts.Node, dec: ts.Decorator, expression: ts.CallExpression): ts.Decorator {
     const ogN = (CoreUtil.hasOriginal(node) ? node.original : node);
     const n = ts.isMethodDeclaration(ogN) ? ogN : undefined;
 
@@ -40,7 +62,7 @@ export class AnnotationTransformer {
   }
 
   @OnClass('Suite')
-  static annotateSuiteDetails(state: TransformerState, node: ts.ClassDeclaration, dm?: DecoratorMeta): ts.ClassDeclaration {
+  static annotateSuiteDetails(state: TransformerState & AnnotateState, node: ts.ClassDeclaration, dm?: DecoratorMeta): ts.ClassDeclaration {
     const dec = dm?.dec;
 
     if (dec && ts.isCallExpression(dec.expression)) {
@@ -57,7 +79,9 @@ export class AnnotationTransformer {
   }
 
   @OnMethod('Test')
-  static annotateTestDetails(state: TransformerState, node: ts.MethodDeclaration, dm?: DecoratorMeta): ts.MethodDeclaration {
+  static annotateTestDetails(state: TransformerState & AnnotateState, node: ts.MethodDeclaration, dm?: DecoratorMeta): ts.MethodDeclaration {
+    this.initState(state);
+
     const dec = dm?.dec;
 
     if (dec && ts.isCallExpression(dec.expression)) {
@@ -70,7 +94,11 @@ export class AnnotationTransformer {
         node.typeParameters,
         node.parameters,
         node.type,
-        node.body
+        node.body ? state.factory.updateBlock(node.body, [
+          state.factory.createIfStatement(state[RunUtilⲐ]!,
+            state.factory.createExpressionStatement(state.factory.createIdentifier('debugger'))),
+          ...node.body.statements
+        ]) : node.body
       );
     }
     return node;