Issue 19665: test_logging fails with SMTPHandler timeout (original) (raw)

Issue19665

Created on 2013-11-20 18:02 by canisdirus, last changed 2022-04-11 14:57 by admin. This issue is now closed.

Files
File name Uploaded Description Edit
test_logging.patch canisdirus,2013-11-20 18:02 Patch to up timeout on test_logging.SMTPHandlerTest by 1 sec review
Messages (3)
msg203515 - (view) Author: Mark Wolf (canisdirus) Date: 2013-11-20 18:02
Similar to http://bugs.python.org/issue14644 but with Arch Linux instead of OS X. I built python according the dev guide then ran the test suite with `./python -m test -vj3 test_logging` and failed in the SMTPHandlerTest. `self.handled.is_set()` returns false after a 5 second timeout. In one of the comments in the OS X bug linked above, it suggests changing the timeout to 15 seconds. This causes the test to pass (actually anything slightly over 5 sec seems to be okay). Not sure if this is a fragile test or a bug in Lib.logging. The patch ups the timeout by a second, which passes on my machine but might still fail on others if it actually is a fragile test issue. The failing test runner output for test_logging can be found here: https://gist.github.com/m3wolf/7563541 I'm just jumping into python dev for the first time; any constructive comments are appreciated.
msg205100 - (view) Author: Vinay Sajip (vinay.sajip) * (Python committer) Date: 2013-12-03 11:19
I think the test is fragile, but I'll bump the timeout as suggested.
msg205102 - (view) Author: Roundup Robot (python-dev) (Python triager) Date: 2013-12-03 11:30
New changeset 4c5fc9f08b4d by Vinay Sajip in branch '3.3': Issue #19665: Increased timeout for SMTPHandler test. http://hg.python.org/cpython/rev/4c5fc9f08b4d New changeset bfd45dc46569 by Vinay Sajip in branch 'default': Closes #19665: Merged fi from 3.3. http://hg.python.org/cpython/rev/bfd45dc46569
History
Date User Action Args
2022-04-11 14:57:53 admin set github: 63864
2013-12-03 11:30:06 python-dev set status: open -> closednosy: + python-devmessages: + resolution: fixedstage: resolved
2013-12-03 11:19:28 vinay.sajip set nosy: + vinay.sajipmessages: +
2013-11-20 18:02:46 canisdirus create