diff --git a/lib/internal/bootstrap/realm.js b/lib/internal/bootstrap/realm.js index 9dc0aee75125e3..8e35213234e995 100644 --- a/lib/internal/bootstrap/realm.js +++ b/lib/internal/bootstrap/realm.js @@ -132,6 +132,7 @@ const legacyWrapperList = new SafeSet([ // Modules that can only be imported via the node: scheme. const schemelessBlockList = new SafeSet([ 'test', + 'test/reporters', ]); // Modules that will only be enabled at run time. const experimentalModuleList = new SafeSet(); 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); }