Skip to content

Commit

Permalink
test: track callback invocations
Browse files Browse the repository at this point in the history
Use `common.mustCall()` and `common.mustNotCall()` to check that
callbacks are invoked the expected number of times in
test-net-listen-shared-ports.

PR-URL: #13010
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Gibson Fahnestock <gibfahn@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Refael Ackermann <refack@gmail.com>
  • Loading branch information
Trott authored and addaleax committed May 18, 2017
1 parent 28ddac2 commit 7e5ed8b
Showing 1 changed file with 14 additions and 12 deletions.
26 changes: 14 additions & 12 deletions test/sequential/test-net-listen-shared-ports.js
Original file line number Diff line number Diff line change
Expand Up @@ -28,19 +28,19 @@ const net = require('net');
if (cluster.isMaster) {
const worker1 = cluster.fork();

worker1.on('message', function(msg) {
worker1.on('message', common.mustCall(function(msg) {
assert.strictEqual(msg, 'success');
const worker2 = cluster.fork();

worker2.on('message', function(msg) {
worker2.on('message', common.mustCall(function(msg) {
assert.strictEqual(msg, 'server2:EADDRINUSE');
worker1.kill();
worker2.kill();
});
});
}));
}));
} else {
const server1 = net.createServer(common.noop);
const server2 = net.createServer(common.noop);
const server1 = net.createServer(common.mustNotCall());
const server2 = net.createServer(common.mustNotCall());

server1.on('error', function(err) {
// no errors expected
Expand All @@ -56,10 +56,12 @@ if (cluster.isMaster) {
host: 'localhost',
port: common.PORT,
exclusive: false
}, function() {
server2.listen({port: common.PORT + 1, exclusive: true}, function() {
// the first worker should succeed
process.send('success');
});
});
}, common.mustCall(function() {
server2.listen({port: common.PORT + 1, exclusive: true},
common.mustCall(function() {
// the first worker should succeed
process.send('success');
})
);
}));
}

0 comments on commit 7e5ed8b

Please sign in to comment.