# HG changeset patch
# User Matt Mackall <mpm@selenic.com>
# Date 1302382339 18000
# Node ID 97ec5d6590399cb6fc83124cf9541e06f8549119
# Parent  2ffafb688274379bdb8e073c8dbb1c97f1683017
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".

[ original upstream message ]

diff -r 2ffafb688274 -r 97ec5d659039 tests/run-tests.py
--- a/tests/run-tests.py	Tue Apr 05 11:07:25 2011 +0200
+++ 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: