-
Notifications
You must be signed in to change notification settings - Fork 40
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
Retain the number of total tests after mapping the tests to classes #131
Conversation
Partial score: 0/0🚧 The grading has been aborted due to an error. Exception
Error Messages
Information Messages
Created by Quality Monitor v1.11.0 (#99404b7). More details are shown in the GitHub Checks Result. |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #131 +/- ##
============================================
+ Coverage 93.08% 93.19% +0.10%
Complexity 840 840
============================================
Files 28 28
Lines 2301 2306 +5
Branches 340 339 -1
============================================
+ Hits 2142 2149 +7
+ Misses 86 85 -1
+ Partials 73 72 -1 ☔ View full report in Codecov by Sentry. |
☀️ Quality Monitor 🚦 Tests: 100 % successful (✔️ 293 passed) Created by Quality Monitor v1.12.0-SNAPSHOT (#e2728ef). More details are shown in the GitHub Checks Result. |
☀️ Quality Monitor 🚦 Tests: 100 % successful (✔️ 293 passed) Created by Quality Monitor v1.12.0-SNAPSHOT (#e2728ef). More details are shown in the GitHub Checks Result. |
db13a7d
to
1abafd2
Compare
☀️ Quality Monitor 🚦 Tests: 100 % successful (✔️ 293 passed) Created by Quality Monitor v1.12.0-SNAPSHOT (#e2728ef). More details are shown in the GitHub Checks Result. |
When the test cases are mapped to the coverage tree then we should not ignore those tests that could not be automatically be mapped. It makes more sense to attach the correct metric value to the tree, even though the tests within the tree are smaller.