Skip to content

fix: Change test_load_builtin_modules to use module present in PyPy#363

Merged
pawamoy merged 1 commit intomkdocstrings:mainfrom
mgorny:test-pypy
Feb 25, 2025
Merged

fix: Change test_load_builtin_modules to use module present in PyPy#363
pawamoy merged 1 commit intomkdocstrings:mainfrom
mgorny:test-pypy

Conversation

@mgorny
Copy link
Contributor

@mgorny mgorny commented Feb 25, 2025

Change test_load_builtin_modules to use _operator rather than _json, to fix test failure on PyPy. PyPy does not feature a _json module, whereas _operator is present both on CPython and on PyPy.

Fixes #362

Change `test_load_builtin_modules` to use `_operator` rather than
`_json`, to fix test failure on PyPy.  PyPy does not feature a `_json`
module, whereas `_operator` is present both on CPython and on PyPy.

Fixes mkdocstrings#362
@pawamoy pawamoy merged commit 628d5e8 into mkdocstrings:main Feb 25, 2025
24 of 26 checks passed
@pawamoy
Copy link
Member

pawamoy commented Feb 25, 2025

Thanks a lot! 🚀 Do you need a new release?

@mgorny mgorny deleted the test-pypy branch February 25, 2025 15:23
@mgorny
Copy link
Contributor Author

mgorny commented Feb 25, 2025

Thanks! No need for a release, I've just skipped that one test on PyPy for the time being.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

bug: tests/test_loader.py::test_load_builtin_modules fails on PyPy3 (assumes internal CPython _json module)

2 participants