Skip to content

Commit d27e463

Browse files
legendecastargos
authored andcommitted
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]>
1 parent f8feb02 commit d27e463

File tree

1 file changed

+4
-6
lines changed

1 file changed

+4
-6
lines changed

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

+4-6
Original file line numberDiff line numberDiff line change
@@ -26,23 +26,21 @@ const os = require('os');
2626

2727
const spawn = require('child_process').spawn;
2828

29-
const env = {
29+
const env = Object.assign({}, process.env, {
3030
'HELLO': 'WORLD',
3131
'UNDEFINED': undefined,
3232
'NULL': null,
3333
'EMPTY': ''
34-
};
34+
});
3535
Object.setPrototypeOf(env, {
3636
'FOO': 'BAR'
3737
});
3838

3939
let child;
4040
if (common.isWindows) {
41-
child = spawn('cmd.exe', ['/c', 'set'],
42-
Object.assign({}, process.env, { env }));
41+
child = spawn('cmd.exe', ['/c', 'set'], { env });
4342
} else {
44-
child = spawn('/usr/bin/env', [],
45-
Object.assign({}, process.env, { env }));
43+
child = spawn('/usr/bin/env', [], { env });
4644
}
4745

4846

0 commit comments

Comments
 (0)