fix scroll

[paweltomaszewskisaucelabs]

Because of nested scrollable element time to time scroll down (swipe up) will end up with going into wrong direction. I did a fix to make scroll works for native full page screenshot test in visual e2e repo.

[diegoperini]

We can merge this if this is urgent but I'd rather not rely upon presence of espresso classes as a way to enable a feature or not. Mocks of those classes can go into an app even outside of a testing context.

An alternative approach is we can define a property to enable/disable this explicitly. The property can be false by default and test code can have access to the Activity reference and call the setter to set it to true.

[shahrukhamd]

I would like to understand if this check is actually needed. I mean, why can't we just enable nested scrolling android:nestedScrollingEnabled="true"

[diegoperini]

This is a better approach indeed. If it works, please ignore my alternative solution (accessing activity reference in test code) above.

[paweltomaszewskisaucelabs]

I would like to understand if this check is actually needed. I mean, why can't we just enable nested scrolling android:nestedScrollingEnabled="true"

As I mentioned in description, scrolling down time to time ends up with scrolling up which breaks taking full page screenshot in our tests. The issue is fixed after turning off nestedScrollingEnabled

[shahrukhamd]

Thanks!

[diegoperini]

Then please create a function in the fragment that does this:

binding.productRV.setNestedScrollingEnabled(true);

and call that function explicitly in the test class so that you can safely remove isRunningEspressoTest() from the fragment entirely.