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

Add v7 to v8 e2e upgrade test #4216

Closed
1 of 2 tasks
damiannolan opened this issue Aug 1, 2023 · 0 comments · Fixed by #4649 or #4660
Closed
1 of 2 tasks

Add v7 to v8 e2e upgrade test #4216

damiannolan opened this issue Aug 1, 2023 · 0 comments · Fixed by #4649 or #4660
Assignees
Labels
e2e testing Testing package and unit/integration tests
Milestone

Comments

@damiannolan
Copy link
Member

damiannolan commented Aug 1, 2023

We should add an upgrade handler and e2e chain upgrade test for v7 -> v8 which performs the automatic migrations and does some parameter lookups and assertions to make sure everything is set correctly. The test should:

These two could be done in two separate PRs, but the second check depends on having the overall test set up in the first one.

@damiannolan damiannolan added testing Testing package and unit/integration tests e2e labels Aug 1, 2023
@damiannolan damiannolan moved this to Todo in ibc-go Aug 1, 2023
@damiannolan damiannolan added this to the v8.0.0 milestone Aug 1, 2023
@crodriguezvega crodriguezvega changed the title Add v8 e2e upgrade test for params migrations Add v7 to v8 e2e upgrade test Aug 18, 2023
@chatton chatton self-assigned this Sep 5, 2023
@chatton chatton moved this from Todo to In progress in ibc-go Sep 11, 2023
@crodriguezvega crodriguezvega moved this from In progress to In review in ibc-go Sep 16, 2023
@github-project-automation github-project-automation bot moved this from In review to Done in ibc-go Sep 18, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
e2e testing Testing package and unit/integration tests
Projects
Archived in project
2 participants