Closed Bug 544082 Opened 14 years ago Closed 14 years ago

Update failing Mozmill tests

Categories

(Calendar :: General, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: merike, Assigned: merike)

Details

Attachments

(1 file)

Attached patch Mozmill tests' updates — — Splinter Review
Fixing:
* using formatTime for times, since 9am can be displayed 9:00 or 09:00
* doubleclick for new calendar can't be at 0,0 since it opens home calendar properties instead of new calendar properties
* lookup path updates where needed
* controller.mouseover -> controller.mouseOver
* today-pane isn't the only child using collapsed=true

Improvements:
* bad indentation in switchToView()
* shortening sleeps in back() and forward()
* removing unnecessary imports
* use createCalendar() and deleteCalendar() to not rely on Mozmill calendar created before running tests
Attachment #425050 - Flags: review?(ctalbert)
Assignee: nobody → gasell+mozilla
Status: NEW → ASSIGNED
Comment on attachment 425050 [details] [diff] [review]
Mozmill tests' updates

This looks great.  Thanks for the thorough work, sorry the review took so long.
Attachment #425050 - Flags: review?(ctalbert) → review+
Pushed as: changeset 1bca8ebb93f1
Status: ASSIGNED → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: