| Age | Commit message (Collapse) | Author |
|---|
|
Added -DDEMO_STRIP_ALL=OFF to cmake configuration in gen_coverage_report.sh
to ensure all test code is included in coverage analysis.
Previously the script relied on the default value of STRIP_ALL, which
could potentially exclude test infrastructure code from coverage reports.
The remaining warnings in coverage output are benign lcov/genhtml warnings
about unknown categories and data inconsistencies, normal for coverage analysis.
Coverage: 57.8% lines, 76.0% functions (77 source files)
Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
|
|
Problem: Coverage script failed with error:
lcov: ERROR: (source) unable to open /Users/skal/demo/src/platform.cc
Root Cause:
- Old .gcno/.gcda coverage files referenced old src/platform.cc path
- File was moved to src/platform/platform.cc in earlier refactor
- Stale coverage data persisted between runs
Solution:
1. Added 'source' to LCOV_OPTS ignore list
- Handles missing source files gracefully
- Common when files are moved/renamed between coverage runs
2. Enable automatic cleanup of build_coverage/ directory
- Removes stale coverage data before each run
- Prevents conflicts from moved/renamed files
- Changed from commented-out to active cleanup
Result:
- Coverage report generates successfully
- 57.8% line coverage, 76.0% function coverage
- No errors about missing src/platform.cc
- Clean builds prevent stale data accumulation
The script now handles project reorganizations gracefully.
|
|
Updated gen_coverage_report.sh to accept an optional argument for targeting specific directories using lcov --extract.
|
|
Added CMake support for coverage builds and a script to generate HTML reports using lcov on macOS. Also cleaned up .gitignore.
|
