# HG changeset patch
# User Gregory Szorc <gregory.szorc@gmail.com>
# Date 1397967459 25200
# Node ID 822050714e72437a8b98e1f42896d3d344c3a76b
# Parent  6a90ecb6597a9e35b9867b0f9109e7b94dc4711e
run-tests: move output writing to Test.run()

diff -r 6a90ecb6597a -r 822050714e72 tests/run-tests.py
--- a/tests/run-tests.py	Sat Apr 19 21:14:31 2014 -0700
+++ b/tests/run-tests.py	Sat Apr 19 21:17:39 2014 -0700
@@ -627,8 +627,6 @@
         try:
             ret, out = self._run(testtmp, replacements, env)
             updateduration()
-            result.ret = ret
-            result.out = out
         except KeyboardInterrupt:
             updateduration()
             log('INTERRUPTED: %s (after %d seconds)' % (self._test,
@@ -698,6 +696,14 @@
                 f.write(line)
             f.close()
 
+        vlog("# Ret was:", ret)
+
+        if not options.verbose:
+            iolock.acquire()
+            sys.stdout.write(res[0])
+            sys.stdout.flush()
+            iolock.release()
+
         return res
 
     def _run(self, testtmp, replacements, env):
@@ -793,8 +799,6 @@
     """Holds the result of a test execution."""
 
     def __init__(self):
-        self.ret = None
-        self.out = None
         self.duration = None
         self.skipped = False
 
@@ -1157,17 +1161,7 @@
     res = TestResult()
     result = t.run(res)
 
-    ret = res.ret
-    out = res.out
-
     times.append((test, res.duration))
-    vlog("# Ret was:", ret)
-
-    if not options.verbose:
-        iolock.acquire()
-        sys.stdout.write(result[0])
-        sys.stdout.flush()
-        iolock.release()
 
     t.cleanup()