Skip to content

Commit

Permalink
[CHORE] collect all tests before running pytest so check for import e…
Browse files Browse the repository at this point in the history
…rrors (#2150)
  • Loading branch information
samster25 authored Apr 18, 2024
1 parent c7402ea commit 96f0dd9
Showing 1 changed file with 1 addition and 0 deletions.
1 change: 1 addition & 0 deletions .github/workflows/python-package.yml
Original file line number Diff line number Diff line change
Expand Up @@ -89,6 +89,7 @@ jobs:
# export CARGO_INCREMENTAL=1
# cargo llvm-cov clean --workspace
maturin develop
pytest --ignore tests/integration --collect-only -qq # run this to ensure no weird imports that result in `Collector` errors
mkdir -p report-output && pytest --cov=daft --ignore tests/integration --durations=50
coverage combine -a --data-file='.coverage' || true
coverage xml -o ./report-output/coverage-${{ runner.os }}-${{ matrix.python-version }}-${{ matrix.daft-runner }}-${{ matrix.pyarrow-version }}.xml
Expand Down

0 comments on commit 96f0dd9

Please sign in to comment.