Closed
Bug 1105772
Opened 10 years ago
Closed 10 years ago
Test failure "Expected to be two visible results in the autocomplete list" in 2 awesomebar tests
Categories
(Mozilla QA Graveyard :: Mozmill Tests, defect)
Tracking
(firefox36 fixed)
RESOLVED
FIXED
Tracking | Status | |
---|---|---|
firefox36 | --- | fixed |
People
(Reporter: AndreeaMatei, Assigned: AndreeaMatei)
References
()
Details
(Keywords: regression, Whiteboard: [mozmill-test-failure])
Attachments
(1 file)
(deleted),
patch
|
andrei
:
review+
|
Details | Diff | Splinter Review |
Started today with latest nightly, on all platforms affected tests:
testCheckItemHighlight.js
testSuggestHistory.js
Affected line: http://hg.mozilla.org/qa/mozmill-tests/file/default/firefox/tests/functional/testAwesomeBar/testCheckItemHighlight.js#l71
I could reproduce locally.
Assignee | ||
Updated•10 years ago
|
Assignee | ||
Comment 1•10 years ago
|
||
So before we had 2 autocomplete items, the Grants page for example in testCheckItemHighlight.js and a search item (at least in nightly from 3 days ago). Now, only the Grants page is present, the assert expects 2 items.
I'll look for the changeset and provide a quick fix.
Assignee | ||
Comment 2•10 years ago
|
||
testSwitchToTab was also affected, different failure message but it was the same issue. We have 1 autocomplete item now and it was expecting more than 1 ("> 1").
http://mozmill-crowd.blargon7.com/#/functional/report/8ed3979adbcd8525966aaf10b001c61c
Assignee: nobody → andreea.matei
Status: NEW → ASSIGNED
Attachment #8529821 -
Flags: review?(mihaela.velimiroviciu)
Attachment #8529821 -
Flags: review?(andrei.eftimie)
Comment 3•10 years ago
|
||
Comment on attachment 8529821 [details] [diff] [review]
v1.patch
Review of attachment 8529821 [details] [diff] [review]:
-----------------------------------------------------------------
Ugh. This changed again?
We need to find the regressor before amending this test since this was changed last month exactly in the opposite direction.
I'm hesitant to change the test before knowing for sure this is a wanted change in Firefox and not a regression.
Attachment #8529821 -
Flags: review?(mihaela.velimiroviciu)
Attachment #8529821 -
Flags: review?(andrei.eftimie)
Attachment #8529821 -
Flags: review-
Comment 4•10 years ago
|
||
I totally agree with Andrei here! Also the test does not say which 2 items it expects exactly. We should start to add better explaining comments so it is easier to understand the behavior later in case of failures and regressions.
Comment 5•10 years ago
|
||
mc pushlog:
https://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=ced1402861b8&tochange=cef590a6f946
inbound pushlog:
https://hg.mozilla.org/integration/mozilla-inbound/pushloghtml?fromchange=52f3f4a553a5&tochange=cef590a6f946
Seems to have come from fx-team, need to test a bit more builds.
Comment 6•10 years ago
|
||
Ugh, and on fx-team this came from mc:
https://hg.mozilla.org/integration/fx-team/pushloghtml?fromchange=6e82407a7c65&tochange=679466398b30
Manual steps:
1. Open a tab
2. Type something in (preferably something that's not into the history). eg: "grants"
Expected results:
3. Autocomplete Popup opens with "Default Action" selected (Google for "grants")
Actual results:
3. Nothing happens
Not sure if I messed something up with the regression (used mozregression). I'll cross check and see if I can find the regressor.
Comment 7•10 years ago
|
||
Ehh, bug 1105456 (and a couple followup bugs along with it).
Indeed, unified autocomplete has been "temporarily disabled".
Blocks: 1105456
Comment 8•10 years ago
|
||
Marco, what's the plan for Unified Complete?
Will it be reenabled on 37 after the merge? Or only when the found regressions are dealt with?
Flags: needinfo?(mak77)
Comment 9•10 years ago
|
||
(In reply to Andrei Eftimie from comment #8)
> Marco, what's the plan for Unified Complete?
> Will it be reenabled on 37 after the merge? Or only when the found
> regressions are dealt with?
it depends on the timing and the patches complexity. if the regressions fixes will be simple and complete in the first couple weeks, we could uplift them to Aurora and re-enable it. Otherwise we will re-enable it only in Nightly.
What we are doing for now is writing tests so that they check the unifiedcomplete pref and take different path based on that, or, they are disabled if unifiedcomplete gets disabled.
Flags: needinfo?(mak77)
Comment 10•10 years ago
|
||
Comment on attachment 8529821 [details] [diff] [review]
v1.patch
Review of attachment 8529821 [details] [diff] [review]:
-----------------------------------------------------------------
Thanks Marko.
I say we go forward with fixing our test for the time being to work without Unified Complete.
We'll amend them again at a later date when it gets reenabled.
Attachment #8529821 -
Flags: review- → review+
Comment 11•10 years ago
|
||
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
•