This ensures that the report is flagged as failed even if the timeout happened at the task level.
Signed-off-by: Francois Gouget fgouget@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"); + } } }