# HG changeset patch
# User Matt Mackall <mpm@selenic.com>
# Date 1302382339 18000
# Node ID 55f4941f98c8bf9cf9fc4ca5f1d7c525ef5e3562
# Parent  3259a067c102bca72ea21c8d580c690cfc1136c4
tests: better output on timeouts

Formerly, timeouts would generate huge scrollback-destroying diffs and
weird -15 return codes in the summary. Now we simply report "timed
out".

diff -r 3259a067c102 -r 55f4941f98c8 tests/run-tests.py
--- a/tests/run-tests.py	Mon Feb 07 16:37:03 2011 -0500
+++ b/tests/run-tests.py	Sat Apr 09 15:52:19 2011 -0500
@@ -620,7 +620,8 @@
                 ret = os.WEXITSTATUS(ret)
         except Timeout:
             vlog('# Process %d timed out - killing it' % proc.pid)
-            ret = cleanup()
+            cleanup()
+            ret = 'timeout'
             output += ("\n### Abort: timeout after %d seconds.\n"
                        % options.timeout)
         except KeyboardInterrupt:
@@ -752,11 +753,13 @@
             skip(missing[-1])
     elif out != refout:
         mark = '!'
-        if ret:
+        if ret == 'timeout':
+            fail("timed out")
+        elif ret:
             fail("output changed and returned error code %d" % ret)
         else:
             fail("output changed")
-        if not options.nodiff:
+        if ret != 'timeout' and not options.nodiff:
             if options.view:
                 os.system("%s %s %s" % (options.view, ref, err))
             else: