Closed
Bug 1038698
Opened 10 years ago
Closed 10 years ago
Intermittent test_a11y_utility_tray_visibility.py TestUtilityTrayVisibilityAccessibility.test_a11y_utility_tray_visibility | AssertionError: False is not true
Categories
(Firefox OS Graveyard :: Gaia::UI Tests, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: cbook, Assigned: yzen)
References
()
Details
(Keywords: intermittent-failure)
Attachments
(1 file)
b2g_macosx64 b2g-inbound opt test gaia-ui-test on 2014-07-15 05:51:41 PDT for push f927b28f8e5b
slave: talos-mtnlion-r5-031
https://tbpl.mozilla.org/php/getParsedLog.php?id=43823004&tree=B2g-Inbound
TEST-UNEXPECTED-FAIL | test_a11y_utility_tray_visibility.py TestUtilityTrayVisibilityAccessibility.test_a11y_utility_tray_visibility | AssertionError: False is not true
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment 2•10 years ago
|
||
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment 15•10 years ago
|
||
I think we'll have to disable this test, it seems very fragile today.
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment 22•10 years ago
|
||
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Assignee | ||
Comment 34•10 years ago
|
||
This is not an intermittent: a regression is caused by bug 1038185 and this commit https://github.com/mozilla-b2g/gaia/commit/8e90912c8d3edacd56fb0eb68b9cee0792a75785
Flags: needinfo?(yzenevich)
Comment 35•10 years ago
|
||
(In reply to Yura Zenevich [:yzen] from comment #34)
> This is not an intermittent: a regression is caused by bug 1038185 and this
> commit
> https://github.com/mozilla-b2g/gaia/commit/
> 8e90912c8d3edacd56fb0eb68b9cee0792a75785
Great! You know what to do! (also, back out my commit that disabled the test)
Assignee | ||
Comment 36•10 years ago
|
||
Re-enabled the tests after reverting the regression: https://github.com/mozilla-b2g/gaia/commit/b3cf3755539cac6006415de4743d0990c9e4e3d4
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment 38•10 years ago
|
||
So I notice the test doesn't actually seem to wait for the utility tray to close after it closes it...(?) I'll investigate on the reopened bug.
Assignee | ||
Comment 39•10 years ago
|
||
(In reply to Chris Lord [:cwiiis] from comment #38)
> So I notice the test doesn't actually seem to wait for the utility tray to
> close after it closes it...(?) I'll investigate on the reopened bug.
Yeah, I would think that with the fix for the original bug the test needs to be updated to account for that.
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Reporter | ||
Comment 66•10 years ago
|
||
this is back and permaorange on b2g-inbound - maybe it needs to be disabled again ?
Flags: needinfo?(zach)
Flags: needinfo?(yzenevich)
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment 70•10 years ago
|
||
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Assignee | ||
Comment 73•10 years ago
|
||
Will provide a fix shortly.
Status: RESOLVED → REOPENED
Flags: needinfo?(yzenevich)
Resolution: FIXED → ---
Assignee | ||
Updated•10 years ago
|
Assignee: nobody → yzenevich
Assignee | ||
Comment 74•10 years ago
|
||
Attachment #8481465 -
Flags: review?(zcampbell)
Comment 75•10 years ago
|
||
Comment on attachment 8481465 [details]
Github pull request.
I don't have time to run it today so I can give it f+ now.
Attachment #8481465 -
Flags: review?(zcampbell) → feedback+
Assignee | ||
Updated•10 years ago
|
Attachment #8481465 -
Flags: review?(jgriffin)
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment 77•10 years ago
|
||
Comment on attachment 8481465 [details]
Github pull request.
The patch looks totally reasonable, and it works ok on try. I think the concern is that it might break something in the on-device automation, which I'm not in a position to test. So, I'll give this an r+, but with the caveat that you might want to wait until Zac has a chance to confirm it doesn't break on-device tests, unless this is urgent enough to trump that risk.
Attachment #8481465 -
Flags: review?(jgriffin) → review+
Comment 78•10 years ago
|
||
Zac, what's your take on comment 77? This is a contributing factor to Gaia being closed on the weekend before FL.
Flags: needinfo?(zach) → needinfo?(zcampbell)
Comment 79•10 years ago
|
||
I will give it an r? today and let y'all know.
Comment 80•10 years ago
|
||
There are no negative effects when running this on device.
Flags: needinfo?(zcampbell)
Assignee | ||
Updated•10 years ago
|
Keywords: checkin-needed
Assignee | ||
Comment 81•10 years ago
|
||
Status: REOPENED → RESOLVED
Closed: 10 years ago → 10 years ago
Keywords: intermittent-failure
Resolution: --- → FIXED
Assignee | ||
Updated•10 years ago
|
Keywords: checkin-needed → intermittent-failure
Comment 82•10 years ago
|
||
Why was that merged with failures on try that were not even retriggered?
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
You need to log in
before you can comment on or make changes to this bug.
Description
•