-
Notifications
You must be signed in to change notification settings - Fork 86
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
Conversation
5f87e3c
to
da1057f
Compare
There was a problem hiding this 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 💯
ce47504
to
b86f077
Compare
658ad6f
to
46ad25c
Compare
There was a problem hiding this 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); |
There was a problem hiding this comment.
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?
There was a problem hiding this comment.
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.
94aa815
to
3ce7027
Compare
Description
Follow-up for the
.43
update work where only some LP integration tests were added to the generic framework.Checklist: