test: use common.PORT instead of an extraneous variable

This test is not parallelized and so we can use the test commons PORT variable. Refs: #27565 (comment) PR-URL: #27565 Fixes: #27341 Reviewed-By: Rich Trott <[email protected]> Reviewed-By: James M Snell <[email protected]> Reviewed-By: Ruben Bridgewater <[email protected]>
nodejs · May 14, 2019 · 9ed5882 · 9ed5882
1 parent f01183c
commit 9ed5882
Showing 1 changed file with 3 additions and 5 deletions.
diff --git a/test/internet/test-dgram-connect.js b/test/internet/test-dgram-connect.js
@@ -5,17 +5,15 @@ const { addresses } = require('../common/internet');
 const assert = require('assert');
 const dgram = require('dgram');
 
-const PORT = 12345;
-
 const client = dgram.createSocket('udp4');
-client.connect(PORT, addresses.INVALID_HOST, common.mustCall((err) => {
+client.connect(common.PORT, addresses.INVALID_HOST, common.mustCall((err) => {
   assert.ok(err.code === 'ENOTFOUND' || err.code === 'EAI_AGAIN');
 
   client.once('error', common.mustCall((err) => {
     assert.ok(err.code === 'ENOTFOUND' || err.code === 'EAI_AGAIN');
     client.once('connect', common.mustCall(() => client.close()));
-    client.connect(PORT);
+    client.connect(common.PORT);
   }));
 
-  client.connect(PORT, addresses.INVALID_HOST);
+  client.connect(common.PORT, addresses.INVALID_HOST);
 }));