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

Revision history for this message
Omer Akram (om26er) 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.

I don't think the issue is with autopilot, I am trying this patch http://paste.ubuntu.com/5713531/ to set objectName but it seems I cannot find any thing in autopilot vis with that objectName.

« Back to merge proposal