test: add tests for 3 methods in utils

Signed-off-by: Daijiro Wachi <daijiro.wachi@gmail.com>
PR-URL: #63765
Refs: https://app.codecov.io/gh/nodejs/node/blob/main/lib%2Futil.js
Reviewed-By: Chemi Atlow <chemi@atlow.co.il>

Author: watilde

test/fixtures/source-map/get-call-sites-mapped.js

@@ -0,0 +1,6 @@
+{
+ "version": 3,
+ "sources": ["get-call-sites-original.js"],
+ "names": [],
+ "mappings": "AAUA"
+}

test/fixtures/source-map/get-call-sites-mapped.map

@@ -0,0 +1,57 @@
+'use strict';
+
+require('../common');
+constassert=require('node:assert');
+constos=require('node:os');
+constutil=require('node:util');
+
+constENOENT=-os.constants.errno.ENOENT;
+constEACCES=-os.constants.errno.EACCES;
+
+// Address and port are included in the message
+{
+conste=util._exceptionWithHostPort(ENOENT,'connect','127.0.0.1',8080);
+assert.ok(einstanceofError);
+assert.strictEqual(e.errno,ENOENT);
+assert.strictEqual(e.code,util.getSystemErrorName(ENOENT));
+assert.strictEqual(e.syscall,'connect');
+assert.strictEqual(e.address,'127.0.0.1');
+assert.strictEqual(e.port,8080);
+assert.ok(e.message.includes('127.0.0.1:8080'),`expected 127.0.0.1:8080 in "${e.message}"`);
+}
+
+// Port=0 omits host:port detail; port property is not set
+{
+conste=util._exceptionWithHostPort(ENOENT,'connect','127.0.0.1',0);
+assert.strictEqual(e.address,'127.0.0.1');
+assert.strictEqual(e.port,undefined);
+assert.ok(e.message.includes('127.0.0.1'),`expected address in "${e.message}"`);
+assert.ok(!e.message.includes(':0'),`unexpected :0 in "${e.message}"`);
+}
+
+// Additional is appended as "- Local (...)"
+{
+conste=util._exceptionWithHostPort(ENOENT,'connect','127.0.0.1',80,'0.0.0.0:12345');
+assert.ok(e.message.includes('- Local (0.0.0.0:12345)'),`expected local info in "${e.message}"`);
+}
+
+// No address or port
+{
+conste=util._exceptionWithHostPort(ENOENT,'connect',null,0);
+assert.strictEqual(e.address,null);
+assert.strictEqual(e.port,undefined);
+assert.ok(!e.message.includes('null'),`unexpected null in "${e.message}"`);
+}
+
+// Different syscall and error code
+{
+conste=util._exceptionWithHostPort(EACCES,'bind','0.0.0.0',443);
+assert.strictEqual(e.code,util.getSystemErrorName(EACCES));
+assert.strictEqual(e.syscall,'bind');
+}
+
+// Stack trace should not include _exceptionWithHostPort itself
+{
+conste=util._exceptionWithHostPort(ENOENT,'connect','127.0.0.1',80);
+assert.ok(!e.stack.includes('_exceptionWithHostPort'),'stack must not mention the wrapper');
+}

test/parallel/test-util-exception-with-host-port.js

@@ -0,0 +1,52 @@
+'use strict';
+
+require('../common');
+constassert=require('node:assert');
+const{ spawnSync }=require('node:child_process');
+const{ getCallSites }=require('node:util');
+constfixtures=require('../common/fixtures');
+
+// mapCallSite falls back to the original call site when no source map is
+// registered for the current file.
+{
+constwithoutMap=getCallSites(1);
+constwithMap=getCallSites(1,{sourceMap: true});
+// scriptName must be identical — no source map registered for this test file
+assert.strictEqual(withMap[0].scriptName,withoutMap[0].scriptName);
+// Both calls report the actual line in this file (they differ by one because
+// they are on consecutive source lines, which is expected).
+assert.ok(typeofwithMap[0].lineNumber==='number');
+assert.ok(typeofwithMap[0].columnNumber==='number');
+}
+
+// reconstructCallSite remaps call sites to the original source when
+// --enable-source-maps is active and a source map is found.
+{
+constfile=fixtures.path('source-map','get-call-sites-mapped.js');
+const{ status, stderr, stdout }=spawnSync(
+process.execPath,
+['--enable-source-maps',file],
+);
+assert.strictEqual(status,0,stderr.toString());
+constcallSite=JSON.parse(stdout.toString());
+// scriptName should point to the original (unmapped) source file
+assert.ok(
+callSite.scriptName.endsWith('get-call-sites-original.js'),
+`expected original source in scriptName, got "${callSite.scriptName}"`,
+);
+// lineNumber and columnNumber should reflect the original source position
+assert.strictEqual(callSite.lineNumber,11);
+assert.strictEqual(callSite.columnNumber,1);
+}
+
+// Without --enable-source-maps the generated file path is preserved.
+{
+constfile=fixtures.path('source-map','get-call-sites-mapped.js');
+const{ status, stderr, stdout }=spawnSync(process.execPath,[file]);
+assert.strictEqual(status,0,stderr.toString());
+constcallSite=JSON.parse(stdout.toString());
+assert.ok(
+callSite.scriptName.endsWith('get-call-sites-mapped.js'),
+`expected generated file in scriptName, got "${callSite.scriptName}"`,
+);
+}