test: verify the shell option works properly on execFile

Useful for executing in a shell because it accepts arguments as an array instead of a string as exec does. Depending on the circumstances, that can prove to be useful if the arguments are already prepared. PR-URL: #18384 Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>

Author: jvelezpo

test/parallel/test-child-process-execfile.js

@@ -6,6 +6,7 @@ const uv = process.binding('uv');
 const fixtures = require('../common/fixtures');
 
 const fixture = fixtures.path('exit.js');
+const execOpts = { encoding: 'utf8', shell: true };
 
 {
  execFile(
@@ -38,3 +39,10 @@ const fixture = fixtures.path('exit.js');
  child.kill();
  child.emit('close', code, null);
 }
+
+{
+ // Verify the shell option works properly
+ execFile(process.execPath, [fixture, 0], execOpts, common.mustCall((err) => {
+ assert.ifError(err);
+ }));
+}

test/sequential/test-child-process-execsync.js

@@ -8,6 +8,7 @@ const TIMER = 200;
 const SLEEP = 2000;
 
 const start = Date.now();
+const execOpts = { encoding: 'utf8', shell: true };
 let err;
 let caught = false;
 
@@ -103,3 +104,8 @@ assert.strictEqual(ret, `${msg}\n`);
  return true;
  });
 }
+
+// Verify the shell option works properly
+assert.doesNotThrow(() => {
+ execFileSync(process.execPath, [], execOpts);
+});