Update tests sometimes fail with 'New wizard page has been selected - got finishedBackground, expected finished'

RESOLVED FIXED

Status

Mozilla QA
Mozmill Tests
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: whimboo, Assigned: whimboo)

Tracking

unspecified

Firefox Tracking Flags

(firefox13 fixed, firefox14 fixed, firefox15 fixed, firefox16 fixed, firefox-esr10 fixed)

Details

(Whiteboard: [mozmill-test-failure][qa-], URL)

Attachments

(2 attachments)

(Assignee)

Description

5 years ago
As the attached Mozmill report shows we are waiting for 'finishedBackground' under some conditions. Given that it is not always failing I'm not sure when we will see the page but in our case it shouldn't matter. We have to wait for 'finished' and 'finishedBackground'.

New wizard page has been selected - got finishedBackground, expected finished
(Assignee)

Comment 1

5 years ago
Created attachment 631889 [details] [diff] [review]
Patch v1
Attachment #631889 - Flags: review?(anthony.s.hughes)
(Assignee)

Updated

5 years ago
status-firefox15: --- → affected
status-firefox16: --- → affected
Comment on attachment 631889 [details] [diff] [review]
Patch v1

Review of attachment 631889 [details] [diff] [review]:
-----------------------------------------------------------------

Looks good to me, please land.
Attachment #631889 - Flags: review?(anthony.s.hughes) → review+
(Assignee)

Comment 3

5 years ago
Pushed:
http://hg.mozilla.org/qa/mozmill-tests/rev/0672153467e8 (master)

I will land on Aurora if we don't regress anything in the next two days or so.
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
status-firefox16: affected → fixed
Resolution: --- → FIXED
(Assignee)

Comment 4

5 years ago
Looks fine on default. Pushed to aurora:
http://hg.mozilla.org/qa/mozmill-tests/rev/8e5c303efb7c
status-firefox14: --- → unaffected
status-firefox15: affected → fixed
(Assignee)

Comment 5

5 years ago
I would say we get modified patches landed on other branches to make them more stable. In those cases we should just leave out the backgroundupdate cases.
status-firefox-esr10: --- → affected
status-firefox13: --- → affected
status-firefox14: unaffected → affected
(Assignee)

Comment 6

5 years ago
Created attachment 637406 [details] [diff] [review]
Backport v1 (beta, release, esr10)

This patch contains only the fixes necessary for the beta, release, and esr10 branch. Also it fixes a missing change from bug 759333.
Attachment #637406 - Flags: review?(dave.hunt)

Updated

5 years ago
Attachment #637406 - Flags: review?(dave.hunt) → review+
(Assignee)

Comment 7

5 years ago
Pushed:
http://hg.mozilla.org/qa/mozmill-tests/rev/0afe8ec88b01 (beta)
http://hg.mozilla.org/qa/mozmill-tests/rev/3e949991e200 (release)
http://hg.mozilla.org/qa/mozmill-tests/rev/b48c45314461 (esr10)
status-firefox-esr10: affected → fixed
status-firefox13: affected → fixed
status-firefox14: affected → fixed
Whiteboard: [mozmill-test-failure] → [mozmill-test-failure][qa-]
You need to log in before you can comment on or make changes to this bug.