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
Add coverage script in the admin #14109
Add coverage script in the admin #14109
Conversation
Codecov Report
@@ Coverage Diff @@
## main #14109 +/- ##
==========================================
+ Coverage 49.38% 55.58% +6.19%
==========================================
Files 263 1275 +1012
Lines 9290 31829 +22539
Branches 2020 5733 +3713
==========================================
+ Hits 4588 17692 +13104
- Misses 3878 12323 +8445
- Partials 824 1814 +990
Flags with carried forward coverage won't be shown. Click here to find out more.
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. |
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'm fine with that and think it is a good addition, but would also like to hear e.g. @alexandrebodin s opinion.
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.
@simotae14 Could you also update the test:front:cov
script in the helper-plugin please, so that they are consistent?
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.
Great addition! Thank you.
What does it do?
I have added a script to collect the coverage of the front end unit tests inside the admin
Why is it needed?
To know the percentage of the code covered with the tests
How to test it?
Just run
yarn test:front:ce:cov
from the packages/core/admin folder