Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Made public: #209

Conversation

gopito
Copy link

@gopito gopito commented Feb 27, 2018

We have a lot of complex selectors. Lots of "allOf(...)" in our tests on java. So it will be very convenient to have additional public methods with matchers as argument.

Made public:
assertDisplayed(viewMatcher: Matcher)
assertNotDisplayed(viewMatcher: Matcher)
assertNotExist(viewMatcher: Matcher)
clickOn(viewMatcher: Matcher)
scrollTo(matcher: Matcher)
safelyScrollTo(matcher: Matcher)

Added:
assertEffectivelyVisible(viewMatcher: Matcher)

assertDisplayed(viewMatcher: Matcher<View>)
assertNotDisplayed(viewMatcher: Matcher<View>)
assertNotExist(viewMatcher: Matcher<View>)
clickOn(viewMatcher: Matcher<View>)
scrollTo(matcher: Matcher<View>)
safelyScrollTo(matcher: Matcher<View>)

Added:
assertEffectivelyVisible(viewMatcher: Matcher<View>)
@gopito gopito requested a review from rocboronat as a code owner February 27, 2018 12:31
@Sloy
Copy link
Member

Sloy commented Mar 1, 2018

Hi! Thanks for the contribution @gopito.

In the case of the assert methods, do you think the new method added in #198 will be enough for your use case?

It basically adds a public assertAnyView(view, condition) so you can use any matchers you want. Every other assertion method in Barista will be using this method.

@gopito
Copy link
Author

gopito commented Mar 2, 2018

Hi @Sloy.
Thanks for answer.
I'll rewrite my assertions with assertAnyView(view, condition)

@gopito gopito closed this Mar 2, 2018
@Sloy
Copy link
Member

Sloy commented Mar 2, 2018

I'm glad it helps you.

I just triggered the release of v2.4.0 with the new change. It should be available in a few minutes.

Thanks for reporting you case and contributing :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants