-
-
Notifications
You must be signed in to change notification settings - Fork 29
Added support for TRACE32 parser and metrics #662
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
base: main
Are you sure you want to change the base?
Changes from 11 commits
87568f7
56da7ea
31a6e3d
e884a24
74250a3
030877f
310c2b9
4e6c437
3683d03
8bbe3d8
ec6be94
c605b7a
1d25de4
18cdf26
5c501b5
fc75231
23be799
63914be
f0f3f5b
3681137
12cb560
b9b39e9
19e5c32
23c0dc9
7932062
3effe53
e6144d6
009f35e
3f96229
b83b6af
527bd72
5251410
f77f581
877978d
67214f4
c890d4c
72430ab
b09db42
f883b42
e71dace
f9caf6b
69d7d1b
30fde01
b9f83a1
327cbbf
cc45760
703a749
56bf522
18aff18
ee1d93d
8e8fff4
5003345
6007ac9
8f46b93
d80aa72
d0f86de
4e2b5c2
f443b65
f88193d
4cf8e23
23e338b
2be5b18
5aa0b91
a1d90a3
0668958
7cfdd43
7b206dd
0527599
c19f6bc
db280eb
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -81,8 +81,9 @@ public class CoverageViewModel extends DefaultAsyncTableContentProvider implemen | |
|
|
||
| private static final ElementFormatter FORMATTER = new ElementFormatter(); | ||
| private static final Set<Metric> TREE_METRICS = Set.of( | ||
| Metric.LINE, Metric.BRANCH, Metric.MUTATION, Metric.TEST_STRENGTH, Metric.CYCLOMATIC_COMPLEXITY, Metric.TESTS, | ||
| Metric.MCDC_PAIR, Metric.FUNCTION_CALL, Metric.COGNITIVE_COMPLEXITY, Metric.NCSS, Metric.NPATH_COMPLEXITY); | ||
| Metric.LINE, Metric.BRANCH, Metric.INSTRUCTION, Metric.MUTATION, Metric.TEST_STRENGTH, Metric.CYCLOMATIC_COMPLEXITY, | ||
| Metric.TESTS, Metric.MCDC_PAIR, Metric.FUNCTION_CALL, Metric.COGNITIVE_COMPLEXITY, Metric.NCSS, Metric.NPATH_COMPLEXITY, | ||
| Metric.STMT_DC, Metric.STMT_CC, Metric.CONDITION, Metric.BYTES); | ||
|
Comment on lines
+84
to
+86
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I think we need to make these things configurable. It makes no sense to show all of these for everyone.
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Hello, I’m sorry for the delayed response. Instruction coverage (along with all other T32 metric) is needed in the view model. I understand that Instruction coverage will now be visible to everyone, so that's why I wanted it to be named as "Statement" and only associated with T32 coverage.
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. @uhafner just a gentle follow-up, it's been about a month since my last message, I'd really appreciate a reply when you have a moment. Thanks! |
||
| private final Run<?, ?> owner; | ||
| private final String displayName; | ||
| private final CoverageStatistics statistics; | ||
|
|
||
Uh oh!
There was an error while loading. Please reload this page.