From 2d2a2d7c780b71adec00096b7dd836714816ba54 Mon Sep 17 00:00:00 2001 From: Santiago Gimeno Date: Fri, 2 Sep 2016 13:14:37 +0200 Subject: [PATCH] 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: https://github.com/nodejs/node/issues/8380 PR-URL: https://github.com/nodejs/node/pull/8383 Reviewed-By: Colin Ihrig Reviewed-By: Michael Dawson Reviewed By: James M Snell --- test/parallel/test-cluster-dgram-1.js | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/test/parallel/test-cluster-dgram-1.js b/test/parallel/test-cluster-dgram-1.js index 30420850a37..3b27fa8e20c 100644 --- 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));