Predicate locator added in IOSDriver, IOSElement, and iOSFindBy#475
Predicate locator added in IOSDriver, IOSElement, and iOSFindBy#475mt1729 wants to merge 4 commits intoappium:masterappium/java-client:masterfrom mt1729:masterCopy head branch name to clipboard
Conversation
SrinivasanTarget
left a comment
There was a problem hiding this comment.
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
xin the boxes that apply