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

Integration E2E test for v1 -> v2 smart contract migration #2474

Closed
Tracked by #2745
lumtis opened this issue Jul 12, 2024 · 0 comments · Fixed by #2763
Closed
Tracked by #2745

Integration E2E test for v1 -> v2 smart contract migration #2474

lumtis opened this issue Jul 12, 2024 · 0 comments · Fixed by #2763
Assignees
Labels
E2E E2E tests related

Comments

@lumtis
Copy link
Member

lumtis commented Jul 12, 2024

Describe the Issue

Depends on #2553 and #2554

Add a E2E test for full migration as part of #2402

This is a subset of the upgrade test as it requires a upgrade of the protocol

  • Deploy and setup env with V1 smart contract (will likely require using a fixed version of node)
  • Run some test to populate ERC20Custody and Connector contract
  • Upgrade to new version
  • Run the contract migration on both ZEVM and EVM
  • Run tests with new smart contract version

We will write the tests in the same time as integrating the new contracts, therefore we can keep the PR fixing this issue as a feature branch.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
E2E E2E tests related
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant