test: removed mustCallAsync from common and added inside testcase

PR-URL: #23467
Reviewed-By: Gireesh Punathil <[email protected]>
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Trivikram Kamat <[email protected]>

Author: quinnlangille

test/common/README.md

@@ -234,17 +234,6 @@ fail.
 
 If `fn` is not provided, an empty function will be used.
 
-### mustCallAsync([fn][, exact])
-* `fn` [<Function>]
-* `exact` [<number>] default = 1
-* return [<Function>]
-
-The same as `mustCall()`, except that it is also checked that the Promise
-returned by the function is fulfilled for each invocation of the function.
-
-The return value of the wrapped function is the return value of the original
-function, if necessary wrapped as a promise.
-
 ### mustCallAtLeast([fn][, minimum])
 * `fn` [<Function>] default = () => {}
 * `minimum` [<number>] default = 1

test/common/index.js

@@ -308,12 +308,6 @@ function mustCallAtLeast(fn, minimum) {
   return _mustCallInner(fn, minimum, 'minimum');
 }
 
-function mustCallAsync(fn, exact) {
-  return mustCall((...args) => {
-    return Promise.resolve(fn(...args)).then(mustCall((val) => val));
-  }, exact);
-}
-
 function _mustCallInner(fn, criteria = 1, field) {
   if (process._exiting)
     throw new Error('Cannot use common.mustCall*() in process exit handler');
@@ -721,7 +715,6 @@ module.exports = {
   isWindows,
   localIPv6Hosts,
   mustCall,
-  mustCallAsync,
   mustCallAtLeast,
   mustNotCall,
   nodeProcessAborted,

test/common/index.mjs

@@ -26,7 +26,6 @@ const {
   allowGlobals,
   mustCall,
   mustCallAtLeast,
-  mustCallAsync,
   hasMultiLocalhost,
   skipIfEslintMissing,
   canCreateSymLink,
@@ -74,7 +73,6 @@ export {
   allowGlobals,
   mustCall,
   mustCallAtLeast,
-  mustCallAsync,
   hasMultiLocalhost,
   skipIfEslintMissing,
   canCreateSymLink,

test/parallel/test-http2-backpressure.js

@@ -12,7 +12,7 @@ const makeDuplexPair = require('../common/duplexpair');
 {
   let req;
   const server = http2.createServer();
-  server.on('stream', common.mustCallAsync(async (stream, headers) => {
+  server.on('stream', mustCallAsync(async (stream, headers) => {
     stream.respond({
       'content-type': 'text/html',
       ':status': 200
@@ -45,3 +45,9 @@ function event(ee, eventName) {
     ee.once(eventName, common.mustCall(resolve));
   });
 }
+
+function mustCallAsync(fn, exact) {
+  return common.mustCall((...args) => {
+    return Promise.resolve(fn(...args)).then(common.mustCall((val) => val));
+  }, exact);
+}