-
-
Notifications
You must be signed in to change notification settings - Fork 745
espresso fixes and replacing manual smoke test #6150
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
base: main
Are you sure you want to change the base?
Conversation
- adds isItemPresent before scrolling with log
- code fixes, adds new test functions - github workflow updates to test on different api level and device
- adds two new function onParentViewWithChildIdAndText and selectTabWithText - code fixes
| fun isItemPresent( | ||
| recyclerViewId: Int = R.id.feed_view, | ||
| title: String, | ||
| textViewId: Int = R.id.view_card_header_title | ||
| ): Boolean { | ||
| return try { | ||
| onView(withId(recyclerViewId)) | ||
| .check(matches(hasDescendant(allOf( | ||
| withId(textViewId), | ||
| withText(title) | ||
| )))) | ||
| true | ||
| } catch (_: NoMatchingViewException) { | ||
| false | ||
| } catch (_: AssertionFailedError) { | ||
| false | ||
| } | ||
| } |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is this used?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this was used before but its not used now, there are lot of functions which are not used or i added for testing. I will remove them and refactor more after i finish fixing the tests.
| }) | ||
| } | ||
|
|
||
| private fun verifyItemAtPosition( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Same for this, is this still being used? and also this findItemPosition
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this was used before but its not used now, there are lot of functions which are not used or i added for testing. I will remove them and refactor more after i finish fixing the tests.
What does this do?
Phabricator:
https://phabricator.wikimedia.org/T378649