Skip to content

Commit 2faef8a

Browse files
committed
fix: update JS max_loops test assertion to match constant (1_000)
PR #1764 changed JS_BENCHMARKING_MAX_LOOPS from 100_000 to 1_000 but the test was updated to assert 5_000 instead of 1_000.
1 parent 59ec38e commit 2faef8a

1 file changed

Lines changed: 2 additions & 2 deletions

File tree

tests/languages/javascript/test_support_dispatch.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -182,9 +182,9 @@ def test_passes_loop_parameters(self, mock_vitest_runner: MagicMock, js_support:
182182

183183
call_kwargs = mock_vitest_runner.call_args.kwargs
184184
assert call_kwargs["min_loops"] == 10
185-
# JS/TS uses JS_BENCHMARKING_MAX_LOOPS (5_000) regardless of passed value
185+
# JS/TS uses JS_BENCHMARKING_MAX_LOOPS (1_000) regardless of passed value
186186
# Actual loop count is limited by target_duration, not max_loops
187-
assert call_kwargs["max_loops"] == 5_000
187+
assert call_kwargs["max_loops"] == 1_000
188188
assert call_kwargs["target_duration_ms"] == 5000
189189

190190

0 commit comments

Comments
 (0)