mirror of https://github.com/nodejs/node.git
test: replace Object.assign with object spread
Replaces Object.assign with spread where object is simply cloned PR-URL: https://github.com/nodejs/node/pull/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>pull/30459/head
parent
1e0679e56d
commit
35ec01097b
|
@ -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);
|
||||
|
|
|
@ -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);
|
||||
|
|
|
@ -1366,7 +1366,7 @@ if (typeof Symbol !== 'undefined') {
|
|||
const arr = new Array(101).fill();
|
||||
const obj = { a: { a: { a: { a: 1 } } } };
|
||||
|
||||
const oldOptions = Object.assign({}, util.inspect.defaultOptions);
|
||||
const oldOptions = { ...util.inspect.defaultOptions };
|
||||
|
||||
// Set single option through property assignment.
|
||||
util.inspect.defaultOptions.maxArrayLength = null;
|
||||
|
|
|
@ -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.
|
||||
const copiedUrlObj = Object.assign({}, urlObj);
|
||||
const copiedUrlObj = { ...urlObj };
|
||||
const copiedOpts = urlToOptions(copiedUrlObj);
|
||||
assert.strictEqual(copiedOpts instanceof URL, false);
|
||||
assert.strictEqual(copiedOpts.protocol, undefined);
|
||||
|
|
|
@ -8,7 +8,7 @@ const fs = require('fs');
|
|||
const v8 = require('v8');
|
||||
const fsPromises = fs.promises;
|
||||
const net = require('net');
|
||||
const providers = Object.assign({}, internalBinding('async_wrap').Providers);
|
||||
const providers = { ...internalBinding('async_wrap').Providers };
|
||||
const fixtures = require('../common/fixtures');
|
||||
const tmpdir = require('../common/tmpdir');
|
||||
const { getSystemErrorName } = require('util');
|
||||
|
|
|
@ -14,7 +14,7 @@ if (process.env.BE_CHILD)
|
|||
return beChild();
|
||||
|
||||
const child = 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');
|
||||
|
|
|
@ -328,11 +328,11 @@ function workerProcessMain() {
|
|||
function spawnMaster({ execArgv, workers, clusterSettings = {} }) {
|
||||
return new Promise((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);
|
||||
|
|
Loading…
Reference in New Issue