8000 Replace doctest runner by pytest in ci by tobiasdiez · Pull Request #36981 · sagemath/sage · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Replace doctest runner by pytest in ci #36981

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.

Sign up for GitHub

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

Jump to bottom
Draft
wants to merge 84 commits into
base: develop
Choose a base branch
from
Draft

Replace doctest runner by pytest in ci #36981

wants to merge 84 commits into from

Conversation

tobiasdiez
Copy link
Contributor
@tobiasdiez tobiasdiez commented Dec 29, 2023 •
edited
Loading

Replace sage's custom doctest runner by pytest in ci.

TODO:

  • Require pytest >= 8.1 (because consider_namespace_packages was added there)

📝 Checklist

  • The title is concise, informative, and self-explanatory.
  • The description explains in detail what this PR is about.
  • I have linked a relevant issue or discussion.
  • I have created tests covering the changes.
  • I have updated the documentation accordingly.

⌛ Dependencies

@tobiasdiez
Run pytest in conda ci
5fc47b2
@tobiasdiez
Fix missing feature import errors in SageDoctestModule
a1f88a4
@tobiasdiez
Skip when module not found error is raised during collection
af6959c
@tobiasdiez
Fix pytest command in build and ci-conda workflows
36eea75
@tobiasdiez
Update pytest options in build and ci-conda workflows to show summary…
bdfe4d8 
… of errors and failures
@tobiasdiez
Add rich output backend for doctest
fd809d9
@tobiasdiez
Replace _get_runner with SageDocTestRunner in conftest.py
86defa1
@tobiasdiez
Update pytest command to include verbose output
8a6995e
@tobiasdiez
Add pytest-github-actions-annotate-failures to test dependencies
bf6ac0d
@tobiasdiez
Only use sage spoofer if output file is specified
6c6582f
@tobiasdiez
Always run pytest
96f9e58
@tobiasdiez
Fix start_spoofing and stop_spoofing calls in SageDocTestRunner
038695a
@tobiasdiez
Refactor exception printing to use short names instead of Sage's hack…
490146f 
…y DoctestRunner
@tobiasdiez
add missing imports
a94a678
@tobiasdiez
Add imports to conftest.py
1c6fa26
@tobiasdiez
revert changes to forker
ad7a14b
@tobiasdiez
Fix a few display errors
6a77fd7
@tobiasdiez
Merge branch 'develop' into pytest_replace_doctest
4d3f882
@github-actions GitHub Actions
Copy link
github-actions bot commented Feb 25, 2024 •
edited
Loading

Documentation preview for this PR (built with commit ddf825f; changes) is ready! 🎉
This preview will update shortly after each push to this PR.

@tobiasdiez
Copy link
Contributor Author
tobiasdiez commented Mar 30, 2025 •
edited
Loading

Another error in CI:

src/sage/algebras/lie_conformal_algebras/abelian_lie_conformal_algebra.py ...
src/sage/algebras/lie_conformal_algebras/affine_lie_conformal_algebra.py ....
src/sage/algebras/lie_conformal_algebras/bosonic_ghosts_lie_conformal_algebra.py ...
src/sage/algebras/lie_conformal_algebras/fermionic_ghosts_lie_conformal_algebra.py ...
src/sage/algebras/lie_conformal_algebras/finitely_freely_generated_lca.py ......
src/sage/algebras/lie_conformal_algebras/free_bosons_lie_conformal_algebra.py ....
src/sage/algebras/lie_conformal_algebras/free_fermions_lie_conformal_algebra.py ....
src/sage/algebras/lie_conformal_algebras/freely_generated_lie_conformal_algebra.py ...
src/sage/algebras/lie_conformal_algebras/graded_lie_conformal_algebra.py ..
src/sage/algebras/lie_conformal_algebras/lie_conformal_algebra.py ...
/home/runner/miniconda3/envs/sage-dev/lib/python3.12/multiprocessing/resource_tracker.py:255: UserWarning: resource_tracker: There appear to be 5 leaked shared_memory objects to clean up at shutdown
  warnings.warn('resource_tracker: There appear to be %d '
src/sage/algebras/lie_conformal_algebras/lie_conformal_algebra_element.py

and

src/sage/algebras/hecke_algebras/ariki_koike_algebra.py .............................................
src/sage/algebras/hecke_algebras/ariki_koike_specht_modules.py ............
/home/runner/miniconda3/envs/sage-dev/lib/python3.12/multiprocessing/resource_tracker.py:255: UserWarning: resource_tracker: There appear to be 5 leaked shared_memory objects to clean up at shutdown
  warnings.warn('resource_tracker: There appear to be %d '
src/sage/algebras/hecke_algebras/cubic_hecke_algebra.py ........

@tobiasdiez
Copy link
Contributor Author

And

// linux
Error: [doctest] sage.calculus.calculus.laplace
src/sage/calculus/calculus.py .............F.........
Error: The operation was canceled.

// macos
src/sage/calculus/transforms/fft.pyx F.....FFF.FF
src/sage/calculus/var.pyx ...
Error: [doctest] src.sage.calculus.wester
src/sage/calculus/wester.py F

@tobiasdiez
Copy link
Contributor Author 
//linux
src/sage/coding/codes_catalog.py .
src/sage/coding/cyclic_code.py ...........................................
src/sage/coding/databases.py ..
src/sage/coding/decoder.py ............
src/sage/coding/decoders_catalog.py .
src/sage/coding/delsarte_bounds.py ......
Error: The operation was canceled.

tobiasdiez and others added 21 commits April 3, 2025 08:37
@tobiasdiez
Merge remote-tracking branch 'upstream/develop' into pytest_replace_d…
00a554a 
…octest
@tobiasdiez
Merge remote-tracking branch 'upstream/develop' into pytest_replace_d…
c7b99d6 
…octest
@tobiasdiez
Merge branch 'develop' into pytest_replace_doctest
1b1a230
@tobiasdiez
Merge branch 'develop' into pytest_replace_doctest
73c63bc
@tobiasdiez
Merge branch 'develop' into pytest_replace_doctest
42490cb
@tobiasdiez
Don't try to import src/sage_docbuild/build_options.py
892670d
@tobiasdiez
Ignore builders.py in addition to build_options.py for Meson comp…
9967f48 
…atibility
@tobiasdiez
Ignore more fatal python errors
a5dacb9
@tobiasdiez
Ignore atexit.pyx in cpython directory to prevent fatal Python 57AE er…
dc37fa6 
…rors
@tobiasdiez
Conditional pytest-isolate installation not on Windows
dd13d79
@tobiasdiez
Rename more test_ to check_
19295a0
@tobiasdiez
Improve error message for skipped doctests on import failure
f549d82
@tobiasdiez
Fix condition for installing pytest-isolate based on matrix OS
0af1a89
@tobiasdiez
Enhance pytest command with full trace option for better error visibi…
c1da4b3 
…lity
@tobiasdiez
Ignore specific files in pytest collection to prevent output stream i…
6faf68b 
…ssues
@tobiasdiez
Remove incompatible test
1eb59cb
@tobiasdiez
Fix docstring formatting to raw string literals
47f56b1
@tobiasdiez
Add ignore condition for calculus_method.py in pytest collection
79f434f
@tobiasdiez
Prevent fatal errors in specific test files
a6e05f9
@tobiasdiez
Add more to pytest ignore logic to prevent fatal errors
cbd89a9
@tobiasdiez
Exclude a few more files
ddf825f
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
c: scripts
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@tobiasdiez @mkoeppe @user202729
0