Code review comment for lp://qastaging/~vrruiz/gallery-app/autopilot-app-class

Revision history for this message
VĂ­ctor R. Ruiz (vrruiz) wrote :

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.

« Back to merge proposal