Commit 5a1afb2
test_runner: remove outdated comment
Network imports were removed, so remove the comment.
Refs: #53822
PR-URL: #54146
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Jake Yuesong Li <jake.yuesong@gmail.com>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
Reviewed-By: Moshe Atlow <moshe@atlow.co.il>1 parent 425b956 commit 5a1afb2
1 file changed
Lines changed: 0 additions & 7 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -36,13 +36,6 @@ const { createRequire, isBuiltin } = require('module'); | |||
| 36 | 36 | // https://github.com/nodejs/node/issues/49472 | |
| 37 | 37 | // or https://github.com/nodejs/node/issues/52219 | |
| 38 | 38 | ||
| 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 | - | ||
| 46 | 39 | const mocks = new SafeMap(); | |
| 47 | 40 | ||
| 48 | 41 | async function initialize(data) { | |
0 commit comments