cluster: Using common.mustCall to wrap callback in test-inspect-suppo…

…rt-for-node_options Using common.mustCall to wrap callback and added assert to replace if-fail in test-inspect-support-for-node_options. PR nodejs#19165 Fixes : nodejs#19026
sameer-coder · Mar 16, 2018 · d2cf4a4 · d2cf4a4
1 parent ff81da0
commit d2cf4a4
Showing 1 changed file with 4 additions and 5 deletions.
diff --git a/test/parallel/test-inspect-support-for-node_options.js b/test/parallel/test-inspect-support-for-node_options.js
@@ -21,10 +21,9 @@ function checkForInspectSupport(flag) {
       worker.disconnect();
     });
 
-    cluster.on('exit', (worker, code, signal) => {
-      if (worker.exitedAfterDisconnect === false) {
-        assert.fail(`For NODE_OPTIONS ${nodeOptions}, failed to start cluster`);
-      }
-    });
+    cluster.on('exit', common.mustCall((worker, code, signal) => {
+      const errMsg = `For NODE_OPTIONS ${nodeOptions}, failed to start cluster`;
+      assert.strictEqual(worker.exitedAfterDisconnect, true, errMsg);
+    }, 2));
   }
 }