Use pytest-beartype-tests plugin#2144
Merged
Merged
Conversation
Replace manual pytest_collection_modifyitems hooks with the https://github.com/adamtheturtle/pytest-beartype-tests dev dependency.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This PR adds the
pytest-beartype-testsdev dependency and removes redundantpytest_collection_modifyitems/@beartypewiring from conftest where it duplicated the plugin.The plugin registers via
pytest11and applies@beartypeto collected test functions, matching the previous local hook behavior (see the upstream README).Note
Low Risk
Low risk: changes are limited to test tooling/dependencies and removal of a redundant pytest collection hook, with no production code impact.
Overview
Test-time runtime type checking is now provided via the
pytest-beartype-testsplugin instead of a localpytest_collection_modifyitemshook intests/conftest.py.The PR adds
pytest-beartype-teststo dev dependencies anduv.lock, removes the now-unused beartype wiring fromconftest.py, and adjusts config/metadata (including droppingpytest_collection_modifyitemsfrom vulture ignores and adding an empty[dependency-groups]section).Reviewed by Cursor Bugbot for commit 6cd6356. Bugbot is set up for automated code reviews on this repo. Configure here.