This repository has been archived by the owner on Oct 27, 2022. It is now read-only.
-
Notifications
You must be signed in to change notification settings - Fork 46
Timeout doesn't work properly #67
Comments
jozo
added a commit
to onecommons/unfurl
that referenced
this issue
May 17, 2021
Description of the problem with subprocess: google/python-subprocess32#67
Here it's described: https://bugs.python.org/issue37424 |
aszs
pushed a commit
to onecommons/unfurl
that referenced
this issue
May 18, 2021
* #43 - Add tests for dryrun param in shell configurator * #44 - Add test for timing out Refactor the file and remove unittest * #43 - Change name of a function * #44 - Change timeout test to check for error, not time * #44 - Fix tests for python2 * #44 - Skip integration test for timeout in shell in case of Python 2 Description of the problem with subprocess: google/python-subprocess32#67 * #44 - Skip integration test for timeout in shell for Python version < 3.7.5 * #44 - Reduce duration of a test * #43 - Fix return value of canRunTask to make more sense
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Timeout parameter doesn't really force timeout. It's happening on both Python 3.5 and Python 2.7 + subprocess32. However it works in Python 3.9.5. Not sure if it's worth to fix it, but at least it will be documented here. :)
Tests:
Results
Python 3.5.10:
Python 2.7.18
Python 3.9.5
The text was updated successfully, but these errors were encountered: