Skip to content
Permalink
Browse files

test: use spread instead of object.assign

PR-URL: #30412
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
  • Loading branch information
19shubham11 authored and MylesBorins committed Nov 12, 2019
1 parent e7f1d57 commit 78c7118ab774ef0597f55b2bf022010b3a731381
@@ -64,7 +64,7 @@ const expectedEnv = { foo: 'bar' };

argsLists.forEach((args) => {
const cp = fork(fixtures.path('child-process-echo-options.js'), args, {
env: Object.assign({}, process.env, expectedEnv)
env: { ...process.env, ...expectedEnv }
});

cp.on(
@@ -51,7 +51,7 @@ assert.strictEqual(fs.existsSync(copyPath), false);
fs.copyFileSync(nodePath, copyPath, fs.constants.COPYFILE_FICLONE);
fs.chmodSync(copyPath, '0755');

const envCopy = Object.assign({}, process.env, { 'FORK': 'true' });
const envCopy = { ...process.env, FORK: 'true' };
const child = fork(__filename, { execPath: copyPath, env: envCopy });
child.on('message', common.mustCall(function(recv) {
assert.deepStrictEqual(recv, msg);

0 comments on commit 78c7118

Please sign in to comment.
You can’t perform that action at this time.