test: refactor exitedAfterDisconnect test

* minor refactoring to conform with test-writing layout guidelines * rename test to use current terminology rather than deprecated terminology * assert.strictEqual() -> assert.ok() PR-URL: #16729 Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Eugene Ostroukhov <[email protected]> Reviewed-By: Gireesh Punathil <[email protected]> Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Michaël Zasso <[email protected]>
nodejs · Nov 6, 2017 · 0ab3d37 · 0ab3d37
1 parent 72d0e7e
commit 0ab3d37
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/...l/test-cluster-disconnect-suicide-race.js → ...-disconnect-exitedAfterDisconnect-race.js b/...l/test-cluster-disconnect-suicide-race.js → ...-disconnect-exitedAfterDisconnect-race.js
@@ -1,8 +1,9 @@
 'use strict';
 
+const common = require('../common');
+
 // Test should fail in Node.js 5.4.1 and pass in later versions.
 
-const common = require('../common');
 const assert = require('assert');
 const cluster = require('cluster');
 
@@ -19,7 +20,7 @@ let eventFired = false;
 cluster.worker.disconnect();
 
 process.nextTick(common.mustCall(() => {
-  assert.strictEqual(eventFired, false, 'disconnect event should wait for ack');
+  assert.ok(!eventFired, 'disconnect event should wait for ack');
 }));
 
 cluster.worker.on('disconnect', common.mustCall(() => {