From 6f8de31cde682f9d90455433e8ee72fffa27a101 Mon Sep 17 00:00:00 2001
From: jvelezpo <sebasvelez@gmail.com>
Date: Thu, 25 Jan 2018 14:47:52 -0500
Subject: [PATCH] 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: https://github.com/nodejs/node/pull/18384
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
---
 test/parallel/test-child-process-execfile.js   | 8 ++++++++
 test/sequential/test-child-process-execsync.js | 6 ++++++
 2 files changed, 14 insertions(+)

diff --git a/test/parallel/test-child-process-execfile.js b/test/parallel/test-child-process-execfile.js
index 415489e9db8..af69d685789 100644
--- a/test/parallel/test-child-process-execfile.js
+++ b/test/parallel/test-child-process-execfile.js
@@ -7,6 +7,7 @@ const { getSystemErrorName } = require('util');
 const fixtures = require('../common/fixtures');
 
 const fixture = fixtures.path('exit.js');
+const execOpts = { encoding: 'utf8', shell: true };
 
 {
   execFile(
@@ -39,3 +40,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);
+  }));
+}
diff --git a/test/sequential/test-child-process-execsync.js b/test/sequential/test-child-process-execsync.js
index da80e80bce0..133217dcdf7 100644
--- a/test/sequential/test-child-process-execsync.js
+++ b/test/sequential/test-child-process-execsync.js
@@ -29,6 +29,7 @@ const TIMER = 200;
 const SLEEP = 2000;
 
 const start = Date.now();
+const execOpts = { encoding: 'utf8', shell: true };
 let err;
 let caught = false;
 
@@ -141,3 +142,8 @@ assert.strictEqual(ret, `${msg}\n`);
     return true;
   });
 }
+
+// Verify the shell option works properly
+assert.doesNotThrow(() => {
+  execFileSync(process.execPath, [], execOpts);
+});