Francois Gouget : testbot: Detect the 'invalid unit test' error messages.

Alexandre Julliard julliard at winehq.org
Tue Mar 21 14:11:14 CDT 2017


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

Author: Francois Gouget <fgouget at codeweavers.com>
Date:   Mon Mar 20 23:57:08 2017 +0100

testbot: Detect the 'invalid unit test' error messages.

Take them into account so we don't complain the test returns non-zero
when it has no test failure.
Also show these errors in the web report summary.

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

---

 testbot/bin/WineRunTask.pl | 9 +++++++++
 testbot/web/JobDetails.pl  | 1 +
 2 files changed, 10 insertions(+)

diff --git a/testbot/bin/WineRunTask.pl b/testbot/bin/WineRunTask.pl
index fcde9f8..faf8314 100755
--- a/testbot/bin/WineRunTask.pl
+++ b/testbot/bin/WineRunTask.pl
@@ -484,6 +484,15 @@ if ($TA->GetFile($RptFileName, $FullLogFileName))
         # If the failure is not for the current test unit we'll let its
         # developer hash it out with the polluter ;-)
         $CurrentIsPolluted = 1 if ($1 ne $CurrentUnit);
+
+        $LineFailures++;
+      }
+      elsif ($Line =~ /^Fatal: test '([_a-z0-9]+)' does not exist/)
+      {
+        # This also replaces a test summary line.
+        $CurrentPids{0} = 1;
+        $SummaryFailures++;
+
         $LineFailures++;
       }
       elsif ($Line =~ /^(?:([0-9a-f]+):)?([_.a-z0-9]+): unhandled exception [0-9a-fA-F]{8} at / or
diff --git a/testbot/web/JobDetails.pl b/testbot/web/JobDetails.pl
index c57094b..4a5596c 100644
--- a/testbot/web/JobDetails.pl
+++ b/testbot/web/JobDetails.pl
@@ -350,6 +350,7 @@ sub GenerateBody($)
           $CurrentDll = $1;
         }
         if ($FullLog || $Line =~ m/: Test failed: / ||
+            $Line =~ m/Fatal: test '[^']+' does not exist/ ||
             $Line =~ m/ done \(258\)/ ||
             $Line =~ m/: unhandled exception [0-9a-fA-F]{8} at /)
         {




More information about the wine-cvs mailing list