fixup!: unbreak caller location

Author: JakobJingleheimer

lib/internal/test_runner/test.js

@@ -359,10 +359,10 @@ class TestContext {
     this.#test.todo(message);
   }
 
-  #runTest(name, options, fn) {
+  #runTest(name, options, fn, loc) {
     const overrides = {
       __proto__: null,
-      loc: getCallerLocation(),
+      loc,
     };
 
     const { plan } = this.#test;
@@ -378,12 +378,39 @@ class TestContext {
     return subtest.start();
   }
 
-  test = ObjectAssign((...args) => this.#runTest(...args), {
+  // [1] `getCallerLocation` cannot be nested (that causes it to cite the nested location instead).
+
+  test = ObjectAssign((name, options, fn) => this.#runTest(
+    name,
+    options,
+    fn,
+    getCallerLocation(), // [1]
+  ), {
     __proto__: null,
-    expectFailure: (name, opts, fn) => this.#runTest(name, { __proto__: null, ...opts, expectFailure: true }, fn),
-    only: (name, opts, fn) => this.#runTest(name, { __proto__: null, ...opts, only: true }, fn),
-    skip: (name, opts, fn) => this.#runTest(name, { __proto__: null, ...opts, skip: true }, fn),
-    todo: (name, opts, fn) => this.#runTest(name, { __proto__: null, ...opts, todo: true }, fn),
+    expectFailure: (name, opts, fn) => this.#runTest(
+      name,
+      { __proto__: null, ...opts, expectFailure: true },
+      fn,
+      getCallerLocation(), // [1]
+    ),
+    only: (name, opts, fn) => this.#runTest(
+      name,
+      { __proto__: null, ...opts, only: true },
+      fn,
+      getCallerLocation(), // [1]
+    ),
+    skip: (name, opts, fn) => this.#runTest(
+      name,
+      { __proto__: null, ...opts, skip: true },
+      fn,
+      getCallerLocation(), // [1]
+    ),
+    todo: (name, opts, fn) => this.#runTest(
+      name,
+      { __proto__: null, ...opts, todo: true },
+      fn,
+      getCallerLocation(), // [1]
+    ),
   });
 
   before(fn, options) {