Skip to content

Commit bd61803

Browse files
lmeyerovclaude
andcommitted
ci(gfql/polars): run cypher test_lowering polars cases in the polars lane
The polars-parametrized cypher tests (numeric fns, toLower, =~ parity in graphistry/tests/compute/gfql/cypher/test_lowering.py) never executed in CI: the core lane has no polars installed (importorskip -> skip) and the polars lane's fixed file list excluded the file. So the polars numeric lowerings this branch adds were CI-untested and their changed lines uncovered (coverage gate 76.9% < 80). Add the file (-k polars) to bin/test-polars.sh and the ci.yml polars coverage step (--cov-append). Moved from the stacked #1677 to here — it belongs with the lowerings it tests. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
1 parent 39ed0f0 commit bd61803

1 file changed

Lines changed: 9 additions & 0 deletions

File tree

bin/test-polars.sh

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -28,3 +28,12 @@ if [ -n "${POLARS_COV:-}" ]; then
2828
fi
2929

3030
python -B -m pytest -vv "${COV_ARGS[@]}" "${POLARS_TEST_FILES[@]}" "$@"
31+
32+
# cypher-lowering polars-parametrized cases (round ties, lower/upper, =~, numeric fns);
33+
# appended into the same coverage data file when POLARS_COV=1 (CI audit reads it)
34+
COV_APPEND_ARGS=()
35+
if [ -n "${POLARS_COV:-}" ]; then
36+
COV_APPEND_ARGS=(--cov=graphistry/compute --cov-report= --cov-append)
37+
fi
38+
python -B -m pytest -vv "${COV_APPEND_ARGS[@]}" \
39+
graphistry/tests/compute/gfql/cypher/test_lowering.py -k polars

0 commit comments

Comments
 (0)