-
Notifications
You must be signed in to change notification settings - Fork 1.9k
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Help with #95
- Loading branch information
Showing
1 changed file
with
3 additions
and
0 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
febf554
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.
@LeoNatan : used this to search inside earlgrey. looks like they are doing something similar and more "magic": inside the method grey_enableAccessibility https://github.com/google/EarlGrey/blob/master/EarlGrey/Core/GREYAutomationSetup.m#L147
febf554
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.
Interesting. So this code might not be called in Detox for some reason. Worth investigating.
febf554
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.
The
grey_enableAccessibility
is called only for XCTest cases and only for simulators. They have a compile time branch for devices, but part doesn't run ever.I looked at its code. I actually prefer your fix.