test: use spread instead of object.assign

19shubham11 · targos · commit 6c7e4918055a · 2019-12-01T17:08:34.000+01:00
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>
diff --git a/test/parallel/test-child-process-fork-args.js b/test/parallel/test-child-process-fork-args.js
@@ -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(
diff --git a/test/parallel/test-child-process-fork-exec-path.js b/test/parallel/test-child-process-fork-exec-path.js
@@ -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);

-Original file line number
+Diff line change
 argsLists.forEach((args)=>{
 constcp=fork(fixtures.path('child-process-echo-options.js'),args,{
 -env: Object.assign({},process.env,expectedEnv)
 +env: { ...process.env,...expectedEnv}
 });
 cp.on(
-Original file line number
+Diff line change
 fs.copyFileSync(nodePath,copyPath,fs.constants.COPYFILE_FICLONE);
 fs.chmodSync(copyPath,'0755');
 -constenvCopy=Object.assign({},process.env,{'FORK': 'true'});
 +constenvCopy={ ...process.env,FORK: 'true'};
 constchild=fork(__filename,{execPath: copyPath,env: envCopy});
 child.on('message',common.mustCall(function(recv){
 assert.deepStrictEqual(recv,msg);