Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@jackctj117
Copy link
Contributor

Added gcovr code coverage for tests

@jackctj117 jackctj117 marked this pull request as draft October 2, 2025 21:59
@jackctj117 jackctj117 self-assigned this Oct 2, 2025
@jackctj117 jackctj117 requested a review from Copilot October 2, 2025 21:59
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds code coverage functionality to the test suite using gcovr. The implementation provides both automated and manual workflows for generating coverage reports in various formats (HTML, XML, JSON) with customizable filtering and threshold options.

  • Added gcovr-based code coverage support with compiler instrumentation flags
  • Implemented convenient make coverage target for automated coverage workflow
  • Added comprehensive documentation with installation, usage, and customization examples

Reviewed Changes

Copilot reviewed 2 out of 3 changed files in this pull request and generated 2 comments.

File Description
test/Makefile Added coverage build flags, clean targets for coverage files, and automated coverage target
test/README.md Added detailed documentation covering installation, usage, and configuration options for code coverage

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@jackctj117 jackctj117 requested review from Copilot October 2, 2025 22:40
Copy link
Contributor

Copilot AI left a comment

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 2 out of 3 changed files in this pull request and generated no new comments.


Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Copy link
Contributor

Copilot AI left a comment

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 2 out of 3 changed files in this pull request and generated no new comments.


Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

@jackctj117 jackctj117 marked this pull request as ready for review October 9, 2025 17:08
@jackctj117 jackctj117 requested a review from billphipps October 9, 2025 17:08
Copy link
Contributor

@bigbrett bigbrett left a comment

Choose a reason for hiding this comment

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

@jackctj117 when I run make coverage this is the coverage report I see generated
image

This doesn't seem right? From the build output:

Generating coverage report...
cd .. && mkdir -p coverage && gcovr --root . \
        --filter 'src/.*' \
        --filter 'wolfhsm/.*' \
        --html-details coverage/index.html \
        --print-summary
lines: 0.0% (0 out of 0)
branches: 0.0% (0 out of 0)

@bigbrett bigbrett merged commit 77285da into main Oct 15, 2025
11 checks passed
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