test: properly extend process.env in child_process

PR-URL: #22430 Reviewed-By: Anna Henningsen <[email protected]> Reviewed-By: James M Snell <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Ruben Bridgewater <[email protected]> Reviewed-By: Trivikram Kamat <[email protected]> Reviewed-By: George Adams <[email protected]>
nodejs · Aug 28, 2018 · b058601 · b058601
1 parent 13d3d48
commit b058601
Showing 1 changed file with 4 additions and 6 deletions.
diff --git a/test/parallel/test-child-process-env.js b/test/parallel/test-child-process-env.js
@@ -26,23 +26,21 @@ const os = require('os');
 
 const spawn = require('child_process').spawn;
 
-const env = {
+const env = Object.assign({}, process.env, {
   'HELLO': 'WORLD',
   'UNDEFINED': undefined,
   'NULL': null,
   'EMPTY': ''
-};
+});
 Object.setPrototypeOf(env, {
   'FOO': 'BAR'
 });
 
 let child;
 if (common.isWindows) {
-  child = spawn('cmd.exe', ['/c', 'set'],
-                Object.assign({}, process.env, { env }));
+  child = spawn('cmd.exe', ['/c', 'set'], { env });
 } else {
-  child = spawn('/usr/bin/env', [],
-                Object.assign({}, process.env, { env }));
+  child = spawn('/usr/bin/env', [], { env });
 }