Skip to content

Commit

Permalink
test: add debugging output to test-net-listen-after-destroy-stdin
Browse files Browse the repository at this point in the history
The test failed in CI once with a timeout but there is insufficient
information to further debug. Add additional debugging information.

Refactored callbacks to be arrow functions, since that seems to be the
direction we're moving.

PR-URL: nodejs#31698
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
  • Loading branch information
Trott committed Feb 10, 2020
1 parent 84836df commit 81af195
Showing 1 changed file with 7 additions and 7 deletions.
14 changes: 7 additions & 7 deletions test/parallel/test-net-listen-after-destroying-stdin.js
Original file line number Diff line number Diff line change
Expand Up @@ -8,15 +8,15 @@ const net = require('net');

process.stdin.destroy();

const server = net.createServer(common.mustCall(function(socket) {
console.log('accepted');
socket.end();
server.close();
const server = net.createServer(common.mustCall((socket) => {
console.log('accepted...');
socket.end(common.mustCall(() => { console.log('finished...'); }));
server.close(common.mustCall(() => { console.log('closed'); }));
}));


server.listen(0, function() {
server.listen(0, common.mustCall(() => {
console.log('listening...');

net.createConnection(this.address().port);
});
net.createConnection(server.address().port);
}));

0 comments on commit 81af195

Please sign in to comment.