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

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

ensure_toolbar_visible() works if the toolbar is initially hidden, but will it work if it’s already shown? If not, either the method should be renamed ('ensure' implies that the toolbar will be shown after executing it, regardless of the initial state), or it should account for all possible initial states.

« Back to merge proposal