-
Notifications
You must be signed in to change notification settings - Fork 68
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
ci: code coverage #293
ci: code coverage #293
Conversation
Codecov Report
@@ Coverage Diff @@
## master #293 +/- ##
=========================================
Coverage ? 14.82%
=========================================
Files ? 24
Lines ? 6970
Branches ? 0
=========================================
Hits ? 1033
Misses ? 5937
Partials ? 0 Continue to review full report at Codecov.
|
- name: Tests | ||
run: time cargo test --all --release --verbose --locked | ||
- name: Benchmarking tests | ||
run: cd node && cargo test --release --features runtime-benchmarks -p pallet-exchange-benchmarking -p pallet-xyk -p pallet-multi-payment-benchmarking -p pallet-claims |
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.
benchamrking tests are removed intentionally?
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.
yes, removed. Benchmarks tests are part of the other step and included in the coverage too.
Co-authored-by: lumir-mrkva <[email protected]>
Run tests with code coverage using tarpaulin. Upload to codecov.io