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

Collect code coverage #895

Open
wants to merge 7 commits into
base: main
Choose a base branch
from
Open

Conversation

DaveTryon
Copy link
Contributor

@DaveTryon DaveTryon commented Jan 23, 2025

We currently produce no code coverage in our build pipelines. Clicking on the "Code Coverage" tab (sample link produces a blank page. This PR collects code coverage data during tests and publishes it to the "Code Coverage" tab.

Some observations:

  1. The Code Coverage tab organizes data by path. The paths are different on Windows, Linux, and macOS, and that gets confusing. To keep the tab simple, only the Windows results are published there
  2. Full coverage results are published in HTML format as a build artifact
  3. So far, coverage data is only available at the file level--ideally we'd be able to get method and line level information. Perhaps that's a future improvement.
  4. Collecting and organizing the code coverage data makes the PR builds slower by a minute or so

@DaveTryon DaveTryon requested a review from a team as a code owner January 23, 2025 01:56
@DaveTryon
Copy link
Contributor Author

/azp run

@DaveTryon
Copy link
Contributor Author

/azp run

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