1berry/nodejs
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

test: remove message argument in cluster setup test

Browse Source 
In test/parallel/test-cluster-setup-master-cumulative.js:

Remove the message parameter to ensure that the compared
objects are printed out. Add the original message as a
comment above.

PR-URL: https://github.com/nodejs/node/pull/16838
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
This commit is contained in:
mbornath 2017-11-06 18:01:27 +01:00 committed by Rich Trott
parent a77bfcaac7
commit 6563e56aef
1 changed files with 2 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
test/parallel/test-cluster-setup-master-cumulative.js
View File

@ -26,11 +26,8 @@ const cluster = require('cluster');
assert(cluster.isMaster);
assert.deepStrictEqual(
cluster.settings,
{},
'cluster.settings should not be initialized until needed'
);
// cluster.settings should not be initialized until needed
assert.deepStrictEqual(cluster.settings, {});
cluster.setupMaster();
assert.deepStrictEqual(cluster.settings, {