module: block requiring test/reporters without scheme

PR-URL: #47831Fixes: #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>

Author: MoLow

lib/internal/bootstrap/loaders.js

@@ -125,6 +125,7 @@ const legacyWrapperList = new SafeSet([
 // Modules that can only be imported via the node: scheme.
 constschemelessBlockList=newSafeSet([
 'test',
+'test/reporters',
 ]);
 
 // Set up process.binding() and process._linkedBinding().

test/parallel/test-runner-import-no-scheme.js

@@ -7,45 +7,60 @@ const fs = require('fs');
 constpath=require('path');
 const{ createRequire }=require('module');
 
-assert.throws(
-()=>require('test'),
-common.expectsError({code: 'MODULE_NOT_FOUND'}),
-);
-
-(async()=>{
-awaitassert.rejects(
-async()=>import('test'),
-common.expectsError({code: 'ERR_MODULE_NOT_FOUND'}),
+for(constnamein['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()=>{
+awaitassert.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();
 
 constpackageRoot=path.join(tmpdir.path,'node_modules','test');
+constreportersDir=path.join(tmpdir.path,'node_modules','test','reporters');
 constindexFile=path.join(packageRoot,'index.js');
+constreportersIndexFile=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 };');
 
-functiontest(argv){
+functiontest(argv,expectedToFail=false){
 constchild=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);
 
 {
 constdummyFile=path.join(tmpdir.path,'file.js');
 constrequire=createRequire(dummyFile);
 assert.strictEqual(require.resolve('test'),indexFile);
+assert.strictEqual(require.resolve('test/reporters'),reportersIndexFile);
 }