Skip to content

Commit

Permalink
test: fix flaky test-cluster-net-listen-ipv6only-none
Browse files Browse the repository at this point in the history
test-cluster-net-listen-ipv6only-none was using port `0` for an
IPv6-only operation and assuming that the operating system would supply
a port that was also available in IPv4. However, CI results seem to
indicate that a port can be supplied that is in use by IPv4 but
available to IPv6, resulting in the test failing. Use `common.PORT` to
avoid this issue.

Fixes: nodejs#29679

PR-URL: nodejs#29681
Reviewed-By: Sam Roberts <vieuxtech@gmail.com>
  • Loading branch information
Trott committed Oct 1, 2019
1 parent 2487f39 commit 1c5a3f0
Showing 1 changed file with 10 additions and 11 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -17,13 +17,16 @@ const WORKER_ACCOUNT = 3;

if (cluster.isMaster) {
const workers = new Map();
let address;

const countdown = new Countdown(WORKER_ACCOUNT, () => {
// Make sure the `ipv6Only` option works.
// Make sure the `ipv6Only` option works. This is the part of the test that
// requires the whole test to use `common.PORT` rather than port `0`. If it
// used port `0` instead, then the operating system can supply a port that
// is available for the IPv6 interface but in use by the IPv4 interface.
// Refs: https://github.com/nodejs/node/issues/29679
const server = net.createServer().listen({
host: '0.0.0.0',
port: address.port,
port: common.PORT,
}, common.mustCall(() => {
// Exit.
server.close();
Expand All @@ -37,13 +40,9 @@ if (cluster.isMaster) {
const worker = cluster.fork().on('exit', common.mustCall((statusCode) => {
assert.strictEqual(statusCode, 0);
})).on('listening', common.mustCall((workerAddress) => {
if (!address) {
address = workerAddress;
} else {
assert.strictEqual(address.addressType, workerAddress.addressType);
assert.strictEqual(address.host, workerAddress.host);
assert.strictEqual(address.port, workerAddress.port);
}
assert.strictEqual(workerAddress.addressType, 6);
assert.strictEqual(workerAddress.address, host);
assert.strictEqual(workerAddress.port, common.PORT);
countdown.dec();
}));

Expand All @@ -52,7 +51,7 @@ if (cluster.isMaster) {
} else {
net.createServer().listen({
host,
port: 0,
port: common.PORT,
ipv6Only: true,
}, common.mustCall());
}

0 comments on commit 1c5a3f0

Please sign in to comment.