Title: [97277] trunk/Tools
Revision
97277
Author
simon.fra...@apple.com
Date
2011-10-12 11:27:54 -0700 (Wed, 12 Oct 2011)

Log Message

Improve output of run-api-tests to make time-outs more obvious
https://bugs.webkit.org/show_bug.cgi?id=69788

Reviewed by Adam Roben.

List the tests that failed and timed out at the bottom of the output,
if run with --verbose.

* Scripts/run-api-tests:
(runAllTests):
(runTest):

Modified Paths

Diff

Modified: trunk/Tools/ChangeLog (97276 => 97277)


--- trunk/Tools/ChangeLog	2011-10-12 18:11:07 UTC (rev 97276)
+++ trunk/Tools/ChangeLog	2011-10-12 18:27:54 UTC (rev 97277)
@@ -1,3 +1,17 @@
+2011-10-12  Simon Fraser  <simon.fra...@apple.com>
+
+        Improve output of run-api-tests to make time-outs more obvious
+        https://bugs.webkit.org/show_bug.cgi?id=69788
+
+        Reviewed by Adam Roben.
+        
+        List the tests that failed and timed out at the bottom of the output,
+        if run with --verbose.
+
+        * Scripts/run-api-tests:
+        (runAllTests):
+        (runTest):
+
 2011-10-12  Sheriff Bot  <webkit.review....@gmail.com>
 
         Unreviewed, rolling out r97252.

Modified: trunk/Tools/Scripts/run-api-tests (97276 => 97277)


--- trunk/Tools/Scripts/run-api-tests	2011-10-12 18:11:07 UTC (rev 97276)
+++ trunk/Tools/Scripts/run-api-tests	2011-10-12 18:27:54 UTC (rev 97277)
@@ -58,6 +58,8 @@
 my $dump = 0;
 my $build = 1;
 my $buildDefault = $build ? "build" : "do not build";
+my @testsFailed;
+my @testsTimedOut;
 
 my $programName = basename($0);
 my $usage = <<EOF;
@@ -120,6 +122,21 @@
             $anyFailures = 1;
         }
     }
+    
+    if ($verbose) {
+        if (@testsFailed) {
+            print "Tests that failed:\n";
+            for my $test (@testsFailed) {
+                print "  $test\n";
+            }
+        }
+        if (@testsTimedOut) {
+            print "Tests that timed out:\n";
+            for my $test (@testsTimedOut) {
+                print "  $test\n";
+            }
+        }
+    }
     return $anyFailures;
 }
 
@@ -190,7 +207,11 @@
         $timedOut = 1;
     }
 
+    if ($result) {
+        push @testsFailed, "$suite.$test";
+    }
     if ($timedOut) {
+        push @testsTimedOut, "$suite.$test";
         print possiblyColored("bold yellow", "Timeout"), "\n";
     } elsif (!$verbose) {
         if ($result) {
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes

Reply via email to