Skip to content
Permalink
Browse files

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: #12915
Ref: #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>
  • Loading branch information...
arturgvieira authored and lpinca committed May 9, 2017
1 parent 529e4f2 commit 56812c81a370c4aa5937b79a40226b6fca060379
Showing with 2 additions and 2 deletions.
  1. +2 −2 test/parallel/test-https-connect-address-family.js
@@ -19,10 +19,10 @@ function runTest() {
https.createServer({ ciphers }, common.mustCall(function(req, res) {
this.close();
res.end();
})).listen(common.PORT, '::1', common.mustCall(function() {
})).listen(0, '::1', common.mustCall(function() {
const options = {
host: 'localhost',
port: common.PORT,
port: this.address().port,
family: 6,
ciphers: ciphers,
rejectUnauthorized: false,

0 comments on commit 56812c8

Please sign in to comment.
You can’t perform that action at this time.