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
Update Compose to dev10. #1118
Update Compose to dev10. #1118
Conversation
e2d9ca1
to
e7cf0b3
Compare
@@ -28,7 +28,7 @@ import org.junit.runner.RunWith | |||
class HelloBindingTest { | |||
|
|||
// Launches the activity. | |||
@Rule @JvmField val scenario = ActivityScenarioRule(HelloBindingActivity::class.java) | |||
@Rule @JvmField val composeRule = AndroidComposeTestRule<HelloBindingActivity>() |
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.
Compose top-level UI test functions now require a ComposeTestRule
to be used, or they will throw.
@@ -84,7 +85,7 @@ internal class ComposeViewFactory<RenderingT : Any>( | |||
) | |||
|
|||
// Entry point to the composition. | |||
composeContainer.setContent { | |||
composeContainer.setContent(Recomposer.current()) { |
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.
Old method is deprecated.
Dev10 finally brings support for coroutines! This is the first compose update in a while to actually bump the compiler version as well as the plugin version, which is why it requires a few more changes to work.
e7cf0b3
to
d28651b
Compare
Looks like this might be an actual test failure, but the Actions UI isn't even showing a log for this task so I github might just be having issues. |
UI test failure is now: |
Description
Dev10 finally brings support for coroutines!
This is the first compose update in a while to actually bump the compiler version
as well as the plugin version, which is why it requires a few more changes
to work.
Fixes # (issue)
Type of change
Please select at least one
Testing Checklist
How was this change tested?
Checklist: