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: https://github.com/nodejs/node/pull/16729
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Eugene Ostroukhov <eostroukhov@google.com>
Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Michaël Zasso <targos@protonmail.com>

1 files changed, 28 insertions, 0 deletions

diff --git a/test/parallel/test-cluster-disconnect-exitedAfterDisconnect-race.js b/test/parallel/test-cluster-disconnect-exitedAfterDisconnect-race.js
new file mode 100644
index 0000000000..db2aa6d34a
--- /dev/null
+++ b/test/parallel/test-cluster-disconnect-exitedAfterDisconnect-race.js
@@ -0,0 +1,28 @@
+'use strict';
+
+const common = require('../common');
+
+// Test should fail in Node.js 5.4.1 and pass in later versions.
+
+const assert = require('assert');
+const cluster = require('cluster');
+
+if (cluster.isMaster) {
+  cluster.on('exit', (worker, code) => {
+    assert.strictEqual(code, 0, `worker exited with code: ${code}, expected 0`);
+  });
+
+  return cluster.fork();
+}
+
+let eventFired = false;
+
+cluster.worker.disconnect();
+
+process.nextTick(common.mustCall(() => {
+  assert.ok(!eventFired, 'disconnect event should wait for ack');
+}));
+
+cluster.worker.on('disconnect', common.mustCall(() => {
+  eventFired = true;
+}));