Skip to content

Commit fc16ee3

Browse files
cjihrigtargos
authored andcommitted
test_runner: remove outdated comment
Network imports were removed, so remove the comment. Refs: #53822 PR-URL: #54146 Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Jake Yuesong Li <[email protected]> Reviewed-By: Trivikram Kamat <[email protected]> Reviewed-By: Moshe Atlow <[email protected]>
1 parent f80f09f commit fc16ee3

File tree

1 file changed

+0
-7
lines changed

1 file changed

+0
-7
lines changed

lib/test/mock_loader.js

-7
Original file line numberDiff line numberDiff line change
@@ -36,13 +36,6 @@ const { createRequire, isBuiltin } = require('module');
3636
// https://github.com/nodejs/node/issues/49472
3737
// or https://github.com/nodejs/node/issues/52219
3838

39-
// TODO(cjihrig): Network imports should be supported. There are two current
40-
// hurdles:
41-
// - The module format returned by the load() hook is not known. This could be
42-
// implemented as an option, or default to 'module' for network imports.
43-
// - The generated mock module imports 'node:test', which is not allowed by
44-
// checkIfDisallowedImport() in the ESM code.
45-
4639
const mocks = new SafeMap();
4740

4841
async function initialize(data) {

0 commit comments

Comments
 (0)