From 0606c1140389d8e036087bb24e03bc43d727ffd0 Mon Sep 17 00:00:00 2001 From: Nayeem Rahman Date: Tue, 10 Oct 2023 14:40:36 +0100 Subject: fix(bench): use total time when measuring wavg (#20862) --- cli/js/40_testing.js | 10 ++-------- 1 file changed, 2 insertions(+), 8 deletions(-) (limited to 'cli/js/40_testing.js') diff --git a/cli/js/40_testing.js b/cli/js/40_testing.js index 7b60dd133..f4eb0a5a7 100644 --- a/cli/js/40_testing.js +++ b/cli/js/40_testing.js @@ -982,20 +982,17 @@ async function benchMeasure(timeBudget, fn, async, context) { fn(context); const t2 = benchNow(); const totalTime = t2 - t1; - let measuredTime = totalTime; if (currentBenchUserExplicitStart !== null) { - measuredTime -= currentBenchUserExplicitStart - t1; currentBenchUserExplicitStart = null; usedExplicitTimers = true; } if (currentBenchUserExplicitEnd !== null) { - measuredTime -= t2 - currentBenchUserExplicitEnd; currentBenchUserExplicitEnd = null; usedExplicitTimers = true; } c++; - wavg += measuredTime; + wavg += totalTime; budget -= totalTime; } } else { @@ -1004,20 +1001,17 @@ async function benchMeasure(timeBudget, fn, async, context) { await fn(context); const t2 = benchNow(); const totalTime = t2 - t1; - let measuredTime = totalTime; if (currentBenchUserExplicitStart !== null) { - measuredTime -= currentBenchUserExplicitStart - t1; currentBenchUserExplicitStart = null; usedExplicitTimers = true; } if (currentBenchUserExplicitEnd !== null) { - measuredTime -= t2 - currentBenchUserExplicitEnd; currentBenchUserExplicitEnd = null; usedExplicitTimers = true; } c++; - wavg += measuredTime; + wavg += totalTime; budget -= totalTime; } } -- cgit v1.2.3