Francois Gouget : winetest: Let dissect detect and report crashes.

Alexandre Julliard julliard at winehq.org
Tue Jun 13 14:40:13 CDT 2017


Module: tools
Branch: master
Commit: ee648b52fea5d4d94cb46ffcffd73003cd9525d4
URL:    http://source.winehq.org/git/tools.git/?a=commit;h=ee648b52fea5d4d94cb46ffcffd73003cd9525d4

Author: Francois Gouget <fgouget at codeweavers.com>
Date:   Tue Jun 13 10:36:49 2017 +0200

winetest: Let dissect detect and report crashes.

A 'failed crash' line in the summary.txt file now identifies tests that
crashed. This way gather does not have to duplicate the dissect code to
correctly identify crashed tests (except temporarily for backward
compatibility with old summary.txt files).

Signed-off-by: Francois Gouget <fgouget at codeweavers.com>
Signed-off-by: Alexandre Julliard <julliard at winehq.org>

---

 winetest/dissect | 2 +-
 winetest/gather  | 4 +++-
 2 files changed, 4 insertions(+), 2 deletions(-)

diff --git a/winetest/dissect b/winetest/dissect
index 4901e97..1ecbd94 100755
--- a/winetest/dissect
+++ b/winetest/dissect
@@ -465,7 +465,7 @@ while (<IN>) {
     } elsif (/$dll:$unit(?::[0-9a-f]+)? done \((-?\d+)\)(?:\r?$| in)/) {
         chomp;                  # current test ended
         if ($lines==0 || $1 < 0) {
-            $result = "failed $1 - -";
+            $result = "failed ". ($1 < 0 ? "crash" : $1) ." - -";
             my $reason = "test failed: error $1";
             if ($1 == 258) { $reason = "test failed: timed out"; }
             elsif ($1 < 0) { $reason = "test failed: crash"; }
diff --git a/winetest/gather b/winetest/gather
index 907b205..fbf0397 100755
--- a/winetest/gather
+++ b/winetest/gather
@@ -324,7 +324,9 @@ foreach my $file (glob "$builddir/*/summary.txt") {
         $alltests{$testname} = $source;
 
         if ($count eq "failed") {
-            if ($todos eq "filelimit") {
+            if ($todos eq "crash") {
+                $report->{$testname} = { status => "crash" };
+            } elsif ($todos eq "filelimit") {
                 $report->{$testname} = { status => "file limit" };
                 $report->{filelimit} = $testname;
             } else {




More information about the wine-cvs mailing list