Increase coverage of GroupChannelEditScreen and update snapshot files

Author: andremion

stream-chat-android-compose/src/main/java/io/getstream/chat/android/compose/ui/channel/info/GroupChannelEditScreen.kt

@@ -38,6 +38,7 @@ import androidx.compose.material3.OutlinedTextFieldDefaults
 import androidx.compose.material3.Scaffold
 import androidx.compose.material3.Text
 import androidx.compose.material3.TopAppBarDefaults
+import androidx.compose.material3.rememberModalBottomSheetState
 import androidx.compose.runtime.Composable
 import androidx.compose.runtime.LaunchedEffect
 import androidx.compose.runtime.getValue
@@ -51,6 +52,7 @@ import androidx.compose.ui.Modifier
 import androidx.compose.ui.focus.FocusRequester
 import androidx.compose.ui.focus.focusRequester
 import androidx.compose.ui.platform.LocalContext
+import androidx.compose.ui.platform.LocalInspectionMode
 import androidx.compose.ui.res.painterResource
 import androidx.compose.ui.res.stringResource
 import androidx.compose.ui.text.TextRange
@@ -151,9 +153,9 @@ internal fun GroupChannelEditScreen(
 private fun ImagePickerSheet(
     visible: Boolean,
     showRemoveOption: Boolean,
-    onDismiss: () -> Unit,
-    onImageSelected: (File) -> Unit,
-    onImageRemoved: () -> Unit,
+    onDismiss: () -> Unit = {},
+    onImageSelected: (File) -> Unit = {},
+    onImageRemoved: () -> Unit = {},
 ) {
     val context = LocalContext.current
     val scope = rememberCoroutineScope()
@@ -174,8 +176,15 @@ private fun ImagePickerSheet(
         onResult = onImageSelected,
     )
 
+    val previewMode = LocalInspectionMode.current
+    val sheetState = rememberModalBottomSheetState()
+    LaunchedEffect(previewMode) {
+        if (previewMode) sheetState.show()
+    }
+
     if (visible) {
         ModalBottomSheet(
+            sheetState = sheetState,
             onDismissRequest = onDismiss,
             containerColor = ChatTheme.colors.backgroundCoreApp,
         ) {
@@ -455,7 +464,7 @@ internal fun GroupChannelEditSaving() {
     )
 }
 
-@Preview(showBackground = true)
+@Preview
 @Composable
 private fun ImagePickerOptionsPreview() {
     ChatTheme {
@@ -465,10 +474,13 @@ private fun ImagePickerOptionsPreview() {
 
 @Composable
 internal fun ImagePickerOptionsWithRemove() {
-    ImagePickerOptions(showRemoveOption = true)
+    ImagePickerSheet(
+        visible = true,
+        showRemoveOption = true,
+    )
 }
 
-@Preview(showBackground = true)
+@Preview
 @Composable
 private fun ImagePickerOptionsNoRemovePreview() {
     ChatTheme {
@@ -478,5 +490,8 @@ private fun ImagePickerOptionsNoRemovePreview() {
 
 @Composable
 internal fun ImagePickerOptionsNoRemove() {
-    ImagePickerOptions(showRemoveOption = false)
+    ImagePickerSheet(
+        visible = true,
+        showRemoveOption = false,
+    )
 }

stream-chat-android-compose/src/test/kotlin/io/getstream/chat/android/compose/ui/PaparazziComposeTest.kt

@@ -32,6 +32,8 @@ import androidx.compose.runtime.Composable
 import androidx.compose.runtime.CompositionLocalProvider
 import androidx.compose.ui.Alignment
 import androidx.compose.ui.Modifier
+import androidx.compose.ui.graphics.Color
+import androidx.compose.ui.graphics.takeOrElse
 import androidx.compose.ui.platform.LocalInspectionMode
 import androidx.core.app.ActivityOptionsCompat
 import androidx.lifecycle.Lifecycle
@@ -53,13 +55,15 @@ internal interface PaparazziComposeTest : MockedChatClientTest {
     fun snapshot(
         isInDarkMode: Boolean = false,
         contentAlignment: Alignment = Alignment.TopStart,
+        backgroundColor: Color = Color.Unspecified,
         composable: @Composable () -> Unit,
     ) {
         paparazzi.snapshot {
             TestEnvironment {
                 ChatTheme(isInDarkMode = isInDarkMode) {
                     Box(
-                        modifier = Modifier.background(ChatTheme.colors.backgroundCoreApp),
+                        modifier = Modifier
+                            .background(backgroundColor.takeOrElse(ChatTheme.colors::backgroundCoreApp)),
                         contentAlignment = contentAlignment,
                     ) {
                         composable()

…/ui/channel/info/GroupChannelEditTest.kt …annel/info/GroupChannelEditScreenTest.ktstream-chat-android-compose/src/test/kotlin/io/getstream/chat/android/compose/ui/channel/info/GroupChannelEditTest.kt renamed to stream-chat-android-compose/src/test/kotlin/io/getstream/chat/android/compose/ui/channel/info/GroupChannelEditScreenTest.kt stream-chat-android-compose/src/test/kotlin/io/getstream/chat/android/compose/ui/channel/info/GroupChannelEditTest.kt renamed to stream-chat-android-compose/src/test/kotlin/io/getstream/chat/android/compose/ui/channel/info/GroupChannelEditScreenTest.kt

@@ -16,17 +16,17 @@
 
 package io.getstream.chat.android.compose.ui.channel.info
 
-import androidx.compose.ui.Alignment
+import androidx.compose.ui.graphics.Color
 import app.cash.paparazzi.DeviceConfig
 import app.cash.paparazzi.Paparazzi
 import io.getstream.chat.android.compose.ui.PaparazziComposeTest
 import org.junit.Rule
 import org.junit.Test
 
-internal class GroupChannelEditTest : PaparazziComposeTest {
+internal class GroupChannelEditScreenTest : PaparazziComposeTest {
 
     @get:Rule
-    override val paparazzi = Paparazzi(deviceConfig = DeviceConfig.PIXEL_4A)
+    override val paparazzi = Paparazzi(deviceConfig = DeviceConfig.PIXEL_2)
 
     @Test
     fun placeholder() {
@@ -50,15 +50,29 @@ internal class GroupChannelEditTest : PaparazziComposeTest {
     }
 
     @Test
-    fun `image picker with remove`() {
-        snapshotWithDarkMode(contentAlignment = Alignment.BottomCenter) {
+    fun `image picker with remove in light mode`() {
+        snapshot(backgroundColor = Color.Transparent) {
             ImagePickerOptionsWithRemove()
         }
     }
 
     @Test
-    fun `image picker no remove`() {
-        snapshotWithDarkMode(contentAlignment = Alignment.BottomCenter) {
+    fun `image picker with remove in dark mode`() {
+        snapshot(backgroundColor = Color.Transparent, isInDarkMode = true) {
+            ImagePickerOptionsWithRemove()
+        }
+    }
+
+    @Test
+    fun `image picker no remove in light mode`() {
+        snapshot(backgroundColor = Color.Transparent) {
+            ImagePickerOptionsNoRemove()
+        }
+    }
+
+    @Test
+    fun `image picker no remove in dark mode`() {
+        snapshot(backgroundColor = Color.Transparent, isInDarkMode = true) {
             ImagePickerOptionsNoRemove()
         }
     }