Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

tests(app): ⛓ use mock consensus for IBC tests #4071

Open
cratelyn opened this issue Mar 21, 2024 · 0 comments
Open

tests(app): ⛓ use mock consensus for IBC tests #4071

cratelyn opened this issue Mar 21, 2024 · 0 comments
Labels
A-mock-consensus Area: Relates to the mock consensus engine A-testing Area: Relates to testing of Penumbra _P-medium Medium priority

Comments

@cratelyn
Copy link
Contributor

https://github.com/penumbra-zone/hermes/blob/64ef92934fa3ea399321a61ef1386b0da2aed9db/crates/relayer/src/chain/handle.rs#L376

@zbuc and i spent some time pairing on how he might use the penumbra-mock-consensus library to test the ibc relayer. in order to work with two mock penumbra chains, we may want to implement the ChainHandle trait for TestNode<C>.
@cratelyn cratelyn added A-testing Area: Relates to testing of Penumbra A-mock-consensus Area: Relates to the mock consensus engine labels Mar 21, 2024
@github-project-automation github-project-automation bot moved this to 🗄️ Backlog in Penumbra Mar 21, 2024
@cratelyn cratelyn mentioned this issue Mar 21, 2024
Closed
@aubrika aubrika added the _P-medium Medium priority label May 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
A-mock-consensus Area: Relates to the mock consensus engine A-testing Area: Relates to testing of Penumbra _P-medium Medium priority
Projects
Penumbra
Status: Backlog
Milestone
No milestone
Development

No branches or pull requests
2 participants
@cratelyn @aubrika