add collect-only step for pytest

Eventual-Inc · May 9, 2024 · 40d2e11 · 40d2e11
1 parent 82c0f6b
commit 40d2e11
Showing 1 changed file with 3 additions and 1 deletion.
diff --git a/.github/workflows/python-package.yml b/.github/workflows/python-package.yml
@@ -94,8 +94,10 @@ jobs:
         export CARGO_TARGET_DIR=$CARGO_LLVM_COV_TARGET_DIR
         cargo llvm-cov clean --workspace
         maturin develop
-        mkdir -p report-output && pytest --cov=daft --ignore tests/integration --durations=50
+        pytest --ignore tests/integration --collect-only -qq # run this to ensure no weird imports that result in `Collector` errors
+        pytest --cov=daft --ignore tests/integration --durations=50
         coverage combine -a --data-file='.coverage' || true
+        mkdir -p report-output
         coverage xml -o ./report-output/coverage-${{ join(matrix.*, '-') }}.xml
         cargo llvm-cov report --lcov --output-path report-output/rust-coverage-${{ join(matrix.*, '-') }}.lcov
       env: