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 v8 to v9 upgrade tests #6642

Closed
2 tasks done
crodriguezvega opened this issue Jun 19, 2024 · 0 comments · Fixed by #6806
Closed
2 tasks done

Add v8 to v9 upgrade tests #6642

crodriguezvega opened this issue Jun 19, 2024 · 0 comments · Fixed by #6806
Assignees
@gjermundgaraba
Labels
e2e type: test coverage Increase code and use case coverage by means of unit, integration, e2e or manual tests
Milestone
v9.0.0

Comments

@crodriguezvega
Copy link
Contributor

crodriguezvega commented Jun 19, 2024 •
edited by gjermundgaraba
Loading

We should add an e2e chain upgrade test for v8 to v9 to check that the automatic migrations execute fine and make sure everything is set correctly. This could be done in two different tests (and PRs):

  • Test a chain upgrade from v8 to v9 and then check that migration MigrateDenomTraceToDenom runs and works as expected.
  • Test a chain upgrade from v8 to v9 and then run a channel upgrade from ics20-1 to ics20-2 and do a transfer for multi-denom (do people think we should also do a forwarding transfer?)
@crodriguezvega crodriguezvega added e2e type: test coverage Increase code and use case coverage by means of unit, integration, e2e or manual tests labels Jun 19, 2024
@crodriguezvega crodriguezvega added this to the v9.0.0 milestone Jun 19, 2024
@gjermundgaraba gjermundgaraba self-assigned this Jul 8, 2024
This was referenced Jul 9, 2024
Merged
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@gjermundgaraba gjermundgaraba

Labels
e2e type: test coverage Increase code and use case coverage by means of unit, integration, e2e or manual tests
Projects
ibc-go
Archived in project
Milestone
v9.0.0
Development

Successfully merging a pull request may close this issue.

test: add v8 to v9 e2e upgrade tests with channel upgrade cosmos/ibc-go
2 participants
@gjermundgaraba @crodriguezvega