Repository navigation
test(dns): remove flaky wall-clock upper bounds - #207
TMHSDigital merged 1 commit into
Conversation
#207 raised TestCloseWaitsForHandlers' ceiling from 2s to 10s. A parked stream connection that Close fails to close is released by serveTCP's 5s read deadline, so Close then returns in ~5s and passed the 10s ceiling: the test no longer caught the regression it exists for (verified by mutating Close to skip closing tracked connections). 4s still leaves ~20x headroom over the normal ~200ms and fails on that mutation. Refs #206 Co-Authored-By: Claude Opus 5.5 <[email protected]> Claude-Session: https://claude.ai/code/session_01LpkeXSPwrzd7ZCuX3no7gD
|
Thanks for this! It's merged as 30d50f9. The rate-limiter and cancelled-context changes went in as you wrote them; asserting I made one follow-up in 3a30307. The
|
What changed:
Why:
Three tests asserted tight wall-clock upper bounds that flake under load on slow CI runners. Lower bounds and hang-only ceilings remain.
Tests:
Related: #206