Minor changes to support pytest #2675
Merged
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.
A pleasant surprise:
pytest
seems to supportunittest
based tests out of box!We've had occasional problems due to our custom test runner, and it seems like in the long term it's worth migrating to a tried and tested tool that handles stuff for us. The good part is that (after this PR)
pytest test -k "not interactive"
will be an alternate way of running tests that behaves likepython -m test
when running from the project directory, and the old machinery will still be default, and continue to work.This PR is is needed because:
test_
, so things that are not actually tests need a name changingFuture work
python3 -m pygame.tests
/python3 -m test
internally invoke pytest, maybe initially this should be controlled by an opt-in flag, but eventually it could be default