Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Generate coverage info file for unit test during package build #746

Merged
merged 1 commit into from
Feb 15, 2024

Conversation

mwperina
Copy link
Member

Prior to the patch .gcda files from unit tests execution are packaged
into bluechi-coverage and later merged with integration test .gcda file
to create unified code coverage report.
As a part of this patch we generate code coverage info file for unit
tests and package this info into bluechi-coverage, which simplifies
generating unified code coverage report.

Signed-off-by: Martin Perina mperina@redhat.com

@coveralls
Copy link

coveralls commented Feb 15, 2024

Coverage Status

coverage: 71.776% (+5.7%) from 66.09%
when pulling 6d799f6 on mwperina:unit-test-info-file-in-rpm
into b717af6 on eclipse-bluechi:main.

Prior to the patch .gcda files from unit tests execution are packaged
into bluechi-coverage and later merged with integration test .gcda file
to create unified code coverage report.
As a part of this patch we generate code coverage info file for unit
tests and package this info into bluechi-coverage, which simplifies
generating unified code coverage report.

Signed-off-by: Martin Perina <mperina@redhat.com>
@ygalblum
Copy link
Contributor

LGTM

Copy link
Member

@engelmi engelmi left a comment

Choose a reason for hiding this comment

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

Small comment, otherwise LGTM

@mwperina mwperina merged commit e8fba28 into eclipse-bluechi:main Feb 15, 2024
20 checks passed
@mwperina mwperina deleted the unit-test-info-file-in-rpm branch February 15, 2024 17:29
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