test: fix test-cluster-dgram-1 flakiness

Check for the number of messages received in the `exit` event listener instead of the `disconnect` listener. Fixes: #8380 PR-URL: #8383 Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Michael Dawson <[email protected]> Reviewed By: James M Snell <[email protected]>
nodejs · Sep 9, 2016 · f0300ed · f0300ed
1 parent 6d421a2
commit f0300ed
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/test/parallel/test-cluster-dgram-1.js b/test/parallel/test-cluster-dgram-1.js
@@ -64,9 +64,10 @@ function master() {
 
     worker.on('message', common.mustCall((msg) => {
       received = msg.received;
+      worker.disconnect();
     }));
 
-    worker.on('disconnect', common.mustCall(() => {
+    worker.on('exit', common.mustCall(() => {
       assert.strictEqual(received, PACKETS_PER_WORKER);
     }));
   }
@@ -85,7 +86,7 @@ function worker() {
     // Every 10 messages, notify the master.
     if (received === PACKETS_PER_WORKER) {
       process.send({received: received});
-      process.disconnect();
+      socket.close();
     }
   }, PACKETS_PER_WORKER));