test: replace Object.assign with object spread

Replaces Object.assign with spread where object is simply cloned
PR-URL: #30306
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: Сковорода Никита Андреевич <chalkerx@gmail.com>
Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
Reviewed-By: James M Snell <jasnell@gmail.com>

Author: Grigoriy Levanov

test/parallel/test-trace-events-async-hooks-dynamic.js

@@ -29,10 +29,10 @@ const proc = cp.spawnSync(
 ['-e',enable+code],
 {
 cwd: tmpdir.path,
-env: Object.assign({},process.env,{
-'NODE_DEBUG_NATIVE': 'tracing',
-'NODE_DEBUG': 'tracing'
-})
+env: { ...process.env,
+'NODE_DEBUG_NATIVE': 'tracing',
+'NODE_DEBUG': 'tracing'
+}
 });
 
 console.log('process exit with signal:',proc.signal);

test/parallel/test-trace-events-async-hooks-worker.js

@@ -36,10 +36,10 @@ const proc = cp.spawnSync(
 ['--trace-event-categories','node.async_hooks','-e',worker],
 {
 cwd: tmpdir.path,
-env: Object.assign({},process.env,{
-'NODE_DEBUG_NATIVE': 'tracing',
-'NODE_DEBUG': 'tracing'
-})
+env: { ...process.env,
+'NODE_DEBUG_NATIVE': 'tracing',
+'NODE_DEBUG': 'tracing'
+}
 });
 
 console.log('process exit with signal:',proc.signal);

test/parallel/test-util-inspect.js

@@ -1363,7 +1363,7 @@ if (typeof Symbol !== 'undefined') {
 constarr=newArray(101).fill();
 constobj={a: {a: {a: {a: 1}}}};
 
-constoldOptions=Object.assign({},util.inspect.defaultOptions);
+constoldOptions={ ...util.inspect.defaultOptions};
 
 // Set single option through property assignment.
 util.inspect.defaultOptions.maxArrayLength=null;

test/parallel/test-whatwg-url-custom-properties.js

@@ -152,7 +152,7 @@ assert.strictEqual(url.searchParams, oldParams);
 // contains the Symbols that Node uses for brand checking, but not the data
 // properties, which are getters. Verify that urlToOptions() can handle such
 // a case.
-constcopiedUrlObj=Object.assign({},urlObj);
+constcopiedUrlObj={ ...urlObj};
 constcopiedOpts=urlToOptions(copiedUrlObj);
 assert.strictEqual(copiedOptsinstanceofURL,false);
 assert.strictEqual(copiedOpts.protocol,undefined);

test/sequential/test-async-wrap-getasyncid.js

@@ -8,7 +8,7 @@ const fs = require('fs');
 constv8=require('v8');
 constfsPromises=fs.promises;
 constnet=require('net');
-constproviders=Object.assign({},internalBinding('async_wrap').Providers);
+constproviders={ ...internalBinding('async_wrap').Providers};
 constfixtures=require('../common/fixtures');
 consttmpdir=require('../common/tmpdir');
 const{ getSystemErrorName }=require('util');

test/sequential/test-inspector-open.js

@@ -14,7 +14,7 @@ if (process.env.BE_CHILD)
 returnbeChild();
 
 constchild=fork(__filename,
-{env: Object.assign({},process.env,{BE_CHILD: 1})});
+{env: { ...process.env,BE_CHILD: 1}});
 
 child.once('message',common.mustCall((msg)=>{
 assert.strictEqual(msg.cmd,'started');

test/sequential/test-inspector-port-cluster.js

@@ -328,11 +328,11 @@ function workerProcessMain() {
 functionspawnMaster({ execArgv, workers, clusterSettings ={}}){
 returnnewPromise((resolve)=>{
 childProcess.fork(__filename,{
-env: Object.assign({},process.env,{
-workers: JSON.stringify(workers),
-clusterSettings: JSON.stringify(clusterSettings),
-testProcess: true
-}),
+env: { ...process.env,
+workers: JSON.stringify(workers),
+clusterSettings: JSON.stringify(clusterSettings),
+testProcess: true
+},
 execArgv: execArgv.concat(['--expose-internals'])
 }).on('exit',common.mustCall((code,signal)=>{
 checkExitCode(code,signal);