Skip to content

Commit

Permalink
test: use dynamic port in test-https-connect-address-family
Browse files Browse the repository at this point in the history
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 MylesBorins committed Jul 11, 2017
1 parent b854d27 commit d35648f
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions test/parallel/test-https-connect-address-family.js
Original file line number Diff line number Diff line change
Expand Up @@ -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,
Expand Down

0 comments on commit d35648f

Please sign in to comment.