From e4ec337d42c24247666b12973d07a176ce7d50b3 Mon Sep 17 00:00:00 2001
From: RedYetiDev <38299977+RedYetiDev@users.noreply.github.com>
Date: Sat, 14 Sep 2024 17:27:04 -0400
Subject: [PATCH] test: remove `getCallSite` from common

---
 test/common/README.md |  7 -------
 test/common/index.js  | 19 +++----------------
 test/common/index.mjs |  2 --
 3 files changed, 3 insertions(+), 25 deletions(-)

diff --git a/test/common/README.md b/test/common/README.md
index 72e34fd4302b15..2c1498a9252245 100644
--- a/test/common/README.md
+++ b/test/common/README.md
@@ -187,13 +187,6 @@ Returns an instance of all possible `ArrayBufferView`s of the provided Buffer.
 Returns an instance of all possible `BufferSource`s of the provided Buffer,
 consisting of all `ArrayBufferView` and an `ArrayBuffer`.
 
-### `getCallSite(func)`
-
-* `func` [\<Function>][<Function>]
-* return [\<string>][<string>]
-
-Returns the file name and line number for the provided Function.
-
 ### `getTTYfd()`
 
 Attempts to get a valid TTY file descriptor. Returns `-1` if it fails.
diff --git a/test/common/index.js b/test/common/index.js
index 172cdb6b049824..dd5f43c8664eb8 100644
--- a/test/common/index.js
+++ b/test/common/index.js
@@ -30,7 +30,7 @@ const net = require('net');
 // Do not require 'os' until needed so that test-os-checked-function can
 // monkey patch it. If 'os' is required here, that test will fail.
 const path = require('path');
-const { inspect } = require('util');
+const { inspect, getCallSite } = require('util');
 const { isMainThread } = require('worker_threads');
 const { isModuleNamespaceObject } = require('util/types');
 
@@ -551,25 +551,13 @@ function canCreateSymLink() {
   return true;
 }
 
-function getCallSite(top) {
-  const originalStackFormatter = Error.prepareStackTrace;
-  Error.prepareStackTrace = (err, stack) =>
-    `${stack[0].getFileName()}:${stack[0].getLineNumber()}`;
-  const err = new Error();
-  Error.captureStackTrace(err, top);
-  // With the V8 Error API, the stack is not formatted until it is accessed
-  err.stack; // eslint-disable-line no-unused-expressions
-  Error.prepareStackTrace = originalStackFormatter;
-  return err.stack;
-}
-
 function mustNotCall(msg) {
-  const callSite = getCallSite(mustNotCall);
+  const callSite = getCallSite()[1];
   return function mustNotCall(...args) {
     const argsInfo = args.length > 0 ?
       `\ncalled with arguments: ${args.map((arg) => inspect(arg)).join(', ')}` : '';
     assert.fail(
-      `${msg || 'function should not have been called'} at ${callSite}` +
+      `${msg || 'function should not have been called'} at ${callSite.scriptName}:${callSite.lineNumber}` +
       argsInfo);
   };
 }
@@ -968,7 +956,6 @@ const common = {
   expectWarning,
   getArrayBufferViews,
   getBufferSources,
-  getCallSite,
   getPrintedStackTrace,
   getTTYfd,
   hasIntl,
diff --git a/test/common/index.mjs b/test/common/index.mjs
index 007ce233fbea0d..42c5d5750965a1 100644
--- a/test/common/index.mjs
+++ b/test/common/index.mjs
@@ -15,7 +15,6 @@ const {
   expectWarning,
   getArrayBufferViews,
   getBufferSources,
-  getCallSite,
   getTTYfd,
   hasCrypto,
   hasIntl,
@@ -69,7 +68,6 @@ export {
   expectWarning,
   getArrayBufferViews,
   getBufferSources,
-  getCallSite,
   getPort,
   getTTYfd,
   hasCrypto,