matrix-org / matrix-rust-sdk

Matrix Client-Server SDK for Rust
Apache License 2.0
1.2k stars 240 forks source link

tests: change strategy for `test_ensure_max_concurrency_is_observed` #4018

Closed bnjbvr closed 1 week ago

bnjbvr commented 1 week ago

This test started to fail intermittently, so I rewrote it slightly:

That way, no response should get through the client, and 2 seconds should enough time to spawn all the requests to the mock endpoint, so the test should be more precise.

codecov[bot] commented 1 week ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 84.23%. Comparing base (7d1bbfa) to head (5979059). Report is 2 commits behind head on main.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #4018 +/- ## ======================================= Coverage 84.23% 84.23% ======================================= Files 266 266 Lines 28349 28349 ======================================= + Hits 23879 23881 +2 + Misses 4470 4468 -2 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.