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

Predicate locator added in IOSDriver, IOSElement, and iOSFindBy #475

Closed
wants to merge 4 commits into from
Closed

Predicate locator added in IOSDriver, IOSElement, and iOSFindBy #475

wants to merge 4 commits into from

Conversation

mt1729
Copy link

@mt1729 mt1729 commented Sep 23, 2016

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

  • Bugfix (non-breaking change which fixes an issue)
  • [ x] New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)

Copy link
Member

@SrinivasanTarget SrinivasanTarget left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@mt1729
Copy link
Author

mt1729 commented Sep 25, 2016

Oops, must have snuck in there. Fixed

@SrinivasanTarget
Copy link
Member

I don't have iOS 9 devices with me to test. @TikhomirovSergey ping

@TikhomirovSergey
Copy link
Contributor

TikhomirovSergey commented Sep 26, 2016

@SrinivasanTarget @mt1729
It looks ok generally. But I need for some claryfication.
Does it work with the common IOSDriver starting? Don't you need some specific automation type?

@mt1729
Copy link
Author

mt1729 commented Oct 2, 2016

@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?

@SrinivasanTarget
Copy link
Member

@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.

@mt1729
Copy link
Author

mt1729 commented Oct 3, 2016

@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
Copy link
Member

@mt1729 Got it. But it will fail on legacy driver.This should probably get in post #398

@rafe-g
Copy link
Contributor

rafe-g commented Nov 10, 2016

@SrinivasanTarget is this not the exact same thing that was removed here?
#394

Seems like you can just revert that commit.

FYI @TikhomirovSergey

@rafe-g
Copy link
Contributor

rafe-g commented Nov 24, 2016

Ping @SrinivasanTarget

@SrinivasanTarget
Copy link
Member

@Rafael-Chavez Yup will look into it soon. Please sign here: #523

@TikhomirovSergey
Copy link
Contributor

Sorry for the late responce. I'm closing this PR in favor #545

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

Successfully merging this pull request may close these issues.

4 participants