-
Notifications
You must be signed in to change notification settings - Fork 21
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
feat(test): parallel unit test execution #379
Conversation
✅ Deploy Preview for contracts-stylus canceled.
|
Codecov ReportAttention: Patch coverage is
Additional details and impacted files
|
.github/workflows/check-publish.yml
Outdated
@@ -24,7 +24,7 @@ jobs: | |||
with: | |||
target: wasm32-unknown-unknown | |||
components: rust-src | |||
toolchain: nightly-2024-01-01 | |||
toolchain: nightly-2024-07-25 |
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.
update nightly-2024-01-01
-> nightly-2024-07-25
was added, since cargo stylus cannot compile on the previous version
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.
Left some minor comments regarding naming.
Refactoring for unit tests, that lets them run in parallel.
Every rust test has an associated thread name. Hence corresponding test mock data are stored in a DashMap (concurrent hashmap). Key is name of the thread (full name of the test). Value is mock onchain storage. In the future Value (
MockStorage
) will contain additionally fields formsg::sender
,msg::value
and so on..Resolves #306