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

Unify unit and integration tests coverage report #711

Merged

Conversation

mwperina
Copy link
Member

  • Add unit test code coverage to integration test coverage report
  • Remove unit test covergae report from unit-tests workflow

@mwperina
Copy link
Member Author

Here is the unified report:
unit-and-integration-tests-coverage

@mwperina
Copy link
Member Author

/cc @ArtiomDivak

Copy link
Contributor

@ArtiomDivak ArtiomDivak left a comment

Choose a reason for hiding this comment

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

LGTM

@mwperina mwperina force-pushed the unit-and-intregation-tests-coverage branch from 6305520 to c0d4923 Compare January 23, 2024 13:26
Unit tests are executed during RPM build phase, so we need to add .gcda
files from unit test execution into bluechi-coverage package to be able
to create unified unit and integration test coverage report as a part of
integration test execution.

Signed-off-by: Martin Perina <mperina@redhat.com>
Unit tests are executed as a part of RPM build. Also we run integration
tests with code coverage report as a part if integration-tests workflow,
so there is no need to run coverage report for unit tests in unit-tests
workflow.

Signed-off-by: Martin Perina <mperina@redhat.com>
@mwperina mwperina force-pushed the unit-and-intregation-tests-coverage branch from c0d4923 to 03874f0 Compare January 26, 2024 16:41
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.

LGTM

@engelmi engelmi merged commit 21bc266 into eclipse-bluechi:main Jan 30, 2024
18 checks passed
@mwperina mwperina deleted the unit-and-intregation-tests-coverage branch January 30, 2024 11: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.

3 participants