Code review comment for lp:~carla-sella/ubuntu-clock-app/add-alarm-test
Revision history for this message
| Carla Sella (carla-sella) wrote | # |
© 2004
Canonical Ltd.
•
Terms of use
•
Data privacy
•
Contact Launchpad Support
•
Blog
•
Careers
•
System status
•
a60fb26
(Get the code!)
> > > 640 +class Page11(Page):
> > > 641 +
> > > 642 + """Autopilot helper for the Alarm page."""
> > >
> > > Why is this Page11 and not AlarmPage?
> >
> > Because in autopilot vis I see Page11:
> > https:/
> >
> > Do you want me to change it in AlarmPage anyway ?
>
> Lets give it a try, and see if it recognizes AlarmPage instead of Page11. If
> it doesn't work, we can stick to Page11. The reason being in the SDK I saw
> Page10, Page11 etc. and I don't want this test to break everytime they make a
> change in the SDK.
Nope, if I use AlarmPage I get errors, I've tried various ways. Maybe it will work if I change the code, but not sure you want me to change quite abit of it.