gossipsub: add test for missed heartbeats
Nobody has claimed this yet.
- Dominant language
- Rust
- Stars
- 5.6k
- Forks
- 1.3k
- Avg merge
- 8h 47m
- Merged PRs (30d)
- 19
Description
We should have a test that ensure we "catch" up to missed heartbeats. For the underlying timer, this means implementing a "skip" behavior.
The test should be on a Swarm level ideally such that we don't rely on implementation details.
The cherry on the cake would be to run the test in wasm as well to make sure the timer implementation is always sane.
Originally discussed in https://github.com/libp2p/rust-libp2p/pull/3973#issuecomment-1562291148
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start with the gossipsub Swarm-level test described in the issue and inspect the underlying timer's skip behavior. Verify that missed heartbeats are caught up, then run the test in wasm as well if the existing test setup supports it.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- rust
- Domain
- networking, testing-qa
- Issue type
- Feature
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 42/100