736344 - automation.py.in: allow 5 (instead of 1) screenshots to be taken

Status: RESOLVED WONTFIX

(Testing :: General, enhancement)

Description

[Serge Gautherie (:sgautherie)]

Bug 414049 limited taking screenshot to the first timeout.
Bug 637103 added a reminder on additional timeouts.

Yet, see bug 637103 comment 13 as an example:
WINNT 5.2 comm-central-trunk debug test mochitest-other on 2012/03/09 03:33:05
{
[...]
38246 ERROR TEST-UNEXPECTED-FAIL | (SimpleTest/TestRunner.js) | 4 test timeouts, giving up.
}

I'm not sure it would be fine to unlimit the number of screenshots,
but it should be safe to increase the limit to 4, to match TestRunner.js.
It would help when the timeouts are unrelated.

***

While here, I would like the two info() added in bug 637103 to have a distinctive string added, fwiw.

Then it would probably not hurt to allow 5 (= 4 tests + 1 kill) screenshots, just in case.

Comment 1

[(not currently active) Ted Mielczarek]

The useful code here lives in mozbase now.