[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