Skip to content

Fixed Jacoco code coverage reporting#232

Draft
zack-rma wants to merge 2 commits intomainfrom
bugfix/sonar_jacoco
Draft

Fixed Jacoco code coverage reporting#232
zack-rma wants to merge 2 commits intomainfrom
bugfix/sonar_jacoco

Conversation

@zack-rma
Copy link
Contributor

@zack-rma zack-rma commented Mar 6, 2026

Description

Adds Jacoco test reporting to CI tasks. Downloads reports for Sonar Analysis.

Motivation and Context

Sonar Analysis was not being provided with Jacoco reports.

Types of changes

Yes/No Pull Request Type Description
Bug fix non-breaking change which fixes an issue
New feature non-breaking change which adds functionality
Breaking change fix or feature that would cause existing functionality to change
Documentation change non-breaking change which modifies or updates documentation
New tests new unit tests, test scenarios, or test case documentation
Triggers regression testing change affects downstream modules and will require regression testing
Y Other CI/CD

Notes for Reviewers

Please consider the following when reviewing this PR:

  • Correctness: Does the code do what it claims? Are edge cases handled appropriately?
  • Clarity: Is the code readable, maintainable and aligned with SOLID design principles?
  • Impact: Will this change affect other parts of the system? Any potential regressions?
  • Testing: Are the test cases sufficient and appropriate? Are there gaps in coverage?
  • Documentation: Does this require updates to code comments, README, or other docs?

@sonarqubecloud
Copy link

sonarqubecloud bot commented Mar 6, 2026

@dwr-zroy dwr-zroy moved this to In Progress in WRIMS 3 Development Mar 6, 2026
@dwr-zroy dwr-zroy added testing related Relates to unit, integration, regression, or smoke tests ci cd Relates to the CI/CD process labels Mar 6, 2026
@dwr-zroy dwr-zroy added this to the 3.0 Release Candidate milestone Mar 6, 2026
@dwr-zroy dwr-zroy linked an issue Mar 6, 2026 that may be closed by this pull request
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci cd Relates to the CI/CD process testing related Relates to unit, integration, regression, or smoke tests

Projects

Status: In Progress

Development

Successfully merging this pull request may close these issues.

WRIMS Engine Jacoco coverage not being reported

2 participants