[Merge] lp:~vrruiz/gallery-app/autopilot-app-class into lp:gallery-app
Leo Arias
leo.arias at canonical.com
Fri Jun 27 22:41:33 UTC 2014
> One thing to think about: the UI-driver methods were moved from the case
> classes to GalleryApp. The problem is that some of them use asserts, that make
> sense as part of tests, but not so much a pure UI test. Right now, it's not a
> big deal, because the test_case is a required argument to instantiate
> GalleryApp, but in the medium term, we won't have it.
I think that's just what we want, move the assertions out of the helpers and into the tests.
Many assertions can be replaced by wait_for.
--
https://code.launchpad.net/~vrruiz/gallery-app/autopilot-app-class/+merge/222667
Your team Ubuntu Phablet Team is subscribed to branch lp:gallery-app.
More information about the Ubuntu-reviews
mailing list