Garbage threads hang the testrunner when using subunit output
Bug #589787 reported by
Māris Fogels
This bug affects 1 person
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Launchpad itself |
Fix Released
|
High
|
Māris Fogels | ||
| zope.testing |
Invalid
|
Undecided
|
Unassigned | ||
| zope.testrunner |
Fix Released
|
Undecided
|
Unassigned | ||
Bug Description
Any tests that leave live thread garbage cause the testrunner to hang when the following conditions are met:
a) The test is in a layer
b) The layer is in a subprocess
c) The testrunner is producing subunit-formatted output (via the --subunit flag)
d) A test leaves live thread garbage
The child layer's testrunner raises an Exception trying to format the thread garbage, which hangs the child process. The parent layer does not know how to deal with this loss, so it hangs. Here is the traceback from the formatting error: http://
This bug was observed in zope.testing 3.9.4. Presumably it has been present since subunit output was introduced in 3.9.0.
Related branches
lp:~lifeless/zope.testing/subunit
- Benji York (community): Approve
- ZTK steering group: Pending requested
-
Diff: 30 lines (+3/-3)1 file modifiedsrc/zope/testing/testrunner/formatter.py (+3/-3)
lp:~mars/launchpad/use-zope.testing-3.9.4-p0
- Gary Poster (community): Approve
-
Diff: 13 lines (+2/-1)1 file modifiedversions.cfg (+2/-1)
| Changed in launchpad-foundations: | |
| status: | New → In Progress |
| importance: | Undecided → High |
| assignee: | nobody → Māris Fogels (mars) |
Revision history for this message
| Māris Fogels (mars) wrote | #1 |
| Changed in launchpad-foundations: | |
| status: | In Progress → Fix Committed |
Revision history for this message
| Ursula Junque (ursinha) wrote Bug fixed by a commit | #2 |
| Changed in launchpad-foundations: | |
| milestone: | none → 10.06 |
| tags: | added: qa-needstesting |
| tags: |
added: qa-untestable removed: qa-needstesting |
| Changed in launchpad-foundations: | |
| status: | Fix Committed → Fix Released |
| Changed in zope.testing: | |
| status: | New → Invalid |
| Changed in zope.testrunner: | |
| status: | New → Fix Committed |
Revision history for this message
| Marius Gedminas (mgedmin) wrote | #3 |
| Changed in zope.testrunner: | |
| status: | Fix Committed → Fix Released |
To post a comment you must log in.
This was fixed in Launchpad by landing a patched version of zope.testing 3.9.4, devel/r10956. We can integrate 3.9.7 at a later time.