Skip to content

Commit 1e1a736

Browse files
krystophnyclaude
andcommitted
fix: ensure consistent coverage compilation flags across all CI test steps
The test-coverage CI workflow was building the library with coverage flags but then running certain tests without those flags, causing compilation interface mismatches. This resulted in implicit interface errors for the contains_unicode function despite it being properly declared in the module. Fixed by adding coverage flags to both test_system_fpm_example and test_system_cmake_example test steps to maintain compilation consistency. Resolves CI failures on PR #369 for test-coverage and ensures all test steps use the same compilation flags as the library build. 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
1 parent 25c1f7b commit 1e1a736

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

.github/workflows/test-coverage.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -88,11 +88,11 @@ jobs:
8888
8989
- name: Test FPM example build
9090
run: |
91-
make test ARGS="--target test_system_fpm_example"
91+
make test ARGS="--target test_system_fpm_example --flag '-fprofile-arcs -ftest-coverage'"
9292
9393
- name: Test CMake example build
9494
run: |
95-
make test ARGS="--target test_system_cmake_example"
95+
make test ARGS="--target test_system_cmake_example --flag '-fprofile-arcs -ftest-coverage'"
9696
9797
- name: Validate PNG output with PIL
9898
continue-on-error: true

0 commit comments

Comments
 (0)