Francois Gouget : testbot/LogUtils: Let ParseWineTestReport() do the error detection.

[Alexandre Julliard]

Module: tools
Branch: master
Commit: 91edb4e2ed9650cae7192694be7b4ad002ac82a6
URL:    https://source.winehq.org/git/tools.git/?a=commit;h=91edb4e2ed9650cae7192694...

Author: Francois Gouget <fgouget(a)codeweavers.com>
Date:   Mon Mar  1 16:05:23 2021 +0100

testbot/LogUtils: Let ParseWineTestReport() do the error detection.

This reduces duplication and lets ParseWineTestReport()'s more
sophisticated analysis have the last word on which report lines contain
errors.

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

---

 testbot/lib/WineTestBot/LogUtils.pm | 16 ----------------
 1 file changed, 16 deletions(-)

diff --git a/testbot/lib/WineTestBot/LogUtils.pm b/testbot/lib/WineTestBot/LogUtils.pm
index 267dcd9..4d2619c 100644
--- a/testbot/lib/WineTestBot/LogUtils.pm
+++ b/testbot/lib/WineTestBot/LogUtils.pm
@@ -328,14 +328,6 @@ sub GetReportLineCategory($)
   {
     return "skip";
   }
-  if ($Line =~ /: Test (?:failed|succeeded inside todo block): / or
-      $Line =~ /Fatal: test .* does not exist/ or
-      $Line =~ / done \(258\)/ or
-      $Line =~ /: unhandled exception [0-9a-fA-F]{8} (?:at|in) / or
-      $Line =~ /^Unhandled exception: /)
-  {
-    return "error";
-  }
   if ($Line =~ /^[_.a-z0-9-]+:[_a-z0-9]* start / or
       $Line =~ /: this is the last test seen before the exception/)
   {
@@ -736,14 +728,6 @@ sub ParseWineTestReport($$$)
       }
       $Cur->{Rc} = $Rc;
     }
-    elsif (GetReportLineCategory($Line) eq "error")
-    {
-      # This should only happen on very garbled lines, or if there is an
-      # inconsistency between GetReportLineCategory() and
-      # ParseWineTestReport().
-      _AddReportError($LogInfo, $Cur, $LineNo, $Line);
-    }
-
   }
   $Cur->{IsBroken} = 1 if ($LogInfo->{TaskTimedOut});
   _CloseTestUnit($LogInfo, $Cur, 1);