test: refactor test-net-server-bind

* Use common.mustNotCall() and common.mustCall() as appropriate * Use block scoping * Move assertions out of `exit` handler and into callbacks * Order assert.strictEqual() args per docs * Remove console.log() calls * Move test from `parallel` to `sequential` so `common.PORT` can be used without conflicting with OS-provided ports in other `parallel` tests PR-URL: #13273 Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: James M Snell <[email protected]> Reviewed-By: Luigi Pinca <[email protected]>

Author: Trott

test/parallel/test-net-server-bind.js

@@ -0,0 +1,64 @@
+'use strict';
+constcommon=require('../common');
+constassert=require('assert');
+constnet=require('net');
+
+
+// With only a callback, server should get a port assigned by the OS
+{
+constserver=net.createServer(common.mustNotCall());
+
+server.listen(common.mustCall(function(){
+assert.ok(server.address().port>100);
+server.close();
+}));
+}
+
+// No callback to listen(), assume we can bind in 100 ms
+{
+constserver=net.createServer(common.mustNotCall());
+
+server.listen(common.PORT);
+
+setTimeout(function(){
+constaddress=server.address();
+assert.strictEqual(address.port,common.PORT);
+
+if(address.family==='IPv6')
+assert.strictEqual(server._connectionKey,`6::::${address.port}`);
+else
+assert.strictEqual(server._connectionKey,`4:0.0.0.0:${address.port}`);
+
+server.close();
+},100);
+}
+
+// Callback to listen()
+{
+constserver=net.createServer(common.mustNotCall());
+
+server.listen(common.PORT+1,common.mustCall(function(){
+assert.strictEqual(server.address().port,common.PORT+1);
+server.close();
+}));
+}
+
+// Backlog argument
+{
+constserver=net.createServer(common.mustNotCall());
+
+server.listen(common.PORT+2,'0.0.0.0',127,common.mustCall(function(){
+assert.strictEqual(server.address().port,common.PORT+2);
+server.close();
+}));
+}
+
+// Backlog argument without host argument
+{
+constserver=net.createServer(common.mustNotCall());
+
+server.listen(common.PORT+3,127,common.mustCall(function(){
+assert.strictEqual(server.address().port,common.PORT+3);
+server.close();
+}));
+}