Skip to content

Conversation

@hsinn0
Copy link
Contributor

@hsinn0 hsinn0 commented Nov 20, 2024

  • Modified cargo.local to run with jacoco agent if a system property is set.
  • Added a task to generate coverage report from the recorded jacoco data.

Gradle console output:

> Task :jacocoLogCargoCoverage
Test Coverage:
    - Class Coverage: 79.5%
    - Method Coverage: 67.2%
    - Branch Coverage: 42.4%
    - Line Coverage: 64.3%
    - Instruction Coverage: 61.6%
    - Complexity Coverage: 45.7%

- Modified `cargo.local` to run with jacoco agent if a system property is set.
- Added a task to generate coverage report from the recorded jacoco data.
@hsinn0 hsinn0 requested a review from a team November 20, 2024 15:56
Copy link
Member

@strehle strehle left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

not yet adopted the sonar... but in future we should combine unit and integration tests in one run and then coverage in https://sonarcloud.io/summary/overall?id=cloudfoundry-identity-parent should increase

thanks

@hsinn0 hsinn0 requested a review from duanemay November 20, 2024 18:32
@hsinn0 hsinn0 merged commit 3ba7b4e into develop Nov 20, 2024
20 checks passed
@hsinn0 hsinn0 deleted the pr/integ_test_coverage2 branch November 20, 2024 21:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Development

Successfully merging this pull request may close these issues.

3 participants