Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix coverage of the Python tests #467

Merged
merged 1 commit into from
Oct 18, 2021
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
14 changes: 10 additions & 4 deletions .github/workflows/python_build.yml
Original file line number Diff line number Diff line change
Expand Up @@ -57,15 +57,21 @@ jobs:

- name: Build and install deltalake
run: |
# disable manylinux audit checks for test builds
make install MATURIN_EXTRA_ARGS="--manylinux off"
pip install virtualenv
virtualenv venv
source venv/bin/activate
make develop

- name: Run tests
run: make unit-test
run: |
source venv/bin/activate
make unit-test

# - name: Run Integration tests
# run: |
# py.test --cov tests -m integration

- name: Build Sphinx documentation
run: make build-documentation
run: |
source venv/bin/activate
make build-documentation
2 changes: 1 addition & 1 deletion python/Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -59,7 +59,7 @@ check-python: ## Run check on Python
.PHONY: unit-test
unit-test: ## Run unit test
$(info --- Run Python unit-test ---)
py.test --cov tests -m 'not integration'
python -m pytest

.PHONY: build-documentation
build-documentation: ## Build documentation with Sphinx
Expand Down
8 changes: 7 additions & 1 deletion python/pyproject.toml
Original file line number Diff line number Diff line change
Expand Up @@ -66,4 +66,10 @@ src_paths = ["deltalake", "tests"]

[tool.black]
include = '\.pyi?$'
exclude = "venv"
exclude = "venv"

[tool.pytest.ini_options]
addopts = "--cov=deltalake -v -m 'not integration'"
testpaths = [
"tests",
]