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-tests: Use the generic framework for development LP tests #1643

Merged
merged 5 commits into from
Dec 14, 2023

Conversation

cdamian
Copy link
Contributor

@cdamian cdamian commented Dec 8, 2023

Description

Follow-up for the .43 update work where only some LP integration tests were added to the generic framework.

Checklist:

  • I have added Rust doc comments to structs, enums, traits and functions
  • I have made corresponding changes to the documentation
  • I have performed a self-review of my code
  • I have added tests that prove my fix is effective or that my feature works

@cdamian cdamian force-pushed the development-lp-test-refactor branch from 5f87e3c to da1057f Compare December 8, 2023 19:24
Copy link
Contributor

@lemunozm lemunozm left a comment

Choose a reason for hiding this comment

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

Thanks so much for all this work porting this 💯

@cdamian cdamian marked this pull request as ready for review December 11, 2023 10:49
@cdamian cdamian force-pushed the development-lp-test-refactor branch from ce47504 to b86f077 Compare December 11, 2023 10:50
@cdamian cdamian marked this pull request as draft December 11, 2023 11:20
@cdamian cdamian force-pushed the development-lp-test-refactor branch 2 times, most recently from 658ad6f to 46ad25c Compare December 12, 2023 15:07
@cdamian cdamian marked this pull request as ready for review December 12, 2023 15:40
Copy link
Contributor

@lemunozm lemunozm left a comment

Choose a reason for hiding this comment

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

Amazing PR and work @cdamian! 🚀

Really love how things are now organized in integration-tests, thanks a lot for all your effort on this! 🥇

crate::test_for_runtimes!([development], allow_pool_should_fail);
crate::test_for_runtimes!([development], schedule_upgrade);
crate::test_for_runtimes!([development], cancel_upgrade);
crate::test_for_runtimes!([development], update_tranche_token_metadata);
Copy link
Contributor

Choose a reason for hiding this comment

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

Out of curiosity, what's the current problem with the rest of the runtimes?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I think it's mostly related to the native currency that we use there, and probably some other things that @NunoAlexandre might know better. However, I also think that later on we can probably improve these tests a bit more and try to run them for all runtimes, I'm sure there's a way to achieve that.

@cdamian cdamian force-pushed the development-lp-test-refactor branch from 94aa815 to 3ce7027 Compare December 13, 2023 10:44
@cdamian cdamian merged commit 4b5796e into main Dec 14, 2023
9 checks passed
@cdamian cdamian deleted the development-lp-test-refactor branch December 14, 2023 12:01
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.

2 participants