its-consulting/ralph-claude-code
2
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

fix(ci): use full path to bats for kcov coverage

Browse source 
kcov spawns a subprocess that doesn't inherit the npm PATH, so
the 'bats' command wasn't found. Use the full path to the bats
binary in node_modules/.bin/ to fix coverage measurement.
This commit is contained in:
frankbria 2026-01-08 23:09:03 -07:00
parent f286710ee4
commit 0c0b0b63b1
1 changed files with 5 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

7
.github/workflows/test.yml vendored
View file

@ -95,17 +95,20 @@ jobs:
run: | run: |
mkdir -p coverage mkdir -p coverage
# Use full path to bats since kcov subprocess doesn't inherit npm PATH
BATS_CMD="$(pwd)/node_modules/.bin/bats"
# Run CLI parsing tests under kcov # Run CLI parsing tests under kcov
kcov --include-path="$(pwd)/ralph_loop.sh,$(pwd)/lib" \ kcov --include-path="$(pwd)/ralph_loop.sh,$(pwd)/lib" \
--exclude-pattern=tests/,node_modules/ \ --exclude-pattern=tests/,node_modules/ \
coverage/cli-parsing \ coverage/cli-parsing \
bash -c "bats tests/unit/test_cli_parsing.bats" || true bash -c "$BATS_CMD tests/unit/test_cli_parsing.bats" || true
# Run all unit tests under kcov for comprehensive coverage # Run all unit tests under kcov for comprehensive coverage
kcov --include-path="$(pwd)/ralph_loop.sh,$(pwd)/lib" \ kcov --include-path="$(pwd)/ralph_loop.sh,$(pwd)/lib" \
--exclude-pattern=tests/,node_modules/ \ --exclude-pattern=tests/,node_modules/ \
coverage/all-unit \ coverage/all-unit \
bash -c "bats tests/unit/" || true bash -c "$BATS_CMD tests/unit/" || true
- name: Parse coverage results - name: Parse coverage results
id: coverage id: coverage