test: fix test-vm-context-dont-contextify when path contains a space

aduh95 · marco-ippolito · commit 96406610fa12 · 2024-11-17T11:51:35.000+01:00
Because of the double encoding, the test would fail as soon as the path contains a space or any other char that's already encoded by `pathToFileURL`. PR-URL: #55026 Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: Richard Lau <rlau@redhat.com> Reviewed-By: Chengzhong Wu <legendecas@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com>
diff --git a/test/parallel/test-vm-context-dont-contextify.js b/test/parallel/test-vm-context-dont-contextify.js
@@ -176,7 +176,7 @@ function checkFrozen(context) {
     const namespace = await import(moduleUrl.href);
     // Check dynamic import works
     const context = vm.createContext(vm.constants.DONT_CONTEXTIFY);
-    const script = new vm.Script(`import('${encodeURI(moduleUrl.href)}')`, {
+    const script = new vm.Script(`import(${JSON.stringify(moduleUrl)})`, {
       importModuleDynamically: vm.constants.USE_MAIN_CONTEXT_DEFAULT_LOADER,
     });
     const promise = script.runInContext(context);
