Skip to content

Commit

Permalink
test: use common.PORT instead of hardcoded number
Browse files Browse the repository at this point in the history
In sequential/test-https-connect-localport, replace 34567 with
common.PORT.

PR-URL: #26881
Fixes: #26862
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Yongsheng Zhang <[email protected]>
Reviewed-By: Gireesh Punathil <[email protected]>
Reviewed-By: Daijiro Wachi <[email protected]>
  • Loading branch information
Trott authored and targos committed Mar 27, 2019
1 parent 4919e4b commit 011c205
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions test/sequential/test-https-connect-localport.js
Original file line number Diff line number Diff line change
Expand Up @@ -22,10 +22,10 @@ const assert = require('assert');
pathname: '/',
port,
family: 4,
localPort: 34567,
localPort: common.PORT,
rejectUnauthorized: false
}, common.mustCall(() => {
assert.strictEqual(req.socket.localPort, 34567);
assert.strictEqual(req.socket.localPort, common.PORT);
assert.strictEqual(req.socket.remotePort, port);
}));
}));
Expand Down

0 comments on commit 011c205

Please sign in to comment.