test_runner: don't await the same promise for each test

Prior to this commit, each top level test awaited the same global promise for setting up test reporters. This commit updates the logic to only await the promise the first time it is encountered. PR-URL: nodejs#52185 Refs: nodejs#47164 Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Chemi Atlow <[email protected]>
anonrig · Mar 25, 2024 · a7621d9 · a7621d9
1 parent 1e458e0
commit a7621d9
Showing 1 changed file with 5 additions and 1 deletion.
diff --git a/lib/internal/test_runner/harness.js b/lib/internal/test_runner/harness.js
@@ -225,7 +225,11 @@ function getGlobalRoot() {
 }
 
 async function startSubtest(subtest) {
-  await reportersSetup;
+  if (reportersSetup) {
+    // Only incur the overhead of awaiting the Promise once.
+    await reportersSetup;
+    reportersSetup = undefined;
+  }
 
   const root = getGlobalRoot();
   if (!root.harness.bootstrapComplete) {