test: refactor test-async-wrap-getasyncid

santigimeno · addaleax · commit d8833762cb51 · 2018-03-05T10:07:17.000Z
So it does not use `common.PORT` anymore. PR-URL: #18727 Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de> Reviewed-By: Anatoli Papirovski <apapirovski@mac.com>
diff --git a/test/sequential/test-async-wrap-getasyncid.js b/test/sequential/test-async-wrap-getasyncid.js
@@ -284,9 +284,11 @@ if (common.hasCrypto) { // eslint-disable-line crypto-check
 testInitialized(handle, 'UDP');
 testUninitialized(req, 'SendWrap');
 
- handle.bind('0.0.0.0', common.PORT, undefined);
+ handle.bind('0.0.0.0', 0, undefined);
+ const addr = {};
+ handle.getsockname(addr);
 req.address = '127.0.0.1';
- req.port = common.PORT;
+ req.port = addr.port;
 req.oncomplete = () => handle.close();
 handle.send(req, [Buffer.alloc(1)], 1, req.port, req.address, true);
 testInitialized(req, 'SendWrap');

-Original file line number
+Diff line change
 testInitialized(handle,'UDP');
 testUninitialized(req,'SendWrap');
 -handle.bind('0.0.0.0',common.PORT,undefined);
 +handle.bind('0.0.0.0',0,undefined);
 +constaddr={};
 +handle.getsockname(addr);
 req.address='127.0.0.1';
 -req.port=common.PORT;
 +req.port=addr.port;
 req.oncomplete=()=>handle.close();
 handle.send(req,[Buffer.alloc(1)],1,req.port,req.address,true);
 testInitialized(req,'SendWrap');