Browse Source

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

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>
v6.x
Artur G Vieira 8 years ago
committed by Myles Borins
parent
commit
d35648ffc2
No known key found for this signature in database GPG Key ID: 933B01F40B5CA946
  1. 4
      test/parallel/test-https-connect-address-family.js

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

@ -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,

Loading…
Cancel
Save