Francois Gouget : testbot/WineRunTask: Treat a missing test summary line as an error.

[Alexandre Julliard]

Module: tools
Branch: master
Commit: e595de2a5f3d7b486839e159d982effcdae15059
URL:    http://source.winehq.org/git/tools.git/?a=commit;h=e595de2a5f3d7b486839e159d982effcdae15059

Author: Francois Gouget <fgouget at codeweavers.com>
Date:   Tue Oct 23 18:23:43 2012 +0200

testbot/WineRunTask: Treat a missing test summary line as an error.

---

 testbot/bin/WineRunTask.pl |    8 +++-----
 1 files changed, 3 insertions(+), 5 deletions(-)

diff --git a/testbot/bin/WineRunTask.pl b/testbot/bin/WineRunTask.pl
index 6f63640..ca72fc7 100755
--- a/testbot/bin/WineRunTask.pl
+++ b/testbot/bin/WineRunTask.pl
@@ -335,11 +335,7 @@ $Task->Status("completed");
 $Task->ChildPid(undef);
 $Task->Ended(time);
 my $TestFailures = CountFailures($FullLogFileName);
-if (defined($TestFailures))
-{
-  $Task->TestFailures($TestFailures);
-}
-else
+if (!defined $TestFailures)
 {
   my $OldUMask = umask(002);
   if (open ERRFILE, ">>$FullErrFileName")
@@ -348,7 +344,9 @@ else
     close ERRFILE;
   }
   umask($OldUMask);
+  $TestFailures = 1;
 }
+$Task->TestFailures($TestFailures);
 $Task->Save();
 $Job->UpdateStatus();
 if ($Task->VM->Role ne "base")