microsoft / microsoft/ebpf-for-windows

Code coverage numbers should include all C files

Open
#884 0 comments 0 reactions 0 assignees View on GitHub
help wanted triaged
Dominant language
C
Stars
3.6k
Forks
311
Avg merge
6d 10h
Merged PRs (30d)
21

Description

Currently, the code coverage is only for C files linked into unit_tests.exe and ebpf_client.exe.

All files that are not used to build these are then excluded from our code coverage totals.

The ideal scenario would be to have all C files included in our code coverage numbers.

Contributor guide

Open the contributing guide

Research direction

Start by examining the code-coverage configuration and the build inputs for unit_tests.exe and ebpf_client.exe. Determine how C files are selected and how unused files are excluded. Done means the coverage totals include all C files, including those not linked into either executable.

Written by the indexing model from the issue text.

Assessment

Tech stack
c
Domain
build-system, testing-qa
Issue type
Feature
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.