Added support for TRACE32 parser and metrics#662
Open
IgorMiksza wants to merge 70 commits into
Open
Conversation
uhafner
reviewed
Jan 19, 2026
Member
There was a problem hiding this comment.
This needs to be the matching incrementals version of the coverage model:
Deployed version 0.66.0-rc1266.b_e6b_17a_f3c2c to Incrementals
Author
There was a problem hiding this comment.
This needs to be the matching incrementals version of the coverage model:
Deployed version 0.66.0-rc1266.b_e6b_17a_f3c2c to Incrementals
Version bumped, thanks :)
Bumps [com.fasterxml.jackson.core:jackson-databind](https://github.com/FasterXML/jackson) from 2.21.0 to 2.21.2. - [Commits](https://github.com/FasterXML/jackson/commits) --- updated-dependencies: - dependency-name: com.fasterxml.jackson.core:jackson-databind dependency-version: 2.21.2 dependency-type: direct:development update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com>
Bumps [org.jenkins-ci.tools:maven-hpi-plugin](https://github.com/jenkinsci/maven-hpi-plugin) from 3.1795.v6fb_496fc129b_ to 3.1820.ve687007a_69e5. - [Release notes](https://github.com/jenkinsci/maven-hpi-plugin/releases) - [Commits](https://github.com/jenkinsci/maven-hpi-plugin/commits) --- updated-dependencies: - dependency-name: org.jenkins-ci.tools:maven-hpi-plugin dependency-version: 3.1820.ve687007a_69e5 dependency-type: direct:development update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com>
Added `Metric.INSTRUCTION` to ViewModel, TableModel, Summary, Overview Updated results tests for Coverage Publisher Overview and Summary
IgorMiksza
force-pushed
the
trace32-coverage
branch
from
543e641 to
1d25de4
Compare
Now the PR number is written by `quality-monitor-build.yml` and attached as an artifact. This artifact is then read by `quality-monitor-comment-pr.yml`. So the PR number is always available and the delta computation works.
…jenkins-ci.tools-maven-hpi-plugin-3.1820.ve687007a_69e5 Bump org.jenkins-ci.tools:maven-hpi-plugin from 3.1795.v6fb_496fc129b_ to 3.1820.ve687007a_69e5
…fasterxml.jackson.core-jackson-databind-2.21.2 Bump com.fasterxml.jackson.core:jackson-databind from 2.21.0 to 2.21.2
Also, delete temporary folder before uploading reports
Replace Dependabot with Renovate
…dson.plugins-analysis-pom-12.x Bump dependency org.jvnet.hudson.plugins:analysis-pom to v12
…ntainers-java-monorepo Bump dependency org.testcontainers:testcontainers to v2
Chaning the fields to serializable classes breaks the existing serializations.
Restore serialization to use interfaces again
uhafner
requested changes
Apr 15, 2026
Member
uhafner
left a comment
uhafner
left a comment
There was a problem hiding this comment.
Can we find a way to not change the views for existing users?
Comment on lines
+84
to
+86
| 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); |
Member
There was a problem hiding this comment.
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.
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.
Is there a way to display Instruction coverage only with T32 reports?
Author
There was a problem hiding this comment.
@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!
* Revert version of j2html to 1.4.0: Versions starting from 1.5.0 are not backward-compatible anymore. * Replace Jackson 2 references with Jackson 3
Make all properties of the coverage row data class public again. Jackson 3 serializes only non-annotated properties that are marked as public. Fixes jenkinsci#727
Added `Metric.INSTRUCTION` to ViewModel, TableModel, Summary, Overview Updated results tests for Coverage Publisher Overview and Summary
…rage-plugin into trace32-coverage
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Hello, I've added support for TRACE32 parser and metrics that include Statement, Decision, Condition, Function, Object Code and Bytes coverages.
I've added our metrics to
CoverageTrendChart,CoverageBuildAction,CoverageChecksPublisher,CoverageTableModel,CoverageViewModelandMessages.properties.I've also added TRACE32 to
README.md,pom.xmldescription,CoverageTool.javaandMessages.properties.An example of our reports can be found in the coverage-model pull request.