Skip to content

test(server): Fix flaky test_bandwidth_burst_tolerance - #578

Merged
jan-auer merged 1 commit into
mainfrom
fix/flake-bandwidth-burst
Jul 27, 2026
Merged

test(server): Fix flaky test_bandwidth_burst_tolerance#578
jan-auer merged 1 commit into
mainfrom
fix/flake-bandwidth-burst

Conversation

@jan-auer

Copy link
Copy Markdown
Member

The test left only 500ms of margin between the accumulated bandwidth debt and the burst tolerance, so a slow runner would see the debt drain back under the burst and admit the request that should have been rejected. Scaled the rate and burst down so the margin is seconds instead of milliseconds, keeping the same scenario.

Closes FS-459

@jan-auer
jan-auer requested a review from a team as a code owner July 27, 2026 11:59
@linear-code

linear-code Bot commented Jul 27, 2026

Copy link
Copy Markdown

FS-459

@codecov

codecov Bot commented Jul 27, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 87.71%. Comparing base (181df0a) to head (705f812).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #578      +/-   ##
==========================================
+ Coverage   87.64%   87.71%   +0.06%     
==========================================
  Files          94       94              
  Lines       15425    15505      +80     
==========================================
+ Hits        13520    13600      +80     
  Misses       1905     1905              
Components Coverage Δ
Rust Backend 92.41% <ø> (+0.05%) ⬆️
Rust Client 79.89% <ø> (ø)
Python Client 90.90% <ø> (ø)

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@jan-auer
jan-auer merged commit 92d9741 into main Jul 27, 2026
26 checks passed
@jan-auer
jan-auer deleted the fix/flake-bandwidth-burst branch July 27, 2026 12:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants