Update testGoButton.js to match latest version of Litmus tests

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

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

Details

Attachments

(2 attachments)

(Assignee)

Description

5 years ago
The Litmus test has been changed a bit since we have written the test:
https://litmus.mozilla.org/show_test.cgi?id=15423

So we should update the test and get rid of the second test function inside this module.
(Assignee)

Comment 1

5 years ago
Created attachment 645658 [details] [diff] [review]
Patch v1
Attachment #645658 - Flags: review?(dave.hunt)
Comment on attachment 645658 [details] [diff] [review]
Patch v1

Looks good to me. Landed as:
http://hg.mozilla.org/qa/mozmill-tests/rev/24b7dcb70e88 (default)
Attachment #645658 - Flags: review?(dave.hunt) → review+

Updated

5 years ago
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
status-firefox-esr10: --- → affected
status-firefox14: --- → affected
status-firefox15: --- → affected
status-firefox16: --- → affected
status-firefox17: --- → fixed
Resolution: --- → FIXED
(Assignee)

Comment 3

5 years ago
No regressions spotted in todays testrun, so backported the patch to older branches:

http://hg.mozilla.org/qa/mozmill-tests/rev/7c8b8c9aace7 (aurora)
http://hg.mozilla.org/qa/mozmill-tests/rev/721624e2a28a (beta)
http://hg.mozilla.org/qa/mozmill-tests/rev/4617c9dc8ef0 (release)

For esr10 we need a separate patch.
status-firefox14: affected → fixed
status-firefox15: affected → fixed
status-firefox16: affected → fixed
(Assignee)

Comment 4

5 years ago
Created attachment 645787 [details] [diff] [review]
Patch (backport esr10)
Attachment #645787 - Flags: review?(dave.hunt)

Updated

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

Comment 5

5 years ago
Also landed on esr10:
http://hg.mozilla.org/qa/mozmill-tests/rev/dd1638bc5468
status-firefox-esr10: affected → fixed
You need to log in before you can comment on or make changes to this bug.