test_runner: fix test enqueue when test file has syntax error

geeksilva97 · aduh95 · commit 21b23cd0d07c · 2026-02-22T23:16:14.000+01:00
PR-URL: #61573 Reviewed-By: Moshe Atlow <moshe@atlow.co.il> Reviewed-By: Aviv Keller <me@aviv.sh> Reviewed-By: Pietro Marchini <pietro.marchini94@gmail.com>
diff --git a/lib/internal/test_runner/runner.js b/lib/internal/test_runner/runner.js
@@ -874,7 +874,10 @@ function run(options = kEmptyObject) {
             );
             if (topLevelTestCount === root.subtests.length) {
               // This file had no tests in it. Add the placeholder test.
-              const subtest = root.createSubtest(Test, testFile);
+              const subtest = root.createSubtest(Test, testFile, kEmptyObject, undefined, {
+                __proto__: null,
+                loc: [1, 1, resolve(testFile)],
+              });
               if (threw) {
                 subtest.fail(importError);
               }
diff --git a/test/fixtures/test-runner/syntax-error-test.mjs b/test/fixtures/test-runner/syntax-error-test.mjs
@@ -0,0 +1,7 @@
+import { test } from 'node:test';
+
+test('a test!', () => {
+    if true {
+        // syntax error
+    }
+});
diff --git a/test/parallel/test-runner-enqueue-file-syntax-error.js b/test/parallel/test-runner-enqueue-file-syntax-error.js
@@ -0,0 +1,19 @@
+'use strict';
+const common = require('../common');
+const assert = require('node:assert');
+const fixtures = require('../common/fixtures');
+const { run } = require('node:test');
+
+const testFile = fixtures.path('test-runner', 'syntax-error-test.mjs');
+const testRun = run({
+  files: [testFile],
+  isolation: 'none'
+});
+
+testRun.on('test:enqueue', common.mustCall((test) => {
+  assert.strictEqual(test.file, testFile);
+}));
+
+testRun.on('test:fail', common.mustCall((test) => {
+  assert.match(test.details.error.toString(), /SyntaxError: Unexpected token 'true'/);
+}));
