Skip to content

Commit 2cc5522

Browse files
committed
Remove global doctest from pytest and split CI
Eliminate global doctest enablement from pytest defaults by removing the addopts line in pyproject.toml. Split the CI test execution in test.yml:92 to run unit tests. Run doctests in a separate step using --doctest-modules for the datafusion package at test.yml:99.
1 parent 93f4c34 commit 2cc5522

File tree

2 files changed

+8
-3
lines changed

2 files changed

+8
-3
lines changed

.github/workflows/test.yml

Lines changed: 8 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -89,12 +89,18 @@ jobs:
8989
exit 1
9090
fi
9191
92-
- name: Run tests
92+
- name: Run unit tests
9393
env:
9494
RUST_BACKTRACE: 1
9595
run: |
9696
git submodule update --init
97-
uv run --no-project pytest -v --import-mode=importlib
97+
uv run --no-project pytest -v --import-mode=importlib python/tests
98+
99+
- name: Run doctests
100+
env:
101+
RUST_BACKTRACE: 1
102+
run: |
103+
uv run --no-project pytest -v --import-mode=importlib --doctest-modules python/datafusion
98104
99105
- name: FFI unit tests
100106
run: |

pyproject.toml

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -71,7 +71,6 @@ features = ["substrait"]
7171
[tool.pytest.ini_options]
7272
asyncio_mode = "auto"
7373
asyncio_default_fixture_loop_scope = "function"
74-
addopts = "--doctest-modules"
7574
doctest_optionflags = ["NORMALIZE_WHITESPACE", "ELLIPSIS"]
7675
testpaths = ["python/tests", "python/datafusion"]
7776

0 commit comments

Comments
 (0)