From 4fbec0cd7ba3fefeff2cfdfae91c3b8e1f7eb5ea Mon Sep 17 00:00:00 2001 From: Rich Trott Date: Sat, 23 Mar 2019 09:35:09 -0700 Subject: [PATCH] test: use common.PORT instead of hardcoded number 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 Reviewed-By: Yongsheng Zhang Reviewed-By: Gireesh Punathil Reviewed-By: Daijiro Wachi --- test/sequential/test-https-connect-localport.js | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/test/sequential/test-https-connect-localport.js b/test/sequential/test-https-connect-localport.js index fc9e11f27e7188..d703d2813cd01a 100644 --- a/test/sequential/test-https-connect-localport.js +++ b/test/sequential/test-https-connect-localport.js @@ -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); })); }));