-
Notifications
You must be signed in to change notification settings - Fork 32
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
Update report aligning to the mockup #717
Conversation
Codecov Report
@@ Coverage Diff @@
## master #717 +/- ##
==========================================
- Coverage 82.15% 82.08% -0.08%
==========================================
Files 29 29
Lines 4147 4181 +34
==========================================
+ Hits 3407 3432 +25
- Misses 740 749 +9
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
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.
Only one minor comment about the info
output, but looks good to be merged.
Issue #708 - Updates the category table to report to match the agreed report mockup. Example of new table below. Also adds a new warning when a test is gone to alert to possible misconfiguration
Some items which cropped up and may need addressing in the future is the
'required':'pass'
Some tests report as GONE. Please check for possible misconfiguration
Syntax: Pass / Fail / Skip