Code review comment for lp:~canonical-platform-qa/ubuntu-system-tests/pull_to_refresh_video_scope

Revision history for this message
Santiago Baldassin (sbaldassin) wrote :

> > Interestingly the test is still failing and I don't seem to be able to
> > reproduce it here. Will probably need to replicate a test run exactly like
> the
> > lab.
>
> I would recommend not using a pull action to refresh. - We've had many
> problems with this in the past as it is not reliable. There is already a
> refresh() method in the base class which uses url dispatcher to navigate back
> to the current scope and it causes a refresh without any UI interaction. So I
> would try using that first.

I'm sorry but I disagree with Richard on this one. I think that the use of the pull_to_refresh method is actually a good solution. That's what the end user would do if the local videos/photos are not displayed. Using the url dispatcher is not really a navigation option for the end users so I would avoid it. If the pull to refresh method is not reliable, me can work and making it more robust

« Back to merge proposal