From cf8845d0018277b1a7c32c9938d110b544f41216 Mon Sep 17 00:00:00 2001
From: Moshe Atlow <moshe@atlow.co.il>
Date: Fri, 5 May 2023 10:59:25 +0300
Subject: [PATCH] module: block requiring `test/reporters` without scheme
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

PR-URL: https://github.com/nodejs/node/pull/47831
Fixes: https://github.com/nodejs/node/issues/47828
Reviewed-By: Filip Skokan <panva.ip@gmail.com>
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Darshan Sen <raisinten@gmail.com>
Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com>
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: Jacob Smith <jacob@frende.me>
---
 lib/internal/bootstrap/loaders.js             |  1 +
 test/parallel/test-runner-import-no-scheme.js | 51 ++++++++++++-------
 2 files changed, 34 insertions(+), 18 deletions(-)

diff --git a/lib/internal/bootstrap/loaders.js b/lib/internal/bootstrap/loaders.js
index be7924ea1615fa..37e41bfc796f21 100644
--- a/lib/internal/bootstrap/loaders.js
+++ b/lib/internal/bootstrap/loaders.js
@@ -125,6 +125,7 @@ const legacyWrapperList = new SafeSet([
 // Modules that can only be imported via the node: scheme.
 const schemelessBlockList = new SafeSet([
   'test',
+  'test/reporters',
 ]);
 
 // Set up process.binding() and process._linkedBinding().
diff --git a/test/parallel/test-runner-import-no-scheme.js b/test/parallel/test-runner-import-no-scheme.js
index 45dd83d0251988..85953af77bf9a5 100644
--- a/test/parallel/test-runner-import-no-scheme.js
+++ b/test/parallel/test-runner-import-no-scheme.js
@@ -7,45 +7,60 @@ const fs = require('fs');
 const path = require('path');
 const { createRequire } = require('module');
 
-assert.throws(
-  () => require('test'),
-  common.expectsError({ code: 'MODULE_NOT_FOUND' }),
-);
-
-(async () => {
-  await assert.rejects(
-    async () => import('test'),
-    common.expectsError({ code: 'ERR_MODULE_NOT_FOUND' }),
+for (const name in ['test', 'test/reporters']) {
+  assert.throws(
+    () => require(name),
+    common.expectsError({ code: 'MODULE_NOT_FOUND' }),
   );
-})().then(common.mustCall());
 
-assert.throws(
-  () => require.resolve('test'),
-  common.expectsError({ code: 'MODULE_NOT_FOUND' }),
-);
+  (async () => {
+    await assert.rejects(
+      async () => import(name),
+      common.expectsError({ code: 'ERR_MODULE_NOT_FOUND' }),
+    );
+  })().then(common.mustCall());
+
+  assert.throws(
+    () => require.resolve(name),
+    common.expectsError({ code: 'MODULE_NOT_FOUND' }),
+  );
+}
 
 // Verify that files in node_modules can be resolved.
 tmpdir.refresh();
 
 const packageRoot = path.join(tmpdir.path, 'node_modules', 'test');
+const reportersDir = path.join(tmpdir.path, 'node_modules', 'test', 'reporters');
 const indexFile = path.join(packageRoot, 'index.js');
+const reportersIndexFile = path.join(reportersDir, 'index.js');
 
-fs.mkdirSync(packageRoot, { recursive: true });
+fs.mkdirSync(reportersDir, { recursive: true });
 fs.writeFileSync(indexFile, 'module.exports = { marker: 1 };');
+fs.writeFileSync(reportersIndexFile, 'module.exports = { marker: 1 };');
 
-function test(argv) {
+function test(argv, expectedToFail = false) {
   const child = spawnSync(process.execPath, argv, { cwd: tmpdir.path });
-  assert.strictEqual(child.status, 0);
-  assert.strictEqual(child.stdout.toString().trim(), '{ marker: 1 }');
+  if (expectedToFail) {
+    assert.strictEqual(child.status, 1);
+    assert.strictEqual(child.stdout.toString().trim(), '');
+  } else {
+    assert.strictEqual(child.status, 0);
+    assert.strictEqual(child.stdout.toString().trim(), '{ marker: 1 }');
+  }
 }
 
 test(['-e', 'console.log(require("test"))']);
+test(['-e', 'console.log(require("test/reporters"))']);
 test(['-e', 'import("test").then(m=>console.log(m.default))']);
+test(['-e', 'import("test/reporters").then(m=>console.log(m.default))'], true);
 test(['--input-type=module', '-e', 'import test from "test";console.log(test)']);
+test(['--input-type=module', '-e', 'import test from "test/reporters";console.log(test)'], true);
 test(['--input-type=module', '-e', 'console.log((await import("test")).default)']);
+test(['--input-type=module', '-e', 'console.log((await import("test/reporters")).default)'], true);
 
 {
   const dummyFile = path.join(tmpdir.path, 'file.js');
   const require = createRequire(dummyFile);
   assert.strictEqual(require.resolve('test'), indexFile);
+  assert.strictEqual(require.resolve('test/reporters'), reportersIndexFile);
 }