Skip to content

Collect test coverage data from unit and integration tests#10251

Merged
scholzj merged 1 commit into
mainfrom
collect-test-coverage-data
Jun 23, 2024
Merged

Collect test coverage data from unit and integration tests#10251
scholzj merged 1 commit into
mainfrom
collect-test-coverage-data

Conversation

@scholzj

@scholzj scholzj commented Jun 21, 2024

Copy link
Copy Markdown
Member

Type of change

  • Enhancement / new feature

Description

This PR tries to update the Jacoco configuration to generate test coverage data from our unit and integration tests and it also integrates the coverage data into the Azure Pipeline. Azure provides only basic information in the UI (see the screenshot). But more information can be found in the build artifacts and of course also locally.

Screenshot 2024-06-22 at 15 49 41

@scholzj scholzj added this to the 0.42.0 milestone Jun 21, 2024
Signed-off-by: Jakub Scholz <www@scholzj.com>
@scholzj scholzj force-pushed the collect-test-coverage-data branch from 8672856 to d165630 Compare June 21, 2024 23:37
@scholzj scholzj requested a review from a team June 22, 2024 13:52
@scholzj scholzj marked this pull request as ready for review June 22, 2024 13:52
@scholzj scholzj changed the title Try to collect test coverage data from unit and integration tests Collect test coverage data from unit and integration tests Jun 22, 2024

@im-konge im-konge left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM, the test coverage statistics are really interesting and useful. Thanks! :)

@see-quick see-quick left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM, this is great! 👍 Thanks for it!

@scholzj scholzj merged commit fc21545 into main Jun 23, 2024
@scholzj scholzj deleted the collect-test-coverage-data branch June 23, 2024 12:47
@bboyleonp666 bboyleonp666 mentioned this pull request Jun 6, 2026
1 task
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.

4 participants