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.

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

Integrate CometMock for equivocation #1190

Closed
p-offtermatt opened this issue Aug 9, 2023 · 0 comments · Fixed by #1192
Closed

Integrate CometMock for equivocation #1190

p-offtermatt opened this issue Aug 9, 2023 · 0 comments · Fixed by #1192
Assignees
Labels
scope: testing Code review, testing, making sure the code is following the specification.

Comments

@p-offtermatt
Copy link
Contributor

Problem

Equivocation is hard to cause in the current tests, see #1180.
This issue is concerned with integrating CometMock in the e2e framework to easily convince the app that a double sign infraction has occurred.

Closing criteria

CometMock can be used in the end-to-end tests to cause double-sign infractions.

Related issues

#1180
informalsystems/CometMock#5

@p-offtermatt p-offtermatt added the scope: testing Code review, testing, making sure the code is following the specification. label Aug 9, 2023
@p-offtermatt p-offtermatt self-assigned this Aug 9, 2023
@github-project-automation github-project-automation bot moved this to 🩹 Triage in Cosmos Hub Aug 9, 2023
@p-offtermatt p-offtermatt moved this from 🩹 Triage to 🏗 In progress in Cosmos Hub Aug 9, 2023
@p-offtermatt p-offtermatt linked a pull request Aug 9, 2023 that will close this issue
8 tasks
@p-offtermatt p-offtermatt moved this from 🏗 In progress to 👀 In review in Cosmos Hub Aug 9, 2023
@github-project-automation github-project-automation bot moved this from 👀 In review to ✅ Done in Cosmos Hub Aug 10, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
scope: testing Code review, testing, making sure the code is following the specification.
Projects
Status: ✅ Done
Development

Successfully merging a pull request may close this issue.

1 participant