[Tools] testbot: Detect the 'invalid unit test' error messages.
Francois Gouget
fgouget at codeweavers.com
Mon Mar 20 17:57:08 CDT 2017
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>
---
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 9ff0d95..dc9e9e6 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 /)
{
--
2.11.0
More information about the wine-patches
mailing list