informalsystems / informalsystems/hermes-sdk
Add integration test for ack packets getting relayed properly
- Dominant language
- Rust
- Stars
- 52
- Forks
- 11
- PR merge metrics
- No merged PRs in 30d
Description
## Summary
At the moment there are some integration tests for the relayer, but some cases are not covered.
Following issue https://github.com/informalsystems/hermes/issues/2920 the addition of a test checking that the acknowledgment messages, and the full relaying of messages, is required.
## Proposal
Addition of tests covering the relaying of a message and checking that everything is correctly relayed to the source and destination chain.
## Acceptance Criteria
Improve current integration tests for relayer or adding new tests which cover the missing scenario.
____
#### For Admin Use
- [ ] Not duplicate issue
- [ ] Appropriate labels applied
- [ ] Appropriate milestone (priority) applied
- [ ] Appropriate contributors tagged
- [ ] Contributor assigned/self-assigned
Contributor guide
Research direction
Start by reading issue #2920 and locating the existing relayer integration tests in the repository. Trace how acknowledgment packets and complete message relaying are currently exercised, then add coverage for the missing source-to-destination scenario. Done means the tests verify acknowledgments and full relaying on both chains.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- rust
- Domain
- distributed-systems, testing-qa
- Issue type
- Feature
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100