fixup! test: do not assume process.execPath contains no spaces

nodejs · Sep 20, 2024 · 2ea4fa0 · 2ea4fa0
1 parent 6a3f643
commit 2ea4fa0
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/test/sequential/test-cli-syntax-file-not-found.js b/test/sequential/test-cli-syntax-file-not-found.js
@@ -8,9 +8,10 @@ const fixtures = require('../common/fixtures');
 // The execPath might contain chars that should be escaped in a shell context.
 // On non-Windows, we can pass the path via the env; `"` is not a valid char on
 // Windows, so we can simply pass the path.
-const execNode = (flag, file) => exec(
+const execNode = (flag, file, callback) => exec(
   `"${common.isWindows ? process.execPath : '$NODE'}" ${flag} "${common.isWindows ? file : '$FILE'}"`,
   common.isWindows ? undefined : { env: { ...process.env, NODE: process.execPath, FILE: file } },
+  callback,
 );
 
 // Test both sets of arguments that check syntax

diff --git a/test/sequential/test-cli-syntax-good.js b/test/sequential/test-cli-syntax-good.js
@@ -8,9 +8,10 @@ const fixtures = require('../common/fixtures');
 // The execPath might contain chars that should be escaped in a shell context.
 // On non-Windows, we can pass the path via the env; `"` is not a valid char on
 // Windows, so we can simply pass the path.
-const execNode = (flag, file) => exec(
+const execNode = (flag, file, callback) => exec(
   `"${common.isWindows ? process.execPath : '$NODE'}" ${flag} "${common.isWindows ? file : '$FILE'}"`,
   common.isWindows ? undefined : { env: { ...process.env, NODE: process.execPath, FILE: file } },
+  callback,
 );
 
 // Test both sets of arguments that check syntax