-
Notifications
You must be signed in to change notification settings - Fork 3
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
Added MCDC Report feature #243
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #243 +/- ##
==========================================
+ Coverage 94.20% 94.23% +0.02%
==========================================
Files 9 9
Lines 1450 1457 +7
Branches 157 157
==========================================
+ Hits 1366 1373 +7
Misses 84 84
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
E2e Test results - Vcast 2023sp0 - Group coverage2 files 2 suites 1m 26s ⏱️ Results for commit 9659af3. ♻️ This comment has been updated with latest results. |
E2e Test results - Vcast 2021sp0 - Group coverage2 files 2 suites 1m 24s ⏱️ Results for commit 9659af3. ♻️ This comment has been updated with latest results. |
E2e Test results - Vcast 2024sp4 - Group coverage2 files 2 suites 1m 19s ⏱️ Results for commit 9659af3. ♻️ This comment has been updated with latest results. |
E2e Test results - Vcast 2024sp4 - Group coverage_server2 files 2 suites 1m 16s ⏱️ Results for commit 9659af3. ♻️ This comment has been updated with latest results. |
E2e Test results - Vcast 2024sp5 - Group coverage2 files 2 suites 1m 16s ⏱️ Results for commit 9659af3. ♻️ This comment has been updated with latest results. |
E2e Test results - Vcast 2024sp5 - Group coverage_server2 files 2 suites 1m 16s ⏱️ Results for commit 9659af3. ♻️ This comment has been updated with latest results. |
This reverts commit b04fcf8.
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.
Went through all the changes, everything looks good. I haven't seen anything weird, the main comment is about the suggested refactor for clarity regarding all those nested if conditions.
|
Summary
Added an MC/DC report feature that allows the user to generate a report for a specific line. The report opens in a separate WebviewPanel, similar to the test generation report. Lines that are interactable are visually highlighted in the editor.
Usage
Ensure that
Statement+MCDC
is selected underSettings → VectorCAST Test Explorer → Build → Coverage Kind
. If this setting is not selected, the coverage gutter will display the standard coverage icons and will not be interactable.To generate a report,
right-click
on the coverage icon and chooseGet MCDC Report
. The report will open in a separate WebviewPanel.In
Settings → VectorCAST Test Explorer → MCDC Coverage Gutter Icons
, you can choose to display either anarrow
or anM
as the icon in the coverage gutter.TODOs