Change the ViewPager methods to apply only on displayed widgets

[rocboronat]

In one case, we have a ViewPager inside a ViewPager. Then, the ViewPager that we want to move is repeated over lots of times inside another ViewPager. That makes Espresso to launch a "multiple ViewPagers match that ID". But well, obviously, we want to interact with the ViewPager that is visible.

So:

onView(allOf(ViewMatchers.withId(R.id.the_id), ViewMatchers.isDisplayed())).perform( new ViewAction[] { ViewActions.swipeLeft() });

[rocboronat]

@Sloy I'll love to have feedback on this idea :·D

[Sloy]

Wut? A ViewPager inside a ViewPager both with the same id? That sounds wrong :/

[rocboronat]

Nope, a ViewPager that contains a page that has a ViewPager. So, the ID of the second ViewPager will appear in every page of the first ViewPager, so it will be repeated in the hierarchy tree.

[rocboronat]

Something like this, where the car image is inside a small ViewPager that show lots of pictures of that car, while you can also change the page to take a look to another car:

If you want to swipe the images' ViewPager, you has match by ID and also look only for displayed ViewPagers.

[Sloy]

Now I get it.
I don't have a strong opinion about this...
Seems like adding the isDisplayed is a hack for a specific case. And not only for the ViewPager methods. What about the buttons? Or any other view in that page?

[rocboronat]

Yes, totally. It could apply to lots of things. But, excepting in the Buttons where we do the magic of scrolling until needed, when are you interested in tapping a button that is not visible?

Something to thing about it. No hurries, by the way.

[rocboronat]

Done at #51 !