1berry/nodejs
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

test: use common.PORT instead of hardcoded number

Browse Source 
In sequential/test-https-connect-localport, replace 34567 with
common.PORT.

PR-URL: https://github.com/nodejs/node/pull/26881
Fixes: https://github.com/https://github.com/nodejs/node/issues/26862
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Yongsheng Zhang <zyszys98@gmail.com>
Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
Reviewed-By: Daijiro Wachi <daijiro.wachi@gmail.com>
This commit is contained in:
Rich Trott 2019-03-23 09:35:09 -07:00
parent 0f7e24a0c3
commit 229759f961
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
test/sequential/test-https-connect-localport.js
View File

@ -22,10 +22,10 @@ const assert = require('assert');
pathname: '/', pathname: '/',
port, port,
family: 4, family: 4,
localPort: 34567, localPort: common.PORT,
rejectUnauthorized: false rejectUnauthorized: false
}, common.mustCall(() => { }, common.mustCall(() => {
assert.strictEqual(req.socket.localPort, 34567); assert.strictEqual(req.socket.localPort, common.PORT);
assert.strictEqual(req.socket.remotePort, port); assert.strictEqual(req.socket.remotePort, port);
})); }));
})); }));