test_runner: make built in reporters internal

This commit updates the test runner to make the built in test reporters internal modules. PR-URL: #46092 Backport-PR-URL: #46839 Reviewed-By: Tobias Nießen <[email protected]> Reviewed-By: Antoine du Hamel <[email protected]> Reviewed-By: Moshe Atlow <[email protected]>
nodejs · Mar 5, 2023 · 0d999e3 · 0d999e3
1 parent 79f4b42
commit 0d999e3
Show file tree

Hide file tree

Showing 4 changed files with 26 additions and 11 deletions.
diff --git a/lib/test/reporter/dot.js → lib/internal/test_runner/reporter/dot.js b/lib/test/reporter/dot.js → lib/internal/test_runner/reporter/dot.js
diff --git a/lib/test/reporter/spec.js → lib/internal/test_runner/reporter/spec.js b/lib/test/reporter/spec.js → lib/internal/test_runner/reporter/spec.js
diff --git a/lib/test/reporter/tap.js → lib/internal/test_runner/reporter/tap.js b/lib/test/reporter/tap.js → lib/internal/test_runner/reporter/tap.js
diff --git a/lib/internal/test_runner/utils.js b/lib/internal/test_runner/utils.js
@@ -93,28 +93,43 @@ const kBuiltinDestinations = new SafeMap([
 ]);
 
 const kBuiltinReporters = new SafeMap([
-  ['spec', 'node:test/reporter/spec'],
-  ['dot', 'node:test/reporter/dot'],
-  ['tap', 'node:test/reporter/tap'],
+  ['spec', 'internal/test_runner/reporter/spec'],
+  ['dot', 'internal/test_runner/reporter/dot'],
+  ['tap', 'internal/test_runner/reporter/tap'],
 ]);
 
 const kDefaultReporter = 'tap';
 const kDefaultDestination = 'stdout';
 
+function tryBuiltinReporter(name) {
+  const builtinPath = kBuiltinReporters.get(name);
+
+  if (builtinPath === undefined) {
+    return;
+  }
+
+  return require(builtinPath);
+}
+
 async function getReportersMap(reporters, destinations) {
   return SafePromiseAllReturnArrayLike(reporters, async (name, i) => {
     const destination = kBuiltinDestinations.get(destinations[i]) ?? createWriteStream(destinations[i]);
 
     // Load the test reporter passed to --test-reporter
-    const reporterSpecifier = kBuiltinReporters.get(name) ?? name;
-    let parentURL;
-    try {
-      parentURL = pathToFileURL(process.cwd() + '/').href;
-    } catch {
-      parentURL = 'file:///';
+    let reporter = tryBuiltinReporter(name);
+
+    if (reporter === undefined) {
+      let parentURL;
+
+      try {
+        parentURL = pathToFileURL(process.cwd() + '/').href;
+      } catch {
+        parentURL = 'file:///';
+      }
+
+      const { esmLoader } = require('internal/process/esm_loader');
+      reporter = await esmLoader.import(name, parentURL, ObjectCreate(null));
     }
-    const { esmLoader } = require('internal/process/esm_loader');
-    let reporter = await esmLoader.import(reporterSpecifier, parentURL, ObjectCreate(null));
 
     if (reporter?.default) {
       reporter = reporter.default;