valksor / kvelmo / 23333840437

52%

push

github

Fix flaky TestLimiterConcurrentAccess by freezing time

The test expected at most burst (100) allowed requests from 500
concurrent calls, but the token bucket refills based on elapsed
wall-clock time. With rate=1000 tokens/sec, even microsecond gaps
between goroutine schedules added tokens back, occasionally yielding
101 allowed. Freeze nowFunc to a constant so elapsed time is always
zero, isolating the test to pure concurrency without refill noise.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

799 of 1244 branches covered (64.23%)

Branch coverage included in aggregate %.

20261 of 39258 relevant lines covered (51.61%)

0.9 hits per line