1berry/nodejs
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

test: use dynamic port in test-https-connect-address-family

Browse Source 
Remove common.PORT from test-https-connect-address-family to eliminate
possibility that a dynamic port used in another test will collide with
common.PORT.

PR-URL: https://github.com/nodejs/node/pull/12915
Ref: https://github.com/nodejs/node/issues/12376
Reviewed-By: Sakthipriyan Vairamani <thechargingvolcano@gmail.com>
Reviewed-By: Yuta Hiroto <hello@about-hiroppy.com>
Reviewed-By: Gibson Fahnestock <gibfahn@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
This commit is contained in:
Artur G Vieira 2017-05-09 06:15:36 +00:00 committed by Luigi Pinca
parent 529e4f206a
commit 56812c81a3
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
test/parallel/test-https-connect-address-family.js
View File

@ -19,10 +19,10 @@ function runTest() {
https.createServer({ ciphers }, common.mustCall(function(req, res) { https.createServer({ ciphers }, common.mustCall(function(req, res) {
this.close(); this.close();
res.end(); res.end();
})).listen(common.PORT, '::1', common.mustCall(function() { })).listen(0, '::1', common.mustCall(function() {
const options = { const options = {
host: 'localhost', host: 'localhost',
port: common.PORT, port: this.address().port,
family: 6, family: 6,
ciphers: ciphers, ciphers: ciphers,
rejectUnauthorized: false, rejectUnauthorized: false,