-
-
Notifications
You must be signed in to change notification settings - Fork 764
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
Predicate locator added in IOSDriver, IOSElement, and iOSFindBy #475
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.
https://github.com/appium/java-client/pull/475/files#diff-2fe175c53bb11f987c6da9bb5fedc74eR46 CheckStyle will fail here.
Oops, must have snuck in there. Fixed |
I don't have iOS 9 devices with me to test. @TikhomirovSergey ping |
@SrinivasanTarget @mt1729 |
@TikhomirovSergey I don't believe the common driver has been updated yet to support finding elements directly by predicate, so this seems to be specific to the new XCUITest driver. However, do you think it would be reasonable to use the UI Automation predicate helpers when finding by predicate using the common driver? |
@mt1729 Yup Initially i was confused. Btw Are you able to get this working in XCUITest? If so can you please share your execution logs which succeed. I was not able to use this locator in XCUITest as well. |
@SrinivasanTarget Made a small mistake in the tests. The property should have been 'label' instead of 'name' which I've updated. Here are the logs for one of the tests running on my machine. Let me know if this fix still doesn't work for you. |
@SrinivasanTarget is this not the exact same thing that was removed here? Seems like you can just revert that commit. |
Ping @SrinivasanTarget |
@Rafael-Chavez Yup will look into it soon. Please sign here: #523 |
Sorry for the late responce. I'm closing this PR in favor #545 |
Types of changes
Added support for the existing FindsByIosNSPredicate interface to the classes above and in PageFactory strategies. Tests for each were updated to reflect changes
What types of changes are you proposing/introducing to Java client?
Put an
x
in the boxes that apply