[tools] testbot/LogUtils: Add a timeout error if the task timed out.

Francois Gouget fgouget at codeweavers.com
Thu Apr 8 05:32:21 CDT 2021


This ensures that the report is flagged as failed even if the timeout
happened at the task level.

Signed-off-by: Francois Gouget <fgouget at codeweavers.com>
---
So far if a Wine task ran the tests 3 times (e.g. in different bitnesses 
or locales), often only the first two runs would get an "xxx: Timeout" 
error. One would then have to deduce that the third run timed out too 
from the testbot.log's "The task timed out" error. Now all three runs 
get an "xxx: Timeout" error.

This also helps TestWTBS check the timeouts.
---
 testbot/lib/WineTestBot/LogUtils.pm | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/testbot/lib/WineTestBot/LogUtils.pm b/testbot/lib/WineTestBot/LogUtils.pm
index 5d2dc33a5..0b2edf735 100644
--- a/testbot/lib/WineTestBot/LogUtils.pm
+++ b/testbot/lib/WineTestBot/LogUtils.pm
@@ -455,6 +455,10 @@ sub _CloseTestUnit($$$)
       {
         _AddExtra($LogInfo, "The report seems to have been truncated");
       }
+      elsif ($Last and $LogInfo->{TaskTimedOut})
+      {
+        _AddExtra($LogInfo, "$Cur->{Unit}: Timeout");
+      }
     }
   }
 
-- 
2.20.1



More information about the wine-devel mailing list