Update failing Mozmill tests

RESOLVED FIXED

Status

RESOLVED FIXED
9 years ago
9 years ago

People

(Reporter: merike, Assigned: merike)

Tracking

Details

Attachments

(1 attachment)

(Assignee)

Description

9 years ago
Created attachment 425050 [details] [diff] [review]
Mozmill tests' updates

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 1

9 years ago
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+

Comment 2

9 years ago
Pushed as: changeset 1bca8ebb93f1
Status: ASSIGNED → RESOLVED
Last Resolved: 9 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.