Update Mozmill tests to match the new interface of waitFor and assert

RESOLVED INVALID

Status

Mozilla QA
Mozmill Tests
RESOLVED INVALID
8 years ago
8 years ago

People

(Reporter: whimboo, Assigned: whimboo)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

8 years ago
With the fix on bug 604545 we have to update some of our tests to match the new API. Patch upcoming.
(Assignee)

Comment 1

8 years ago
Created attachment 483429 [details] [diff] [review]
Patch v1
Attachment #483429 - Flags: review?(gmealer)
(Assignee)

Comment 2

8 years ago
This patch should not be checked-in before Mozmill 1.5.1 is in use.
Comment on attachment 483429 [details] [diff] [review]
Patch v1

The patch would look fine, but you had me revert all these calls to waitForEval() (Bug 604743).  My patches for that make this patch invalid.
Attachment #483429 - Flags: review?(gmealer) → review-
(Assignee)

Comment 4

8 years ago
This bug is useless now with bug 604743 implemented and fixed. Sorry for taking up your time on that. Marking as invalid.
Status: ASSIGNED → RESOLVED
Last Resolved: 8 years ago
Resolution: --- → INVALID
(Assignee)

Updated

8 years ago
Whiteboard: [needs-mozmill-1.5.1]
(Assignee)

Comment 5

8 years ago
Move of Mozmill Test related project bugs to newly created components. You can
filter out those emails by using "Mozmill-Tests-to-MozillaQA" as criteria.
Component: Mozmill Tests → Mozmill Tests
Product: Testing → Mozilla QA
Version: Trunk → unspecified
You need to log in before you can comment on or make changes to this bug.