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
adjustSliderToPosition on Android #2888
Conversation
detox/android/detox/src/full/java/com/wix/detox/espresso/matcher/ViewMatchers.kt
Outdated
Show resolved
Hide resolved
detox/android/detox/src/full/java/com/wix/detox/espresso/matcher/ViewMatchers.kt
Outdated
Show resolved
Hide resolved
detox/android/detox/src/full/java/com/wix/detox/espresso/matcher/ViewMatchers.kt
Outdated
Show resolved
Hide resolved
detox/android/detox/src/full/java/com/wix/detox/espresso/matcher/ViewMatchers.kt
Outdated
Show resolved
Hide resolved
...id/detox/src/testFull/java/com/wix/detox/espresso/action/AdjustSliderToPositionActionTest.kt
Outdated
Show resolved
Hide resolved
detox/android/detox/src/main/java/com/wix/detox/espresso/action/AdjustSliderToPositionAction.kt
Outdated
Show resolved
Hide resolved
detox/android/detox/src/main/java/com/wix/detox/espresso/action/AdjustSliderToPositionAction.kt
Outdated
Show resolved
Hide resolved
detox/android/detox/src/main/java/com/wix/detox/espresso/action/AdjustSliderToPositionAction.kt
Outdated
Show resolved
Hide resolved
detox/android/detox/src/main/java/com/wix/detox/espresso/action/AdjustSliderToPositionAction.kt
Outdated
Show resolved
Hide resolved
detox/android/detox/src/main/java/com/wix/detox/espresso/action/AdjustSliderToPositionAction.kt
Outdated
Show resolved
Hide resolved
detox/android/detox/src/main/java/com/wix/detox/espresso/action/AdjustSliderToPositionAction.kt
Outdated
Show resolved
Hide resolved
...id/detox/src/testFull/java/com/wix/detox/espresso/action/AdjustSliderToPositionActionTest.kt
Show resolved
Hide resolved
Hey, thanks for all the work! 🙏🏻 |
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.
Generally looks good on the JS side, but I wonder that you added API to waitFor
chains, but it is not properly tested anywhere.
Also, the percent handling (iOS only) is better off being documented as not supported on Android, or maybe we can handle strings that .endsWith('%')
and just parseFloat(...) * 0.01
, all on JS side?
67213b7
to
58419ee
Compare
# Conflicts: # detox/android/detox/src/full/java/com/wix/detox/espresso/DetoxAction.java # detox/src/android/actions/native.js # detox/src/android/espressoapi/DetoxAction.js
Description
Adding adjustSliderToPosition and toHaveSliderPosition for Android - addresses #2792.