Closed Bug 1304778 Opened 5 years ago Closed 5 years ago
_dbg-navigation .js | This test exceeded the timeout threshold . It should be rewritten or split up . If that's not possible, use request Longer Timeout(N), but only as a last resort . -
58 bytes, text/x-review-board-request
Let's go with a higher priority than that: one in a million, ASan or Win7 debug will hit this, but, on Linux32 debug it is actually permaorange.
Priority: P3 → --
Turns out permaorange was an exaggeration: I did a try run of something related, with 21 runs of this chunk, and only hit this on 20 of the 21.
Phil is right, this should be a P1. Having said this, this should be a very simple one to fix: - the test does pass - it only takes a few seconds too much on Linux Debug (our slowest env) --> So, simple fix here: increase the timeout for this test using requestLongerTimeout(2) Eddy: can you make sure this bug gets handled by someone working on the debugger?
Priority: -- → P1
We could also skip this whole folder on linux 32 debug, as we do with the current debugger tests: https://dxr.mozilla.org/mozilla-central/source/devtools/client/debugger/test/mochitest/browser.ini#7
What do you think? This might also allow us to proceed with Bug 1304178. Try push: https://treeherder.mozilla.org/#/jobs?repo=try&revision=b9edede90e90704c72683099710fe81c47e6bf0b
Comment on attachment 8798526 [details] Bug 1304778 - Skip new debugger mochitests on linux 32 debug; https://reviewboard.mozilla.org/r/83980/#review82596
Attachment #8798526 - Flags: review?(jlong) → review+
Pushed by firstname.lastname@example.org: https://hg.mozilla.org/integration/fx-team/rev/37f78aca8622 Skip new debugger mochitests on linux 32 debug;r=jlongster
Assignee: nobody → bgrinstead
Status: NEW → ASSIGNED
You need to log in before you can comment on or make changes to this bug.