Skip to content

Commit

Permalink
test: fix flaky cluster-disconnect-race
Browse files Browse the repository at this point in the history
On single core Windows systems, process.send() would cause an
EPIPE because of the ordering of the IPC channel disconnect and
the process.send().

The test was originally only relevant for non-Windows platforms,
so this commit merely skips the test on Windows.

Fixes: nodejs#4450
PR-URL: nodejs#4457
Reviewed-By: Rich Trott <rtrott@gmail.com>
  • Loading branch information
mscdex committed Dec 29, 2015
1 parent 3809126 commit fd551c3
Showing 1 changed file with 8 additions and 2 deletions.
10 changes: 8 additions & 2 deletions test/parallel/test-cluster-disconnect-race.js
Original file line number Diff line number Diff line change
Expand Up @@ -7,6 +7,12 @@ const common = require('../common');
const assert = require('assert');
const net = require('net');
const cluster = require('cluster');

if (common.isWindows) {
console.log('1..0 # Skipped: This test does not apply to Windows.');
return;
}

cluster.schedulingPolicy = cluster.SCHED_NONE;

if (cluster.isMaster) {
Expand All @@ -19,9 +25,9 @@ if (cluster.isMaster) {
worker2.on('online', common.mustCall(worker2.disconnect));
}));

cluster.on('exit', function(worker, code) {
cluster.on('exit', common.mustCall(function(worker, code) {
assert.strictEqual(code, 0, 'worker exited with error');
});
}, 2));

return;
}
Expand Down

0 comments on commit fd551c3

Please sign in to comment.