-
Notifications
You must be signed in to change notification settings - Fork 257
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
Move test crates into a "testing" folder and add a ui (trybuild) test and ui-test helpers #567
Conversation
This will fail until #565 is merged in (as one of the tests is for that failure), but is good to go otherwise. |
testing/ui-tests/src/utils/mod.rs
Outdated
pub use metadata_test_runner::MetadataTestRunner; | ||
|
||
/// Given some pallet metadata, generate a [`RuntimeMetadataPrefixed`] struct. | ||
/// We default to a duff extrinsic type, and register a fake `DispatchError` type |
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.
What is "duff"?
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.
Whoops; british slang maybe?! (Every chance I just made it up/misused it too). I just mean that it's no good/made up/naff! I'll tweak the wording!
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.
:) That's awesome to know. TIL: Duff.
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.
👍
testing/ui-tests/Cargo.toml
Outdated
@@ -0,0 +1,15 @@ | |||
[package] | |||
name = "ui-tests" | |||
version = "0.1.0" |
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.
Nit: should we keep in sync with ?
name = "subxt-integration"
version = "0.21.0"
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.
Yup, good catch!
testing
folder to avoid polluting the root too much with test bits.ui-tests
crate with utils to make it easier to generate metadata and usetrybuild
to check that it compiles properly (or not).