1berry/nodejs
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

test: fix test-cluster-dgram-1 flakiness

Browse Source 
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 <cjihrig@gmail.com>
Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com>
Reviewed By: James M Snell <jasnell@gmail.com>
This commit is contained in:
Santiago Gimeno 2016-09-02 13:14:37 +02:00 committed by mhdawson
parent 8361c26cd5
commit 2d2a2d7c78
1 changed files with 3 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
test/parallel/test-cluster-dgram-1.js
View File

@ -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));