-
-
Notifications
You must be signed in to change notification settings - Fork 757
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
#995 add android viewtag for espresso driver #996
Conversation
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.
AndroidDriver
needs to implement FindsByAndroidViewTag
return super.findElement(context); | ||
} | ||
|
||
throw formIllegalArgumentException(contextClass, FindsByAccessibilityId.class, |
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.
FindsByAndroidViewTag
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.
Done
/** | ||
* This locator strategy is available in Espresso Driver mode. | ||
* @since Appium 1.8.2 beta | ||
* @param tag is an an view tag string |
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.
an
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.
Done
import io.appium.java_client.FindsByAndroidUIAutomator; | ||
import io.appium.java_client.HasOnScreenKeyboard; | ||
import io.appium.java_client.LocksDevice; | ||
import io.appium.java_client.*; |
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.
As far as I remember the linter does like star imports
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.
Fixed!
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.
LGTM, except of linter issues
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.
LGTM. Codacy and travis warnings needs to be fixed though.
Change list
Related : appium/appium-espresso-driver#189
Types of changes
What types of changes are you proposing/introducing to Java client?
Put an
x
in the boxes that apply