Closed
Bug 1230042
Opened 10 years ago
Closed 10 years ago
Intermittent TEST-UNEXPECTED-TIMEOUT | /_mozilla/service-workers/service-worker/clients-matchall-include-uncontrolled.https.html | Verify matchAll() respect includeUncontrolled - Test timed out
Categories
(Testing :: web-platform-tests, defect)
Testing
web-platform-tests
Tracking
(firefox45 fixed, firefox46 fixed)
RESOLVED
FIXED
mozilla46
People
(Reporter: philor, Assigned: ehsan.akhgari)
References
Details
(Keywords: intermittent-failure)
Attachments
(2 files)
|
774 bytes,
patch
|
bkelly
:
review+
|
Details | Diff | Splinter Review |
|
2.48 KB,
patch
|
bkelly
:
review+
|
Details | Diff | Splinter Review |
| Comment hidden (Intermittent Failures Robot) |
| Comment hidden (Intermittent Failures Robot) |
Comment 3•10 years ago
|
||
bkelly: this seems to be a relatively common orange. Might just need a long timeout or something, but you seem like the right person to look at it.
Comment 4•10 years ago
|
||
Error message:
TEST-UNEXPECTED-ERROR | /_mozilla/service-workers/service-worker/clients-matchall-include-uncontrolled.https.html | Error: assert_equals: expected 4 but got 5
I wonder if there is an extra window from a previous tests that is being picked up. It could be racing with GC to remove the window or something.
| Comment hidden (Intermittent Failures Robot) |
Comment 6•10 years ago
|
||
Sucks that this mainly triggers on osx 10.10 which doesn't seem to run in try builds.
| Reporter | ||
Comment 7•10 years ago
|
||
It does, you just have to beg for it. try: -b o -p macosx64 -u web-platform-tests-4[10.10] -t none (or everything from checking every checkbox in the "Restrict tests to platform(s)" section of trychooser if you want all platforms including 10.10)
| Reporter | ||
Comment 8•10 years ago
|
||
Or in theory you could add them to an existing try push that has a completed Mac build, with either the treeherder UI from the downarrow menu at the top right of the push, "Add new Jobs", or with http://try-extender.herokuapp.com/, though neither seems to actually work, or with https://mozilla-ci-tools.readthedocs.org/en/latest/scripts.html which will work but requires a fair amount of effort.
Comment 9•10 years ago
|
||
Ehsan asked me to disable this test so he can investigate it next week.
Assignee: bkelly → ehsan
Updated•10 years ago
|
Keywords: leave-open
Comment 11•10 years ago
|
||
| Reporter | ||
Comment 12•10 years ago
|
||
| bugherder | ||
| Comment hidden (Intermittent Failures Robot) |
| Assignee | ||
Comment 14•10 years ago
|
||
Attachment #8698447 -
Flags: review?(bkelly)
Updated•10 years ago
|
Attachment #8698447 -
Flags: review?(bkelly) → review+
| Assignee | ||
Updated•10 years ago
|
Keywords: leave-open
Comment 15•10 years ago
|
||
Comment 16•10 years ago
|
||
| bugherder | ||
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
status-firefox46:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla46
Comment 17•10 years ago
|
||
| bugherder uplift | ||
You need to log in
before you can comment on or make changes to this bug.
Description
•