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

Add E2E tests to the ICS misbehaviour handling feature #1065

Closed
Tracked by #732
sainoe opened this issue Jun 22, 2023 · 0 comments · Fixed by #1118
Closed
Tracked by #732

Add E2E tests to the ICS misbehaviour handling feature #1065

sainoe opened this issue Jun 22, 2023 · 0 comments · Fixed by #1118
Assignees
@sainoe
Labels
scope: testing Code review, testing, making sure the code is following the specification.

Comments

@sainoe
Copy link
Contributor

sainoe commented Jun 22, 2023

Problem

The ICS misbehaviour handling feature has to be e2e tested.

Closing criteria

Update the E2E testing framework so that it covers the ICS misbehavior handling feature.

Problem details

Note that these tests will involve freezing the provider's chain client in the case an ICS misbehavior message is successfully handled, which results in killing the consumer chain.
@sainoe sainoe added the scope: testing Code review, testing, making sure the code is following the specification. label Jun 22, 2023
@sainoe sainoe self-assigned this Jun 22, 2023
@github-project-automation github-project-automation bot moved this to 🩹 Triage in Cosmos Hub Jun 22, 2023
@sainoe sainoe moved this from 🩹 Triage to 🏗 In progress in Cosmos Hub Jun 22, 2023
@sainoe sainoe mentioned this issue Jun 22, 2023
Closed
@sainoe sainoe mentioned this issue Jul 6, 2023
Merged
20 tasks
@sainoe sainoe moved this from 🏗 In progress to 👀 In review in Cosmos Hub Jul 6, 2023
@mpoke mpoke linked a pull request Jul 24, 2023 that will close this issue
feat: add e2e tests for ICS misbehaviour #1118
Merged
20 tasks
@sainoe sainoe closed this as completed Jul 31, 2023
@github-project-automation github-project-automation bot moved this from 👀 In review to ✅ Done in Cosmos Hub Jul 31, 2023
@sainoe sainoe mentioned this issue Sep 4, 2023
Merged
20 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@sainoe sainoe

Labels
scope: testing Code review, testing, making sure the code is following the specification.
Projects
Cosmos Hub
Status: ✅ Done
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

feat: add e2e tests for ICS misbehaviour cosmos/interchain-security
1 participant
@sainoe