feat(cli): expose the test name via t.name in the test runner

Author: dirkdev98

packages/cli/src/testing/runner.js

@@ -1,6 +1,6 @@
 import { AssertionError, deepStrictEqual } from "assert";
 import { url } from "inspector";
-import { isNil, newLogger } from "@compas/stdlib";
+import { AppError, isNil, newLogger } from "@compas/stdlib";
 import { markTestFailuresRecursively } from "./printer.js";
 import { setTestTimeout, state, testLogger, timeout } from "./state.js";
 
@@ -138,6 +138,7 @@ function createRunnerForState(testState, abortSignal) {
       },
     }),
     signal: abortSignal,
+    name: testState.name,
     ok: ok.bind(undefined, testState),
     notOk: notOk.bind(undefined, testState),
     equal: equal.bind(undefined, testState),
@@ -170,9 +171,10 @@ function mutateRunnerEnablingWarnings(runner) {
     const implementation = runner[method];
 
     runner[method] = (...args) => {
-      runner.log.error(
-        `warning: called 't.${method}' on parent 't'. Accept 't' as argument in the callback of 't.test(msg, callback)'.`,
-      );
+      runner.log.error({
+        message: `warning: called 't.${method}' on parent 't'. Accept 't' as argument in the callback of 't.test(msg, callback)'.`,
+        stackTrace: AppError.format(AppError.serverError({})),
+      });
 
       implementation(...args);
     };

packages/cli/src/testing/runner.test.js

@@ -6,6 +6,10 @@ import { runTestsRecursively } from "./runner.js";
 mainTestFn(import.meta);
 
 test("cli/testing/runner", (t) => {
+  t.test("my test name", (t) => {
+    t.equal(t.name, "my test name");
+  });
+
   t.test(
     "runTestsRecursively - catch assertion errors strictEqual",
     async (t) => {

packages/cli/types/advanced-types.d.ts

@@ -7,6 +7,11 @@ export interface TestRunner {
    */
   log: import("@compas/stdlib").Logger;
 
+  /**
+   * The test name as provided as the first argument to 'test' or 't.test'
+   */
+  name: string;
+
   /**
    * Configurable timeout used for sub tests
    */