Skip to content

Commit

Permalink
test: use port number from env in tls socket test
Browse files Browse the repository at this point in the history
Tests normally use common.PORT to allow the user to select which port
number to listen on. Hardcoding the port number will cause parallel
instances of the test to fail.

PR-URL: #3557
Reviewed-By: Ben Noordhuis <[email protected]>
Reviewed-By: Fedor Indutny <[email protected]>
Reviewed-By: Johan Bergström <[email protected]>
  • Loading branch information
stefanmb authored and bnoordhuis committed Oct 28, 2015
1 parent 5d5a4c4 commit c9e682d
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions test/parallel/test-tls-async-cb-after-socket-end.js
Original file line number Diff line number Diff line change
Expand Up @@ -36,9 +36,9 @@ server.on('resumeSession', function(id, cb) {
next();
});

server.listen(1443, function() {
server.listen(common.PORT, function() {
var clientOpts = {
port: 1443,
port: common.PORT,
rejectUnauthorized: false,
session: false
};
Expand Down

0 comments on commit c9e682d

Please sign in to comment.