Skip to content

Commit

Permalink
test: remove message argument in cluster setup test
Browse files Browse the repository at this point in the history
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: nodejs#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>
  • Loading branch information
mbornath authored and Trott committed Nov 7, 2017
1 parent a77bfca commit 6563e56
Showing 1 changed file with 2 additions and 5 deletions.
7 changes: 2 additions & 5 deletions test/parallel/test-cluster-setup-master-cumulative.js
Original file line number Diff line number Diff line change
Expand Up @@ -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, {
Expand Down

0 comments on commit 6563e56

Please sign in to comment.