-
Notifications
You must be signed in to change notification settings - Fork 27
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
test: increase coverage build function #194
Conversation
Codecov ReportAttention: Patch coverage is
@@ Coverage Diff @@
## main #194 +/- ##
==========================================
+ Coverage 55.97% 57.81% +1.83%
==========================================
Files 33 36 +3
Lines 3398 3584 +186
Branches 3398 3584 +186
==========================================
+ Hits 1902 2072 +170
+ Misses 1213 1198 -15
- Partials 283 314 +31
|
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 idea is to perhaps place the parachain tests within this PR together, and the contracts tests. Now there is a split in the tests which can then perhaps be prevented. Not completely sure though
Unit tests for the build parachain and build contracts processes.
Utilize mocks for the external crates that are used:
duct::cmd
for building parachain andcontract_build::execute
for building contracts.