From: Francois Gouget Subject: [Tools] testbot: Detect the 'invalid unit test' error messages. Message-Id: Date: Mon, 20 Mar 2017 23:57:08 +0100 (CET) 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 --- 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