894911 - Take a screenshot when a mochitest-metro-chrome test fails

Status: RESOLVED WONTFIX

(Firefox for Metro Graveyard :: Tests, defect)

Description

[Jim Mathies [:jimm]]

Attachment: patch

I think this would be useful in tracking down intermittent failures.

Comment 1

[Jim Mathies [:jimm]]

Comment on attachment 777128 [details] [diff] [review]
patch

sorry, wrong patch file.

Comment 2

[Jim Mathies [:jimm]]

Attachment: patch

Comment 3

[Sam Foster [:sfoster] (he/him)]

Comment on attachment 777129 [details] [diff] [review]
patch

Review of attachment 777129 [details] [diff] [review]:
-----------------------------------------------------------------

I'll confess to being slightly dubious about the utility of this, but there's one way to find out; lets land it and see how it plays. 
Personally, I would usually not want this behavior when testing locally as it fills the terminal buffer and makes it harder to quickly scan over the output (though I see the value of leaving it on by default so you can catch intermittent failures as they happen). I could do that with my own patch to head.js, but some kind of flag or pref would be better

Comment 4

[Jim Mathies [:jimm]]

This would be useful in diagnosing intermittent test failures like bug 896082 but I also don't want to pollute local builds. Need to find a way to only turn this on in automated runs.