@travetto/test 7.0.5 → 7.1.0

package/README.md

@@ -82,7 +82,7 @@ import * as Δdebug from "@travetto/runtime/src/debug.js";
 import * as Δcheck from "@travetto/test/src/assert/check.js";
 import * as Δfunction from "@travetto/runtime/src/function.js";
 import * as Δschema from "@travetto/schema/src/decorator/schema.js";
-var Δm_1 = ["@travetto/test", "doc/assert-example.ts"];
+const Δm_1 = ["@travetto/test", "doc/assert-example.ts"];
 import assert from 'node:assert';
 import { Suite, Test } from '@travetto/test';
 let SimpleTest = class SimpleTest {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@travetto/test",
-  "version": "7.0.5",
+  "version": "7.1.0",
   "type": "module",
   "description": "Declarative test framework",
   "keywords": [
@@ -28,15 +28,15 @@
     "directory": "module/test"
   },
   "dependencies": {
-    "@travetto/registry": "^7.0.5",
-    "@travetto/runtime": "^7.0.4",
-    "@travetto/terminal": "^7.0.4",
-    "@travetto/worker": "^7.0.4",
+    "@travetto/registry": "^7.1.0",
+    "@travetto/runtime": "^7.1.0",
+    "@travetto/terminal": "^7.1.0",
+    "@travetto/worker": "^7.1.0",
     "yaml": "^2.8.2"
   },
   "peerDependencies": {
-    "@travetto/cli": "^7.0.6",
-    "@travetto/transformer": "^7.0.4"
+    "@travetto/cli": "^7.1.0",
+    "@travetto/transformer": "^7.1.0"
   },
   "peerDependenciesMeta": {
     "@travetto/transformer": {

package/src/execute/run.ts

@@ -4,7 +4,7 @@ import readline from 'node:readline/promises';
 import path from 'node:path';
 import { spawn } from 'node:child_process';
 
-import { Env, ExecUtil, ShutdownManager, Util, RuntimeIndex, Runtime, TimeUtil, JSONUtil } from '@travetto/runtime';
+import { Env, ExecUtil, Util, RuntimeIndex, Runtime, TimeUtil, JSONUtil } from '@travetto/runtime';
 import { WorkPool } from '@travetto/worker';
 import { Registry } from '@travetto/registry';
 
@@ -27,12 +27,6 @@ type RunState = {
  * Test Utilities for Running
  */
 export class RunUtil {
-  /**
-   * Add 50 ms to the shutdown to allow for buffers to output properly
-   */
-  static registerCleanup(scope: string): void {
-    ShutdownManager.onGracefulShutdown(() => Util.blockingTimeout(50), `test.${scope}.bufferOutput`);
-  }
 
   /**
    * Determine if a given file path is a valid test file
@@ -220,11 +214,11 @@ export class RunUtil {
   static async runTests(consumerConfig: TestConsumerConfig, input: TestRunInput): Promise<boolean | undefined> {
     const { runs, removes } = await this.resolveInput(input);
 
-    await this.reinitManifestIfNeeded(runs);
-
     const targetConsumer = await TestConsumerRegistryIndex.getInstance(consumerConfig);
     const consumer = await this.getRunnableConsumer(targetConsumer, runs);
 
+    await this.reinitManifestIfNeeded(runs);
+
     for (const item of removes ?? []) {
       consumer.onRemoveEvent(item);
     }

package/src/registry/registry-adapter.ts

@@ -22,6 +22,8 @@ function combineClasses(baseConfig: SuiteConfig, ...subConfig: Partial<SuiteConf
     if (config.tags) {
       baseConfig.tags = [...baseConfig.tags ?? [], ...config.tags];
     }
+    baseConfig.skip = config.skip ?? baseConfig.skip;
+
     if (config.tests) {
       for (const [key, test] of Object.entries(config.tests ?? {})) {
         baseConfig.tests[key] = {
@@ -66,6 +68,7 @@ export class SuiteRegistryAdapter implements RegistryAdapter<SuiteConfig> {
         class: this.#cls,
         classId: this.#cls.Ⲑid,
         tags: [],
+        skip: false,
         import: Runtime.getImport(this.#cls),
         lineStart: lines?.[0],
         lineEnd: lines?.[1],
@@ -89,6 +92,7 @@ export class SuiteRegistryAdapter implements RegistryAdapter<SuiteConfig> {
       const config = asFull<TestConfig>({
         class: this.#cls,
         tags: [],
+        skip: false,
         import: Runtime.getImport(this.#cls),
         lineStart: lines?.[0],
         lineEnd: lines?.[1],

package/src/worker/child.ts

@@ -42,8 +42,6 @@ export class TestChildWorker extends IpcChannel<TestRun> {
       ConsoleManager.set({ log: () => { } });
     }
 
-    RunUtil.registerCleanup('worker');
-
     // Listen for inbound requests
     this.on('*', event => this.onCommand(event));
 

package/src/worker/standard.ts

@@ -20,16 +20,12 @@ const log = (message: string | TestLogEvent): void => {
 export async function buildStandardTestManager(consumer: TestConsumerShape, run: TestRun | TestDiffInput): Promise<void> {
   log(`Worker Input ${JSON.stringify(run)}`);
 
-  const suiteMod = RuntimeIndex.findModuleForArbitraryImport(run.import)!;
-
   const channel = new IpcChannel<TestEvent & { error?: Error }>(
     fork(
       RuntimeIndex.resolveFileImport('@travetto/cli/support/entry.trv.ts'), ['test:child'],
       {
-        cwd: suiteMod.sourcePath,
         env: {
           ...process.env,
-          ...Env.TRV_MANIFEST.export(suiteMod.outputPath),
           ...Env.TRV_QUIET.export(true)
         },
         stdio: ['ignore', 'ignore', 2, 'ipc']

package/support/bin/run.ts

@@ -12,8 +12,6 @@ import type { TestRunInput } from '../../src/model/test.ts';
 export async function runTests(state: TestConsumerConfig, input: TestRunInput): Promise<void> {
   const { RunUtil } = await import('../../src/execute/run.ts');
 
-  RunUtil.registerCleanup('runner');
-
   try {
     const result = await RunUtil.runTests(state, input);
     process.exitCode = result ? 0 : 1;

package/support/cli.test.ts

@@ -33,10 +33,10 @@ export class TestCommand implements CliCommandShape {
   preMain(): void {
     EventEmitter.defaultMaxListeners = 1000;
     Env.TRV_ROLE.set('test');
-    Env.TRV_ENV.set('test');
     Env.DEBUG.set(false);
     Env.TRV_LOG_PLAIN.set(true);
     Env.TRV_LOG_TIME.clear();
+    Env.TRV_SHUTDOWN_STDOUT_WAIT.set(50);
   }
 
   async preValidate(): Promise<void> {

package/support/cli.test_child.ts

@@ -11,11 +11,11 @@ export class TestChildWorkerCommand {
   preMain(): void {
     EventEmitter.defaultMaxListeners = 1000;
     Env.TRV_ROLE.set('test');
-    Env.TRV_ENV.set('test');
     Env.DEBUG.set(false);
     Env.FORCE_COLOR.set(false);
     Env.TRV_LOG_PLAIN.set(true);
     Env.TRV_LOG_TIME.clear();
+    Env.TRV_SHUTDOWN_STDOUT_WAIT.set(50);
   }
 
   async main(): Promise<void> {

package/support/cli.test_diff.ts

@@ -24,7 +24,6 @@ export class TestDiffCommand {
 
   preMain(): void {
     Env.TRV_ROLE.set('test');
-    Env.TRV_ENV.set('test');
     Env.TRV_LOG_PLAIN.set(true);
     Env.TRV_LOG_TIME.clear();
   }

package/support/cli.test_direct.ts

@@ -23,7 +23,6 @@ export class TestDirectCommand {
 
   preMain(): void {
     Env.TRV_ROLE.set('test');
-    Env.TRV_ENV.set('test');
     Env.TRV_LOG_PLAIN.set(true);
     Env.TRV_LOG_TIME.clear();
   }