Closed
Bug 1074076
Opened 10 years ago
Closed 10 years ago
Multiple test failures in relation to the AwesomeBar
Categories
(Mozilla QA Graveyard :: Mozmill Tests, defect, P1)
Mozilla QA Graveyard
Mozmill Tests
Tracking
(firefox32 unaffected, firefox33 unaffected, firefox34 unaffected, firefox35 unaffected, firefox36 fixed, firefox-esr31 unaffected)
RESOLVED
FIXED
Tracking | Status | |
---|---|---|
firefox32 | --- | unaffected |
firefox33 | --- | unaffected |
firefox34 | --- | unaffected |
firefox35 | --- | unaffected |
firefox36 | --- | fixed |
firefox-esr31 | --- | unaffected |
People
(Reporter: andrei, Assigned: danisielm)
References
Details
(Keywords: regression, Whiteboard: [mozmill-test-failure])
Attachments
(2 files)
1.13 KB,
patch
|
AndreeaMatei
:
review+
|
Details | Diff | Splinter Review |
7.67 KB,
patch
|
andrei
:
review+
andrei
:
checkin+
|
Details | Diff | Splinter Review |
The following 4 tests are affected and constantly failing:
> failed /testAwesomeBar/testAccessLocationBar.js testAccessLocationBarHistory
> Element.waitForElement(): Element 'ID: mozilla_logo' has been found show more
> failed /testAwesomeBar/testCheckItemHighlight.js testCheckItemHighlight
> description is undefined show more
> failed /testAwesomeBar/testSuggestBookmarks.js testStarInAutocomplete
> The auto-complete result is a bookmark - 'action searchengine' should contain 'bookmark' show more
> failed /testAwesomeBar/testSuggestHistory.js testSuggestHistoryAndBookmarks
> Expected to be one visible result in the autocomplete list - '2' should equal '1' show more
Reporter | ||
Comment 1•10 years ago
|
||
A report: http://mozmill-daily.blargon7.com/#/functional/report/f11964362bbc85ebfbc5b36de794fa1e
Reporter | ||
Comment 2•10 years ago
|
||
Attachment #8496708 -
Flags: review?(andreea.matei)
Reporter | ||
Updated•10 years ago
|
status-firefox32:
--- → unaffected
status-firefox33:
--- → unaffected
status-firefox34:
--- → unaffected
status-firefox-esr31:
--- → unaffected
Assignee | ||
Comment 3•10 years ago
|
||
This started on 27.09 and here is the Regression Range from the day before to the day it started: https://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=9e3d649b80a2&tochange=818f353b7aa6 I see 3 candidates here: Bug 1067896, Bug 1067888 and Bug 1067899. Not sure which of them is the cause, but we'll find out once we have the regression range ready.
Comment 4•10 years ago
|
||
Comment on attachment 8496708 [details] [diff] [review] skip.patch Review of attachment 8496708 [details] [diff] [review]: ----------------------------------------------------------------- Looks good, lets land this.
Attachment #8496708 -
Flags: review?(andreea.matei) → review+
Reporter | ||
Comment 5•10 years ago
|
||
Disabled: https://hg.mozilla.org/qa/mozmill-tests/rev/da7bb338173c (default)
Assignee | ||
Comment 6•10 years ago
|
||
testSwitchTab is also failing, see: http://mozmill-daily.blargon7.com/#/functional/report/f11964362bbc85ebfbc5b36de7aaccaa http://mozmill-daily.blargon7.com/#/functional/report/f11964362bbc85ebfbc5b36de7962273 http://mozmill-daily.blargon7.com/#/functional/report/f11964362bbc85ebfbc5b36de7b7eea1
Reporter | ||
Comment 7•10 years ago
|
||
(In reply to daniel.gherasim from comment #6) > testSwitchTab is also failing, see: > http://mozmill-daily.blargon7.com/#/functional/report/ > f11964362bbc85ebfbc5b36de7aaccaa > http://mozmill-daily.blargon7.com/#/functional/report/ > f11964362bbc85ebfbc5b36de7962273 > http://mozmill-daily.blargon7.com/#/functional/report/ > f11964362bbc85ebfbc5b36de7b7eea1 Interesting, that seems to be failing only intermittently. Keep an eye out for those if they keep failing even with the other tests disabled.
Reporter | ||
Comment 8•10 years ago
|
||
I've actually reproduced the switchToTab failure locally in a Win VM: http://mozmill-crowd.blargon7.com/#/functional/report/f11964362bbc85ebfbc5b36de7c3fd31
Assignee | ||
Comment 9•10 years ago
|
||
Pushlog from fx-team: https://hg.mozilla.org/integration/fx-team/pushloghtml?fromchange=06a1582f7d5f&tochange=ba0d3dd1be2c So bug 1067888 is the reason here. Actually a pre-configured item representing the search engine appears always as the first element of autocomplete results. In our tests we press the "DOWN" button to select the first element, we may have to press twice now to select the correct one, and also update the checks to check for the 2nd element in the autocomplete results popup. I will come with a fix soon.
Assignee: nobody → daniel.gherasim
Blocks: 1067888
Status: NEW → ASSIGNED
Keywords: regressionwindow-wanted
Assignee | ||
Comment 10•10 years ago
|
||
And here is the fix patch.
Attachment #8497387 -
Flags: review?(andrei.eftimie)
Attachment #8497387 -
Flags: review?(andreea.matei)
Reporter | ||
Comment 11•10 years ago
|
||
Comment on attachment 8497387 [details] [diff] [review] fix.patch Review of attachment 8497387 [details] [diff] [review]: ----------------------------------------------------------------- https://hg.mozilla.org/qa/mozmill-tests/rev/dc419f74a94f (default)
Attachment #8497387 -
Flags: review?(andrei.eftimie)
Attachment #8497387 -
Flags: review?(andreea.matei)
Attachment #8497387 -
Flags: review+
Attachment #8497387 -
Flags: checkin+
Reporter | ||
Comment 12•10 years ago
|
||
I was looking if there's a better way to handle this, but the tests changed in the original change from mc have the same thing (push twice the VK_DOWN etc). So this should be good enough. Thanks.
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Comment 13•10 years ago
|
||
Thank you Daniel for this quick investigation and fix!
Reporter | ||
Comment 14•10 years ago
|
||
UnifiedComplete has been disabled in Aurora 35, see bug 1081297. Backed out the cahnges: https://hg.mozilla.org/qa/mozmill-tests/rev/b41d38748b02 (mozilla-aurora) https://hg.mozilla.org/qa/mozmill-tests/rev/0349b31dc9d1 (mozilla-aurora) Report: http://mozmill-crowd.blargon7.com/#/functional/report/5560c4beef382ba39c2500563521b524
Updated•5 years ago
|
Product: Mozilla QA → Mozilla QA Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•