-
-
Notifications
You must be signed in to change notification settings - Fork 75
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
fix: Fix compose driver for compose 1.6.x method signature change #995
Conversation
gradle wrapper --gradle-version 8.2
@KazuCocoa @mykola-mokhnach could you please help, The tests passed on API 29 but failed on all other API level. |
I guess slow ci related. let me retry |
Yep, just checked the test have failed similarly on last few |
it.call(EspressoServerRunnerTest.composeTestRule.onRoot(useUnmergedTree = true), true, null) | ||
// INFO: Compose API has added the method parameter in compose 1.6.x+ | ||
if (it.parameters.size == 4) { | ||
it.call(EspressoServerRunnerTest.composeTestRule.onRoot(useUnmergedTree = true), true, null, true) |
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.
we may extract the first argument to avoid unnecessary code duplication
espresso-server/app/src/androidTest/java/io/appium/espressoserver/lib/model/SourceDocument.kt
Show resolved
Hide resolved
## [2.38.1](v2.38.0...v2.38.1) (2024-04-12) ### Bug Fixes * Fix compose driver for compose 1.6.x method signature change ([#995](#995)) ([41f1b8d](41f1b8d))
🎉 This PR is included in version 2.38.1 🎉 The release is available on: Your semantic-release bot 📦🚀 |
Compose library has change method signature of
fetchSemanticsNodes
. This PR addresses that.https://cs.android.com/androidx/platform/frameworks/support/+/androidx-main:compose/ui/ui-test/src/commonMain/kotlin/androidx/compose/ui/test/SemanticsNodeInteraction.kt;l=59;bpv=1;bpt=0;drc=78d5dc8c9b42c32b0c8518c72181c19620a71c05;dlc=7aed8c7ff9c9355d5dfeb28021f8725d58dc4c6f