Skip to content

fix(test): Use async sleep in stresstest#509

Open
jan-auer wants to merge 1 commit into
mainfrom
fix/stresstest-async-sleep
Open

fix(test): Use async sleep in stresstest#509
jan-auer wants to merge 1 commit into
mainfrom
fix/stresstest-async-sleep

Conversation

@jan-auer

@jan-auer jan-auer commented Jun 22, 2026

Copy link
Copy Markdown
Member

std::thread::sleep blocks the tokio runtime thread. Use tokio::time::sleep to yield back to the executor while waiting for the server to start.

Since this is in tests, this is mostly a cosmetic change. I'm changing this for best practices.

@jan-auer jan-auer requested a review from a team as a code owner June 22, 2026 15:58
@codecov

codecov Bot commented Jun 22, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 87.47%. Comparing base (4b0fe8f) to head (2570dbc).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #509   +/-   ##
=======================================
  Coverage   87.47%   87.47%           
=======================================
  Files          86       86           
  Lines       13912    13912           
=======================================
  Hits        12169    12169           
  Misses       1743     1743           
Components Coverage Δ
Rust Backend 92.26% <ø> (ø)
Rust Client 80.06% <ø> (ø)
Python Client 88.51% <ø> (ø)

☔ 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.

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