Add timeout support via pytest-timeout#136
Open
VincentVanlaer wants to merge 4 commits intopython-trio:masterfrom
Open
Add timeout support via pytest-timeout#136VincentVanlaer wants to merge 4 commits intopython-trio:masterfrom
VincentVanlaer wants to merge 4 commits intopython-trio:masterfrom
Conversation
Whenever the trio_timeout option is enabled, this plugin will hook into requests from pytest-timeout to set a timeout. It will then start a thread in the background that, after the timeout has reached, will inject a system task in the test loop. This system task will collect stacktraces for all tasks and raise an exception that will terminate the test. The timeout thread is reused for other tests as well to not incur a startup cost for every test. Since this feature integrates with pytest-timeout, it also honors things like whether a debugger is attached or not. Drawbacks: - Ideally, whether trio does timeouts should not be a global option, but would be better suited for the timeout-method in pytest-timeout. This would require a change in pytest-timeout to let plugins register other timeout methods. - This method requires a functioning loop. Fixes python-trio#53
The pytest tester does not run tests in a subinterpreter or subprocess, so changing trio.run in a testdir test will also change the global trio.run. Monkeypatching resets trio.run after the test.
VincentVanlaer
force-pushed
the
timeout
branch
from
797a19a to
8ff6718
Compare
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
1 participant
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.
Whenever the trio_timeout option is enabled, this plugin will hook into
requests from pytest-timeout to set a timeout. It will then start a
thread in the background that, after the timeout has reached, will
inject a system task in the test loop. This system task will collect
stacktraces for all tasks and raise an exception that will terminate the
test. The timeout thread is reused for other tests as well to not incur
a startup cost for every test.
Since this feature integrates with pytest-timeout, it also honors things
like whether a debugger is attached or not.
Drawbacks:
would be better suited for the timeout-method in pytest-timeout. This
would require a change in pytest-timeout to let plugins register other
timeout methods.
Fixes #53