Merge lp:~vila/testtools/trunk into lp:~testtools-committers/testtools/trunk

Proposed by Vincent Ladeuil
Status: Merged
Merged at revision: 295
Proposed branch: lp:~vila/testtools/trunk
Merge into: lp:~testtools-committers/testtools/trunk
Diff against target: 25 lines (+2/-2)
2 files modified
doc/for-test-authors.rst (+1/-1)
testtools/run.py (+1/-1)
To merge this branch: bzr merge lp:~vila/testtools/trunk
Reviewer Review Type Date Requested Status
Jonathan Lange Approve
Review via email: mp+136915@code.launchpad.net

Description of the change

Two small fixes here.

1) The doc for test authors has a typo.

2) While trying to reuse testtools.run.TestProgram I ran into the case where
   I didn't provide a testRunner argument. The default handling seems to have
   been imported from python where 'runner' is defined. That's the case here so
   I went with what seems to be a sane default.

To post a comment you must log in.
Revision history for this message
Jonathan Lange (jml) wrote :

Thanks!

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'doc/for-test-authors.rst'
2--- doc/for-test-authors.rst 2012-01-29 14:08:39 +0000
3+++ doc/for-test-authors.rst 2012-11-29 11:44:24 +0000
4@@ -432,7 +432,7 @@
5 def test_keys_equal(self):
6 x = {'a': 1, 'b': 2}
7 y = {'a': 2, 'b': 3}
8- self.assertThat(a, KeysEqual(b))
9+ self.assertThat(x, KeysEqual(y))
10
11
12 MatchesRegex
13
14=== modified file 'testtools/run.py'
15--- testtools/run.py 2012-10-17 22:38:13 +0000
16+++ testtools/run.py 2012-11-29 11:44:24 +0000
17@@ -307,7 +307,7 @@
18 and getattr(unittest, 'installHandler', None) is not None):
19 unittest.installHandler()
20 if self.testRunner is None:
21- self.testRunner = runner.TextTestRunner
22+ self.testRunner = TestToolsTestRunner(sys.stdout)
23 if isinstance(self.testRunner, classtypes()):
24 try:
25 testRunner = self.testRunner(verbosity=self.verbosity,

Subscribers

People subscribed via source and target branches