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