Skip to content
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.

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

Proposal: display skip reason inline with test result #2484

Closed
DariusL opened this issue Jun 9, 2017 · 2 comments
Closed

Proposal: display skip reason inline with test result #2484

DariusL opened this issue Jun 9, 2017 · 2 comments

Comments

@DariusL
Copy link

DariusL commented Jun 9, 2017

Currently the skip reason with -rs is displayed after the tests.

(venv) Dariuss-MacBook-Pro:e2e dlapunas$ pytest proposal.py -rs -v
====================================================================================================================================================================== test session starts ======================================================================================================================================================================

proposal.py::test_skipped SKIPPED
proposal.py::test_not_skipped PASSED

==================================================================================================================================================================== short test summary info ====================================================================================================================================================================
SKIP [1] proposal.py:3: some reason
============================================================================================================================================================== 1 passed, 1 skipped in 0.01 seconds ==============================================================================================================================================================

With -v often there's enough space to display the skip reason next to the test result. Should improve readability. Possibly something like.

(venv) Dariuss-MacBook-Pro:e2e dlapunas$ pytest proposal.py -rs -v
====================================================================================================================================================================== test session starts ======================================================================================================================================================================

proposal.py::test_skipped SKIPPED: some reason
proposal.py::test_not_skipped PASSED

============================================================================================================================================================== 1 passed, 1 skipped in 0.01 seconds ==============================================================================================================================================================
@DariusL DariusL changed the title Display skip reason inline with test result Proposal: display skip reason inline with test result Jun 9, 2017
@RonnyPfannschmidt
Copy link
Member

just a note - this kind of display would solve a different use-case (as the current -rs does aggregate, so if you have 100 tests skipping with the same skip, you still get 1 message not 100

@nicoddemus
Copy link
Member

Closing as duplicate of #2044

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

No branches or pull requests

3 participants