test: add debugging output to test-net-listen-after-destroy-stdin

Trott · codebytere · commit 2c0f3028c9ed · 2020-02-17T12:33:02.000-08:00
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>
diff --git a/test/parallel/test-net-listen-after-destroying-stdin.js b/test/parallel/test-net-listen-after-destroying-stdin.js
@@ -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);
+}));
