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

LSM e2e tests #1

Open
wants to merge 39 commits into
base: masa/tests-gaia-lsm-ics
Choose a base branch
from
Open

Conversation

jstr1121
Copy link

@jstr1121 jstr1121 commented Jul 14, 2023

Added LSM happy path e2e test

  • Set parameters (global liquid staking cap, validator liquid staking cap, validator bond factor)
  • Create validator
  • Validator bond
  • Delegates
  • Tokenizes shares
  • Bank send LSM token
  • TransferTokenizeShareRecord (transfer reward ownership)
  • IBC transfer LSM token
  • Redeem tokens for shares

@jstr1121 jstr1121 marked this pull request as draft July 14, 2023 09:51
@jstr1121 jstr1121 marked this pull request as ready for review July 18, 2023 13:15
Copy link

@sampocs sampocs left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good!

Can you provide instructions for how to run this so I can confirm it passes?

We should also have someone on the gaia team review as well since I'm not very familiar with this e2e framework

tests/e2e/e2e_setup_test.go Outdated Show resolved Hide resolved
tests/e2e/e2e_lsm_test.go Show resolved Hide resolved
tests/e2e/e2e_lsm_test.go Outdated Show resolved Hide resolved
tests/e2e/e2e_lsm_test.go Outdated Show resolved Hide resolved
tests/e2e/e2e_lsm_test.go Outdated Show resolved Hide resolved
tests/e2e/e2e_lsm_test.go Outdated Show resolved Hide resolved
tests/e2e/e2e_lsm_test.go Show resolved Hide resolved
tests/e2e/e2e_lsm_test.go Outdated Show resolved Hide resolved
@jstr1121
Copy link
Author

jstr1121 commented Jul 26, 2023

Can you provide instructions for how to run this so I can confirm it passes?

I did run following to test e2e

make docker-build-all
make test-e2e

MSalopek and others added 4 commits July 26, 2023 18:26
* update upgrade checklist

* Update .github/ISSUE_TEMPLATE/upgrade-checklist.md

Co-authored-by: Dante Sanchez <[email protected]>

---------

Co-authored-by: Dante Sanchez <[email protected]>
* bump to v12

* feat: add upgrade handler for v12
@MSalopek
Copy link

MSalopek commented Jul 28, 2023

Thank you for making this!

I checked it out and have no issues with merging it to the branch on gaia. There's small issues like the linter and some sims failing, but I can attempt to fix those on the branch.

Would you mind making a PR to this branch on gai?
A draft PR already exists and is tracking the integration with gaia: cosmos#2643?

Awesome stuff!

yaruwangway and others added 9 commits July 31, 2023 10:32
* fix: upgrade test

* chore: write to log file
* lsm e2e test work - wip

* Add validator bond functionality on lsm happy path

* add tokenize share check on lsm test & redeemShare utility function

* add bank send lsm tokens test

* Add test for transfer reward ownership, and redeem shares

* Add staking param query utility & LSM params update process on LSM e2e test

* Add IBC transfer test on LSM share tokens & Resolve e2e tests by updating gas configs and result checker

* resolve github action bot comments

* bring all e2e test items back to enabled

* Update tests/e2e/e2e_lsm_test.go

Co-authored-by: sampocs <[email protected]>

* resolve comments

* reenable tokenize share record module account balance check

---------

Co-authored-by: jstr1121 <[email protected]>
Co-authored-by: jstr1121 <[email protected]>
Co-authored-by: sampocs <[email protected]>
* set LSM params in upgrade handler

* added comments around constants

* test: add migrateUBDEntries to upgrade handler — required adding exported.go to sdk

* Revert "test: add migrateUBDEntries to upgrade handler — required adding exported.go to sdk"

This reverts commit 7bd9727.

---------

Co-authored-by: Riley Edmunds <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

9 participants