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 an admin local E2E test to test TSS migration #1965

Closed
9 tasks done
Tracked by #1720
lumtis opened this issue Apr 1, 2024 · 2 comments
Closed
9 tasks done
Tracked by #1720

Add an admin local E2E test to test TSS migration #1965

lumtis opened this issue Apr 1, 2024 · 2 comments
Assignees
Labels
E2E E2E tests related epic Group issues under a high-level objective

Comments

@lumtis
Copy link
Member

lumtis commented Apr 1, 2024

Describe the Issue
Add a local E2E test that can do the following:

  • Start the local environment and send cctxs normally
  • Stop cctxs
  • Perform a TSS migration. There are several options we can consider implementing:
    • New observer added
    • Observer removed
    • Observer updated
  • Check the TSS can migrate funds
  • New cctxs can be created
@lumtis lumtis added the E2E E2E tests related label Apr 1, 2024
@kingpinXD kingpinXD self-assigned this Apr 26, 2024
@kingpinXD
Copy link
Contributor

Add, Remove observer is a separate functionality from TSS migration and should be tested separately .
It would make sense to create a e2e test first which does the following

  • Sets a new keygen
  • Restarts zeta-clients so that they pick up the new keygen and generate TSS
  • Migrate TSS funds
  • Update TSS address
  • Test cross-chain transactions

@lumtis lumtis added this to the v17 milestone May 2, 2024
@lumtis
Copy link
Member Author

lumtis commented May 2, 2024

Add, Remove observer is a separate functionality from TSS migration and should be tested separately .
It would make sense to create a e2e test first which does the following

  • Sets a new keygen
  • Restarts zeta-clients so that they pick up the new keygen and generate TSS
  • Migrate TSS funds
  • Update TSS address
  • Test cross-chain transactions

This can be a first step but why not putting everything under the same test in the end?

@lumtis lumtis added the epic Group issues under a high-level objective label Jul 18, 2024
@lumtis lumtis closed this as completed Aug 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
E2E E2E tests related epic Group issues under a high-level objective
Projects
None yet
Development

No branches or pull requests

2 participants