Skip to content
Closed
Changes from 1 commit
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
test: add debugging output to test-net-listen-after-destroy-stdin
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: #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 authored and BethGriggs committed Feb 25, 2020
commit 3f9cec3f51c19e57dfe85a87017cb210ad6d7bda
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);
}));