Skip to content

Add the code coverage job#55

Open
tian-lt wants to merge 5 commits into
ngcpp:mainfrom
tian-lt:main
Open

Add the code coverage job#55
tian-lt wants to merge 5 commits into
ngcpp:mainfrom
tian-lt:main

Conversation

@tian-lt

@tian-lt tian-lt commented Jun 25, 2026

Copy link
Copy Markdown
Contributor

What

Adds a dedicated code coverage job to the CI pipeline. It builds the tests with GCC coverage instrumentation, runs both the hosted and freestanding test suites, generates a coverage report scoped to the public headers (include/proxy/), publishes a summary to the workflow run page, and uploads the full report as an artifact.

Samples

lines: 99.0% (483 out of 488)
functions: 24.6% (7784 out of 31594)
branches: 76.3% (100 out of 131)
image

Copilot AI review requested due to automatic review settings June 25, 2026 15:21

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Adds a dedicated code coverage build/test path and wires it into the existing CI pipeline so coverage reports can be generated and uploaded as artifacts.

Changes:

  • Added CMake configure/build/test presets for coverage (hosted + freestanding).
  • Introduced a reusable GitHub Actions workflow to run coverage-instrumented tests and generate a gcovr report.
  • Hooked the new coverage workflow into the main CI pipeline.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

File Description
CMakePresets.json Adds coverage and coverage-freestanding configure/build/test presets to enable coverage-instrumented builds.
.github/workflows/pipeline-ci.yml Invokes the new code coverage BVT workflow as an additional CI job.
.github/workflows/bvt-codecoverage.yml New reusable workflow that builds/tests with coverage and uploads gcovr reports as an artifact.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread CMakePresets.json
Copilot AI review requested due to automatic review settings June 25, 2026 16:04

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

Comment thread CMakePresets.json
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.

2 participants