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="…").

review: Needs Fixing

« Back to merge proposal