Skip to content

Commit

Permalink
re-enable doctests using pytest-cython
Browse files Browse the repository at this point in the history
Since upstream fixed the issues with pytest-8 in a new release.
  • Loading branch information
radhermit committed Apr 15, 2024
1 parent 994178a commit 1a86ec7
Showing 1 changed file with 7 additions and 10 deletions.
17 changes: 7 additions & 10 deletions tox.ini
Original file line number Diff line number Diff line change
Expand Up @@ -25,15 +25,14 @@ setenv =
deps =
{[tox]deps}
pytest
#pytest-cython
pytest-cython

[testenv:python]
description = run tests for default system python
commands =
python setup.py build_ext --inplace --develop
# run doctests -- disabled until upstream issue with pytest-8 is fixed
# see: https://github.com/lgpage/pytest-cython/issues/58
#pytest --doctest-cython src/pkgcraft
# run doctests
pytest --doctest-cython src/pkgcraft
# run regular tests
pytest {posargs}

Expand All @@ -47,9 +46,8 @@ deps =
commands =
# build cython extensions with coverage enabled
python setup.py build_ext --inplace --cython-coverage --develop
# run doctests -- disabled until upstream issue with pytest-8 is fixed
# see: https://github.com/lgpage/pytest-cython/issues/58
#coverage run -m pytest --doctest-cython src/pkgcraft
# run doctests
coverage run -m pytest --doctest-cython src/pkgcraft
# run regular tests
coverage run -m pytest {posargs}
coverage combine
Expand All @@ -63,9 +61,8 @@ description = run tests under valgrind to find memory handling issues
allowlist_externals = valgrind
commands =
python setup.py build_ext --inplace
# run doctests -- disabled until upstream issue with pytest-8 is fixed
# see: https://github.com/lgpage/pytest-cython/issues/58
#valgrind pytest --doctest-cython src/pkgcraft
# run doctests
valgrind pytest --doctest-cython src/pkgcraft
# run regular tests, skipping parallelized ones that cause failures
valgrind pytest -m "not parallel" {posargs}

Expand Down

0 comments on commit 1a86ec7

Please sign in to comment.