Closed Bug 777242 Opened 9 years ago Closed 9 years ago

Rename utils.assertElementVisible() to utils.isDisplayed()

Categories

(Mozilla QA Graveyard :: Mozmill Tests, defect)

defect
Not set
normal

Tracking

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

RESOLVED FIXED
Tracking Status
firefox14 --- fixed
firefox15 --- fixed
firefox16 --- fixed
firefox17 --- fixed
firefox-esr10 --- fixed

People

(Reporter: whimboo, Assigned: whimboo)

Details

Attachments

(2 files)

There is no need to assert in this method. We now have the assertions module which gives us way nicer feedback. Also with this change tests could check first if an element is visible before clicking on it.
Attached patch Patch v1Splinter Review
Actually this patch depends on bug 777237.
Attachment #645662 - Flags: review?(dave.hunt)
Comment on attachment 645662 [details] [diff] [review]
Patch v1

I like it! Landed as:
http://hg.mozilla.org/qa/mozmill-tests/rev/2934fad7bfa2 (default)
Attachment #645662 - Flags: review?(dave.hunt) → review+
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Attachment #646090 - Flags: review?(dave.hunt)
Attachment #646090 - Flags: review?(dave.hunt) → review+
Product: Mozilla QA → Mozilla QA Graveyard
You need to log in before you can comment on or make changes to this bug.