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: nodejs#26881
Fixes: nodejs#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 committed Mar 24, 2019
1 parent 0f7e24a commit 229759f
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 229759f

Please sign in to comment.