Code review comment for lp:~om26er/ubuntu-calendar-app/autopilot_tests_bootstrap

Revision history for this message
Olivier Tilloy (osomon) wrote :

> > 133 + def get_new_event_button(self):
> > 134 + return self.app.select_many("Button", visible=True)[1]
> > 135 +
> > 136 + def get_timeline_button(self):
> > 137 + return self.app.select_many("Button", visible=True)[2]
> >
> > That’s not robust, instead you should give an objectName to those buttons
> and
> > get a reference to them using self.app.select_single("Button",
> > objectName="…").
>
> That's the problem, I have not been able to assign objectNames to those
> buttons, it seems that doesn't work there.

What’s the error you’re getting? If it’s an issue in autopilot itself, we should make sure it’s reported and fixed, not just worked around silently.

« Back to merge proposal