feat: add enough context to plugin methods to figure out bench task name

RafaelGSS · Nov 16, 2024 · c16cf34 · c16cf34
1 parent fd379d6
commit c16cf34
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 5 deletions.
diff --git a/lib/clock.js b/lib/clock.js
@@ -224,7 +224,7 @@ async function clockBenchmark(bench, recommendedCount) {
   result[0] = Math.max(MIN_RESOLUTION, result[0]);
   for (const p of bench.plugins) {
     if (typeof p.onCompleteBenchmark === 'function') {
-      p.onCompleteBenchmark(result);
+      p.onCompleteBenchmark(result, bench);
     }
   }
 

diff --git a/lib/lifecycle.js b/lib/lifecycle.js
@@ -11,13 +11,13 @@ function getItersForOpDuration(durationPerOp, targetTime) {
   return Math.min(Number.MAX_SAFE_INTEGER, Math.max(1, Math.round(totalOpsForMinTime)));
 }
 
-function parsePluginsResult(plugins) {
+function parsePluginsResult(plugins, name) {
   const result = []
   for (const p of plugins) {
     result.push({
       name: p.toString(),
-      result: p.getResult?.() ?? 'enabled',
-      report: p.getReport?.() ?? '',
+      result: p.getResult?.(name) ?? 'enabled',
+      report: p.getReport?.(name) ?? '',
     });
   }
   return result;
@@ -97,7 +97,7 @@ async function runBenchmark(bench, initialIterations) {
     iterations,
     histogram,
     name: bench.name,
-    plugins: parsePluginsResult(bench.plugins),
+    plugins: parsePluginsResult(bench.plugins, bench.name),
   };
 }