-
Notifications
You must be signed in to change notification settings - Fork 0
Add integration test timeout and flaky test retry mechanism #29
Copy link
Copy link
Open
Labels
testingTest coverage, benchmarks, and test infrastructureTest coverage, benchmarks, and test infrastructure
Description
Description
Network-based integration tests (e.g., test_network_real.rs, test_tendermint_network.rs) involve async I/O and timers. Without proper timeout enforcement, these tests can hang indefinitely in CI. Additionally, UDP-based tests can be inherently flaky.
Proposed Changes
- Add
#[tokio::test(flavor = "multi_thread", worker_threads = 2)]with explicit timeouts - Consider using
tokio::time::timeoutwrappers in network tests - Add retry logic or
#[flaky_test]attributes for known-flaky network tests - Document which tests require network access
Impact
More reliable CI and clearer test expectations.
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
testingTest coverage, benchmarks, and test infrastructureTest coverage, benchmarks, and test infrastructure