Francois Gouget : testbot: Identify testbot.log information and error messages.

[Alexandre Julliard]

Module: tools
Branch: master
Commit: 9d5bd059a2e4fa4597c3f7ce3c8104f5ba64fd29
URL:    https://source.winehq.org/git/tools.git/?a=commit;h=9d5bd059a2e4fa4597c3f7ce...

Author: Francois Gouget <fgouget(a)codeweavers.com>
Date:   Wed Feb 12 16:02:41 2020 +0100

testbot: Identify testbot.log information and error messages.

The testbot.log error messages have no common prefix such as 'error:'
which makes them hard to identify.
Expand GetLogLineCategory() to identify the most important ones.

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

---

 testbot/lib/WineTestBot/LogUtils.pm | 13 +++++++++++--
 1 file changed, 11 insertions(+), 2 deletions(-)

diff --git a/testbot/lib/WineTestBot/LogUtils.pm b/testbot/lib/WineTestBot/LogUtils.pm
index a5c6217..2104ffa 100644
--- a/testbot/lib/WineTestBot/LogUtils.pm
+++ b/testbot/lib/WineTestBot/LogUtils.pm
@@ -86,7 +86,11 @@ sub GetLogLineCategory($)
       $Line =~ /^\+ \S/ or
       $Line =~ /^LANG=/ or
       $Line =~ /^Running (?:the tests|WineTest) / or
-      $Line =~ /^Task: (?:ok|tests|Updated)/)
+      $Line =~ /^Task: (?:ok|tests|Updated)/ or
+      # testbot.log information messages
+      $Line =~ /^----- Run / or
+      $Line =~ /^The test VM has crashed/ or # Context for an error message
+      $Line =~ /^The previous \d+ run\(s\) terminated abnormally/)
   {
     return "info";
   }
@@ -101,7 +105,12 @@ sub GetLogLineCategory($)
       $Line =~ /^Makefile:[0-9]+: recipe for target .* failed$/ or
       $Line =~ /^Task: / or
       # Typical perl errors
-      _IsPerlError($Line))
+      _IsPerlError($Line) or
+      # The main testbot.log errors
+      $Line =~ /^An error occurred while / or
+      $Line =~ /^Could not create / or
+      $Line =~ /^Giving up after \d+ run\(s\)$/ or
+      $Line =~ /^The (?:build|task) timed out$/)
   {
     return "error";
   }