velocious 1.0.653 → 1.0.654

package/src/testing/test-runner.js

@@ -142,6 +142,8 @@ import VelociousTestArguments from "./velocious-test-arguments.js"
  * @property {import("../database/pool/base.js").TestSharedConnectionRegistration | undefined} sharedRegistration - Physical-key shared registration once published.
  */
 
+const testingPackageDirectory = path.dirname(fileURLToPath(import.meta.resolve("@velocious/testing/package.json")))
+
 /**
  * Runs to file slug.
  * @param {string} value - Value to sanitize.
@@ -197,6 +199,8 @@ export default class TestRunner {
     this._successfulTests = 0
     this._testsCount = 0
     this._failedTestDetails = []
+    /** @type {import("@velocious/testing/runner").NonRunTestResult[]} */
+    this._notRunTestDetails = []
     /** @type {{fullDescription: string, filePath: string, line: number} | null} */
     this._lastTestContext = null
     /** @type {Array<{fullDescription: string, filePath: string, line: number, durationMs: number}>} */
@@ -1007,6 +1011,25 @@ export default class TestRunner {
     return this._failedTests
   }
 
+  /**
+   * Counts selected tests blocked by a terminal resource.
+   * @returns {number} - Selected tests not executed because a shared resource failed.
+   */
+  getNotRunTests() { return this._notRunTestDetails.length }
+
+  /**
+   * Returns runtime non-run attribution.
+   * @returns {import("@velocious/testing/runner").NonRunTestResult[]} - Runtime non-run details with the originating failure.
+   */
+  getNotRunTestDetails() { return this._notRunTestDetails }
+
+  /**
+   * Records a selected test that did not execute.
+   * @param {import("@velocious/testing/runner").NonRunTestResult} result - Terminal-resource non-run record.
+   * @returns {void}
+   */
+  recordNotRunTest(result) { this._notRunTestDetails.push(result) }
+
   /**
    * Runs get failed test details.
    * @returns {FailedTestDetail[]} - Failed test details.
@@ -1087,6 +1110,14 @@ export default class TestRunner {
     return this._packageResult?.tests.length ?? this._testDurations.length
   }
 
+  /**
+   * Distinguishes an empty selection from a failure before selected cases execute.
+   * @returns {boolean} - Whether selection matched no declarations.
+   */
+  hasNoMatches() {
+    return this._packageResult?.noMatches === true
+  }
+
   /**
    * Returns the tests recorded during the run, slowest first.
    * @param {number} [limit] - Maximum number of tests to return (0 returns all).
@@ -1105,6 +1136,7 @@ export default class TestRunner {
   async prepare() {
     this.anyTestsFocussed = false
     this._failedTests = 0
+    this._notRunTestDetails = []
     this._successfulTests = 0
     this._testsCount = 0
     this._abortRemainingTests = false
@@ -1177,7 +1209,7 @@ export default class TestRunner {
 
       if (portablePath.endsWith("/src/testing/test-runner.js")) continue
       if (portablePath.endsWith("/src/testing/test.js")) continue
-      if (portablePath.includes("/node_modules/@velocious/testing/")) continue
+      if (resolvedFilePath.startsWith(`${testingPackageDirectory}${path.sep}`)) continue
 
       return {filePath: resolvedFilePath, line: Number(match[2])}
     }

package/src/testing/velocious-runner-reporter.js

@@ -1,7 +1,6 @@
 // @ts-check
 
 import { addTrackedStackToError } from "../utils/with-tracked-stack.js"
-import BacktraceCleaner from "../utils/backtrace-cleaner-node.js"
 import picocolors from "picocolors"
 import restArgsError from "../utils/rest-args-error.js"
 import { testEvents } from "./test.js"
@@ -21,6 +20,8 @@ function errorFromPackageRecord(errorRecord) {
 
   error.name = errorRecord.name
   if (errorRecord.stack) error.stack = errorRecord.stack
+  if (errorRecord.cause) error.cause = errorFromPackageRecord(errorRecord.cause)
+  if (errorRecord.terminalResource) Object.assign(error, {terminalResource: errorRecord.terminalResource})
 
   return error
 }
@@ -68,7 +69,27 @@ export default class VelociousRunnerReporter {
       return
     }
 
-    if (event.type === "run:finish") this.testRunner.recordPackageResult(event.result)
+    if (event.type === "test:not-run") {
+      const test = this.testRunner.findTestDeclaration(event.test.fullName)
+      if (!test) throw new Error(`Package not-run result did not match a declaration: ${event.test.fullName}`)
+      this.testRunner.recordNotRunTest(event.test)
+      this.testRunner.completeTestDeclaration(test)
+      console.error(`Not run: ${event.test.fullName} (terminal resource failure in ${event.test.reason.fullName})`)
+      await this.emitEvent("testNotRun", {
+        configuration: this.testRunner.getConfiguration(),
+        test: event.test,
+        testRunner: this.testRunner
+      })
+      return
+    }
+
+    if (event.type === "run:finish") {
+      this.testRunner.recordPackageResult(event.result)
+      for (const failure of event.result.errors) {
+        console.error(`Suite ${failure.phase} failed: ${failure.suite}`)
+        this.printErrorCauses(errorFromPackageRecord(failure.error))
+      }
+    }
   }
 
   /**
@@ -95,7 +116,7 @@ export default class VelociousRunnerReporter {
     const failed = outcome?.failed ?? Boolean(attempt.error)
     const error = outcome?.error
     const retriesUsed = Math.min(attempt.attemptNumber, retryCount)
-    const willRetry = failed && !outcome?.abortRemainingTests && attempt.attemptNumber <= retryCount
+    const willRetry = failed && !event.terminalFailure && !outcome?.abortRemainingTests && attempt.attemptNumber <= retryCount
     const {descriptions, testDescription} = this.testRunner.testMetadata(test)
     const compatibility = this.testRunner.testData(test)
 
@@ -245,15 +266,8 @@ export default class VelociousRunnerReporter {
 
     if (error instanceof Error) {
       console.error(picocolors.red(`${leftPadding}  Test failed: ${error.message}`))
-      addTrackedStackToError(error)
-
-      const backtraceCleaner = new BacktraceCleaner(error)
-      const cleanedStack = backtraceCleaner.getCleanedStack()
-      const stackLines = cleanedStack?.split("\n")
+      this.printErrorCauses(error)
 
-      if (stackLines) {
-        for (const stackLine of stackLines) console.error(picocolors.red(`${leftPadding}  ${stackLine}`))
-      }
     } else {
       console.error(picocolors.red(`${leftPadding}  Test failed with a ${typeof error}: ${String(error)}`))
     }
@@ -274,6 +288,33 @@ export default class VelociousRunnerReporter {
     testRunner.printRerunCommand({descriptions, testDescription, testData, leftPadding})
   }
 
+  /**
+   * Prints complete primary and secondary stacks once, including cyclic cause graphs.
+   * @param {unknown} error - Thrown value at the reporting boundary.
+   * @param {Set<Error>} [reported] - Error identities already printed.
+   * @returns {void}
+   */
+  printErrorCauses(error, reported = new Set()) {
+    if (!(error instanceof Error)) {
+      console.error(String(error))
+      return
+    }
+    if (reported.has(error)) return
+    reported.add(error)
+    addTrackedStackToError(error)
+    console.error(error.stack || `${error.name}: ${error.message}`)
+    if (error.cause !== undefined) {
+      console.error("Caused by:")
+      this.printErrorCauses(error.cause, reported)
+    }
+    if (error instanceof AggregateError) {
+      for (const secondary of error.errors) {
+        console.error("Related failure:")
+        this.printErrorCauses(secondary, reported)
+      }
+    }
+  }
+
   /**
    * Emits one legacy event and awaits listeners in registration order.
    * @param {string} eventName - Event name.