testtools-dev team mailing list archive

[Jelmer Vernooij <jelmer@xxxxxxxxx>]

Jelmer Vernooij has proposed merging lp:~jelmer/testtools/stdout into lp:testtools.

Requested reviews:
  testtools developers (testtools-dev)


"python -m subunit.run --list" doesn't specify a stdout argument to testtools.run.TestProgram's constructuro. This causes it to use the default of "None", which doesn't make much sense. 

This branch changes the default to be sys.stdout instead.
-- 
https://code.launchpad.net/~jelmer/testtools/stdout/+merge/42946
Your team testtools developers is requested to review the proposed merge of lp:~jelmer/testtools/stdout into lp:testtools.

=== modified file 'testtools/run.py'
--- testtools/run.py	2010-11-29 23:46:10 +0000
+++ testtools/run.py	2010-12-07 14:17:09 +0000
@@ -132,6 +132,8 @@
             self.module = module
         if argv is None:
             argv = sys.argv
+        if stdout is None:
+            stdout = sys.stdout
 
         self.exit = exit
         self.failfast = failfast