From f07300cfa32c197684fe44d39965770b2fcfc23d Mon Sep 17 00:00:00 2001
From: James M Snell <jasnell@gmail.com>
Date: Fri, 24 Jan 2025 17:35:59 -0800
Subject: [PATCH] test: move hasMultiLocalhost to common/net

Given that `common/net` already exists and hasMultiLocalhost
is net specific, let's move it out of common/index to better
encapsulate and simplify common/index more

PR-URL: https://github.com/nodejs/node/pull/56716
Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com>
Reviewed-By: Richard Lau <rlau@redhat.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
---
 test/common/README.md                       |  6 ------
 test/common/index.js                        | 10 ----------
 test/common/index.mjs                       |  2 --
 test/common/net.js                          | 10 ++++++++++
 test/parallel/test-http-localaddress.js     |  4 +++-
 test/parallel/test-http2-connect-options.js |  4 +++-
 test/parallel/test-https-localaddress.js    |  4 +++-
 7 files changed, 19 insertions(+), 21 deletions(-)

diff --git a/test/common/README.md b/test/common/README.md
index ee36503f920001..9ecee39b64a3df 100644
--- a/test/common/README.md
+++ b/test/common/README.md
@@ -238,12 +238,6 @@ Indicates if [internationalization][] is supported.
 
 Indicates whether `IPv6` is supported on this platform.
 
-### `hasMultiLocalhost`
-
-* [\<boolean>][<boolean>]
-
-Indicates if there are multiple localhosts available.
-
 ### `inFreeBSDJail`
 
 * [\<boolean>][<boolean>]
diff --git a/test/common/index.js b/test/common/index.js
index d2c39578324600..e8bf65d0a6edb4 100644
--- a/test/common/index.js
+++ b/test/common/index.js
@@ -489,15 +489,6 @@ function _mustCallInner(fn, criteria = 1, field) {
   return _return;
 }
 
-function hasMultiLocalhost() {
-  const { internalBinding } = require('internal/test/binding');
-  const { TCP, constants: TCPConstants } = internalBinding('tcp_wrap');
-  const t = new TCP(TCPConstants.SOCKET);
-  const ret = t.bind('127.0.0.2', 0);
-  t.close();
-  return ret === 0;
-}
-
 function skipIfEslintMissing() {
   if (!fs.existsSync(
     path.join(__dirname, '..', '..', 'tools', 'eslint', 'node_modules', 'eslint'),
@@ -965,7 +956,6 @@ const common = {
   hasIntl,
   hasCrypto,
   hasQuic,
-  hasMultiLocalhost,
   invalidArgTypeHelper,
   isAlive,
   isASan,
diff --git a/test/common/index.mjs b/test/common/index.mjs
index 23328ac90ea3c9..090659f93be8ef 100644
--- a/test/common/index.mjs
+++ b/test/common/index.mjs
@@ -20,7 +20,6 @@ const {
   hasCrypto,
   hasIntl,
   hasIPv6,
-  hasMultiLocalhost,
   isAIX,
   isAlive,
   isDumbTerminal,
@@ -75,7 +74,6 @@ export {
   hasCrypto,
   hasIntl,
   hasIPv6,
-  hasMultiLocalhost,
   isAIX,
   isAlive,
   isDumbTerminal,
diff --git a/test/common/net.js b/test/common/net.js
index 84eddd0966ed56..3886c542421005 100644
--- a/test/common/net.js
+++ b/test/common/net.js
@@ -17,7 +17,17 @@ function checkSupportReusePort() {
   });
 }
 
+function hasMultiLocalhost() {
+  const { internalBinding } = require('internal/test/binding');
+  const { TCP, constants: TCPConstants } = internalBinding('tcp_wrap');
+  const t = new TCP(TCPConstants.SOCKET);
+  const ret = t.bind('127.0.0.2', 0);
+  t.close();
+  return ret === 0;
+}
+
 module.exports = {
   checkSupportReusePort,
+  hasMultiLocalhost,
   options,
 };
diff --git a/test/parallel/test-http-localaddress.js b/test/parallel/test-http-localaddress.js
index a0e4bb80a3f8c2..da25ab3047613f 100644
--- a/test/parallel/test-http-localaddress.js
+++ b/test/parallel/test-http-localaddress.js
@@ -22,8 +22,10 @@
 // Flags: --expose-internals
 'use strict';
 const common = require('../common');
-if (!common.hasMultiLocalhost())
+const { hasMultiLocalhost } = require('../common/net');
+if (!hasMultiLocalhost()) {
   common.skip('platform-specific test.');
+}
 
 const http = require('http');
 const assert = require('assert');
diff --git a/test/parallel/test-http2-connect-options.js b/test/parallel/test-http2-connect-options.js
index 233ced016974e2..1abcee99e06433 100644
--- a/test/parallel/test-http2-connect-options.js
+++ b/test/parallel/test-http2-connect-options.js
@@ -4,8 +4,10 @@ const common = require('../common');
 if (!common.hasCrypto)
   common.skip('missing crypto');
 
-if (!common.hasMultiLocalhost())
+const { hasMultiLocalhost } = require('../common/net');
+if (!hasMultiLocalhost()) {
   common.skip('platform-specific test.');
+}
 
 const http2 = require('http2');
 const assert = require('assert');
diff --git a/test/parallel/test-https-localaddress.js b/test/parallel/test-https-localaddress.js
index 0de0974dc69b04..2a4629b34e4105 100644
--- a/test/parallel/test-https-localaddress.js
+++ b/test/parallel/test-https-localaddress.js
@@ -25,8 +25,10 @@ const common = require('../common');
 if (!common.hasCrypto)
   common.skip('missing crypto');
 
-if (!common.hasMultiLocalhost())
+const { hasMultiLocalhost } = require('../common/net');
+if (!hasMultiLocalhost()) {
   common.skip('platform-specific test.');
+}
 
 const fixtures = require('../common/fixtures');
 const assert = require('assert');