devs/cproof
1
2
Fork 1
Code Issues 43 Pull Requests 8 Actions 1 Packages Projects 1 Releases Wiki Activity

Improve Functional Test Stability and CI Reliability #79

Manually merged
jabber.developer merged 7 commits from fix/test-CI-stability into master 2026-02-02 16:47:30 +00:00
Conversation 4 Commits 7 Files Changed 8 +284 -56
7 changed files with 284 additions and 56 deletions
Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit 6c5c523a1a - Show all commits

13
ci-build.sh
View File

@@ -259,16 +259,11 @@ build_and_test() {
if [ "$run_coverage" = "yes" ]; then
echo "--> Collecting coverage data..."
if command -v lcov >/dev/null 2>&1; then
lcov --capture --directory . --output-file coverage.info \
lcov --capture --directory . --output-file coverage-full.info \
--rc lcov_branch_coverage=1 --ignore-errors inconsistent 2>&1 || true
jabber.developer marked this conversation as resolved Outdated
jabber.developer commented 2026-01-30 18:31:37 +00:00
Outdated
Review
Copy Link

Wouldn't it get caught in the error handler declared earlier?

Wouldn't it get caught in the error handler declared earlier?
jabber.developer2 commented 2026-02-02 11:54:47 +00:00
Outdated
Review
Copy Link

It is done foe log readability.

It is done foe log readability.
# Remove test files, stubs, system headers - only keep src/ production code
lcov --remove coverage.info \
'/usr/*' \
'*/tests/*' \
'*test*.c' \
'*stub*.c' \
'*/functionaltests/*' \
'*/unittests/*' \
# Extract only production code from src/ directory, exclude tests
# Pattern matches both Docker (/src/src/*) and CI (/usr/src/profanity/src/*)
lcov --extract coverage-full.info '*/profanity/src/*' '*/src/src/*' \
--output-file coverage.info \
--rc lcov_branch_coverage=1 --ignore-errors inconsistent 2>&1 || true
jabber.developer marked this conversation as resolved Outdated
jabber.developer commented 2026-01-30 18:31:48 +00:00
Outdated
Review
Copy Link

(same as above) Wouldn't it get caught in the error handler declared earlier?

(same as above) Wouldn't it get caught in the error handler declared earlier?
jabber.developer2 commented 2026-02-02 11:54:53 +00:00
Outdated
Review
Copy Link

It is done foe log readability.

It is done foe log readability.
if [ -f coverage.info ] && [ -s coverage.info ]; then