Skip to content

refactor: update e2e tests#984

Merged
tsenovilla merged 2 commits intomainfrom
refactor/e2e-test-update
Feb 13, 2025
Merged

refactor: update e2e tests#984
tsenovilla merged 2 commits intomainfrom
refactor/e2e-test-update

Conversation

@tsenovilla
Copy link
Copy Markdown
Contributor

This PR introduces some minor updates in our e2e tests:

  • All moonbeam reserve transfer stuff is removed: The process we'll follow to send LAOS to moonbeam wouldn't imply pallet assets but an actual ERC-20 contract, so our e2e test is senseless. We might add a new one in the future reflecting the actual process, but we shouldn't give it priority right now.

  • Hydration chopsticks config is removed: As the integration with Hydra is completed, we don't need to overwrite their storage anymore, just use the standard config provided by the Chopsticks repo

  • Update the asset hub chopsticks config as well, we don't need to override their storage to register the LAOS asset anymore. Keep the chopsticks config file to give some balance to the accounts used in the teleport test.

asiniscalchi
asiniscalchi previously approved these changes Feb 13, 2025
magecnion
magecnion previously approved these changes Feb 13, 2025
Copy link
Copy Markdown
Contributor

@luispdm luispdm left a comment

Choose a reason for hiding this comment

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

Please. replace this:

            -p chopsticks-configs/moonbeam.yml \
            -p chopsticks-configs/hydration.yml &

for this:

            -p hydradx &

in the GitHub actions file.

@tsenovilla tsenovilla dismissed stale reviews from magecnion and asiniscalchi via ba20ceb February 13, 2025 10:51
@tsenovilla tsenovilla merged commit f02eabd into main Feb 13, 2025
@tsenovilla tsenovilla deleted the refactor/e2e-test-update branch February 13, 2025 12:25
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.

4 participants