# HG changeset patch
# User Gregory Szorc <gregory.szorc@gmail.com>
# Date 2014-04-20 03:53:45
# Node ID 206814c9072af82dee460f0ebb8e691c2f1d7cc9
# Parent  04e04766065f9a3fee093b9894574ac47fde6a7d

run-tests: move remaining result processing to Test.run()

diff --git a/tests/run-tests.py b/tests/run-tests.py
--- a/tests/run-tests.py
+++ b/tests/run-tests.py
@@ -662,6 +662,10 @@ class Test(object):
                 msg += 'output changed'
 
             return self.fail(msg, ret)
+        elif ret:
+            return self.fail(describe(ret), ret)
+        else:
+            return self.success()
 
     def _run(self, testtmp, replacements, env):
         raise NotImplemented('Subclasses must implement Test.run()')
@@ -1163,13 +1167,6 @@ def runone(options, test, count):
             f.write(line)
         f.close()
 
-    if result:
-        pass
-    elif ret:
-        result = t.fail(describe(ret), ret)
-    else:
-        result = t.success()
-
     if not options.verbose:
         iolock.acquire()
         sys.stdout.write(result[0])