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

cmd: github: generate coverage data from C code executed in unit tests workflow #15048

Draft
wants to merge 21 commits into
base: master
Choose a base branch
from

Conversation

bboozzoo
Copy link
Contributor

@bboozzoo bboozzoo commented Feb 7, 2025

Generate coverage data from C code executed in make distcheck. Include the artifacts in upload to codecov.

@bboozzoo bboozzoo added the Skip spread Indicate that spread job should not run label Feb 7, 2025
Copy link

github-actions bot commented Feb 7, 2025

Wed Feb 12 11:09:22 UTC 2025

Spread tests skipped

@bboozzoo bboozzoo force-pushed the bboozzoo/test-coverage branch 5 times, most recently from 29d87ef to 73a2364 Compare February 11, 2025 07:21
Signed-off-by: Maciej Borzecki <[email protected]>
Add --enable-test-coverage flag and check for presence of gcovr if it's
enabled.

Signed-off-by: Maciej Borzecki <[email protected]>
Signed-off-by: Maciej Borzecki <[email protected]>
As it results in the same function appearing multiple times but in
different files and locations in the coverage data.

Signed-off-by: Maciej Borzecki <[email protected]>
Stop including C files directly in unit tests.

Signed-off-by: Maciej Borzecki <[email protected]>
Signed-off-by: Maciej Borzecki <[email protected]>
Signed-off-by: Maciej Borzecki <[email protected]>
Signed-off-by: Maciej Borzecki <[email protected]>
Signed-off-by: Maciej Borzecki <[email protected]>
@bboozzoo bboozzoo force-pushed the bboozzoo/test-coverage branch from d4da291 to 1bf9c64 Compare February 12, 2025 07:34
@bboozzoo
Copy link
Contributor Author

This should benefit from switching to ubuntu-latest once #15066 lands

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Skip spread Indicate that spread job should not run
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant