[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