From 4252c481df7f99fad69f00b437f7bd3896ff0e2f Mon Sep 17 00:00:00 2001 From: mbornath Date: Mon, 6 Nov 2017 18:01:27 +0100 Subject: [PATCH] test: remove message argument in cluster setup test 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 Reviewed-By: Luigi Pinca Reviewed-By: Gireesh Punathil Reviewed-By: Rich Trott --- test/parallel/test-cluster-setup-master-cumulative.js | 7 ++----- 1 file changed, 2 insertions(+), 5 deletions(-) diff --git a/test/parallel/test-cluster-setup-master-cumulative.js b/test/parallel/test-cluster-setup-master-cumulative.js index 977c89a56aa573..76f66e8fea13dd 100644 --- a/test/parallel/test-cluster-setup-master-cumulative.js +++ b/test/parallel/test-cluster-setup-master-cumulative.js @@ -5,11 +5,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, {