Skip to content

Conversation

@radarhere
Copy link
Member

In discussing the difference between pytest -v and pytest -vv with regards to checking iOS wheels, @freakboy3742 said

In case of success, there's no real difference; but in case of failure, -vv will sometimes include failure context (such as complete diffs) that is truncated by -v. If we can put the full context in the logs, why not do so?

I'm interested in consistency, so if that argument holds for testing iOS wheels, then why not everywhere else as well?

You can read about the effect at https://docs.pytest.org/en/stable/how-to/output.html#verbosity

@radarhere
Copy link
Member Author

I've pushed a commit to exit early on test failure in wheel tests - which we're already doing in some wheels that use wheels-test.ps1.

& $venv\Scripts\$python -m pytest -vx Tests\check_wheel.py
if (!$?) { exit $LASTEXITCODE }
& $venv\Scripts\$python -m pytest -vx Tests

@radarhere radarhere merged commit 69c0c42 into python-pillow:main Jun 28, 2025
70 checks passed
@radarhere radarhere deleted the pytest branch June 28, 2025 00:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants