Skip to content

fix(ci): add controls and coverage on main branch#78

Merged
etiennej70 merged 2 commits into
mainfrom
ci/tests_coverage
Jun 22, 2026
Merged

fix(ci): add controls and coverage on main branch#78
etiennej70 merged 2 commits into
mainfrom
ci/tests_coverage

Conversation

@etiennej70

Copy link
Copy Markdown
Collaborator

PR Description

What this PR Provides

  • Tests and especially coverage where computed only on the PR branches. To be able to get coverage diff between the main branch and the working PRs, we need to execute the controls on the main branch as well.
  • It will also guarantee that the main branch stays healthy.

Fixes

  • N/A

Review

The reviewer must double-check these points:

  • The reviewer has tested the feature
  • The reviewer has reviewed the implementation of the feature
  • The documentation has been updated

How to test

  • The CI in this PR should be ISO with the previous execution
  • Once merged, ensure that the coverage of the main branch has been taken into account on Sonar and Github COde Quality dashboards
  • Rebase the other working PR to see if the diff is correctly computed

Breaking changes (if any)

  • N/A

@github-code-quality

github-code-quality Bot commented Jun 22, 2026

Copy link
Copy Markdown

Code Coverage Overview

Languages: Java

Java / code-coverage/jacoco

The overall coverage in the branch is 89%. Coverage data for the branch is not yet available.

Show a code coverage summary of the most covered files.
File ff608c2 +/-
com/decathlon/i...rDslParser.java 99%
com/decathlon/i...ionService.java 99%
com/decathlon/i...ityService.java 96%
com/decathlon/i...MapperImpl.java 90%
com/decathlon/i...MapperImpl.java 88%
com/decathlon/i...ionHandler.java 87%
com/decathlon/i...ionService.java 87%
com/decathlon/i...oOutMapper.java 86%
com/decathlon/i...cification.java 81%
com/decathlon/i...ionService.java 77%

Updated June 22, 2026 09:39 UTC
Code Coverage is in Public Preview. Learn more and provide us with your feedback.

@etiennej70 etiennej70 marked this pull request as ready for review June 22, 2026 09:36
@sonarqubecloud

Copy link
Copy Markdown

@etiennej70 etiennej70 merged commit c9cf757 into main Jun 22, 2026
25 checks passed
@etiennej70 etiennej70 deleted the ci/tests_coverage branch June 22, 2026 16:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants