Make ZopeTestInSubProcess a RunTest thing, rather than a TestCase mixin
Bug #741191 reported by
Jonathan Lange
This bug affects 1 person
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Launchpad itself |
Fix Released
|
Low
|
Colin Watson |
Bug Description
ZopeTestInSubPr
class TestFoo(TestCase):
run_tests_with = ZopeSubProcessR
Rather than::
class TestFoo(
The main advantage to this would be that we could attach the stdout and stderr of the child process to the main test case and have it appear in test failures, which would be handy for debugging.
Related branches
~cjwatson/launchpad:refactor-zope-test-in-subprocess
Merged
into
launchpad:master
- Ioana Lasc (community): Approve
-
Diff: 370 lines (+88/-77)4 files modifiedlib/lp/bugs/scripts/checkwatches/tests/test_core.py (+9/-6)
lib/lp/services/job/tests/test_runner.py (+4/-3)
lib/lp/testing/__init__.py (+44/-40)
lib/lp/testing/tests/test_run_isolated_test.py (+31/-28)
To post a comment you must log in.
I hadn't seen this bug at the time, but I fixed this last year in https:/ /code.launchpad .net/~cjwatson/ launchpad/ +git/launchpad/ +merge/ 387645.