node icon indicating copy to clipboard operation
node copied to clipboard

Add an admin local E2E test to test TSS migration

Open lumtis opened this issue 1 year ago • 2 comments

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
### Tasks
- [ ] https://github.com/zeta-chain/node/issues/2370
- [ ] https://github.com/zeta-chain/node/issues/2371
- [ ] https://github.com/zeta-chain/node/issues/2441
- [ ] https://github.com/zeta-chain/node/issues/2442
- [ ] https://github.com/zeta-chain/node/issues/2443
- [ ] https://github.com/zeta-chain/node/issues/2444
- [ ] https://github.com/zeta-chain/node/issues/2455
- [ ] https://github.com/zeta-chain/node/issues/2486
- [ ] https://github.com/zeta-chain/node/issues/2487
- [ ] https://github.com/zeta-chain/node/issues/2492

lumtis avatar Apr 01 '24 09:04 lumtis

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

kingpinXD avatar Apr 26 '24 05:04 kingpinXD

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 avatar May 02 '24 12:05 lumtis