♻️ Enhanced ContributorsScreen testing.

core/model/src/commonMain/kotlin/io/github/droidkaigi/confsched/model/Contributor.kt

@@ -12,11 +12,11 @@ public data class Contributor(
     public companion object
 }
 
-public fun Contributor.Companion.fakes(): PersistentList<Contributor> = (0..20)
+public fun Contributor.Companion.fakes(): PersistentList<Contributor> = (1..20)
     .map {
         Contributor(
             id = it,
-            username = it.toString(),
+            username = "username $it",
             profileUrl = "https://developer.android.com/",
             iconUrl = "https://placehold.jp/150x150.png",
         )

core/testing/build.gradle.kts

@@ -21,6 +21,7 @@ dependencies {
     implementation(projects.feature.sponsors)
     implementation(projects.feature.favorites)
     implementation(projects.feature.eventmap)
+    implementation(projects.feature.contributors)
     implementation(libs.daggerHiltAndroidTesting)
     implementation(libs.roborazzi)
     implementation(libs.kermit)

core/testing/src/main/java/io/github/droidkaigi/confsched/testing/robot/ContributorsScreenRobot.kt

@@ -0,0 +1,105 @@
+package io.github.droidkaigi.confsched.testing.robot
+
+import androidx.compose.ui.test.assertContentDescriptionEquals
+import androidx.compose.ui.test.assertIsDisplayed
+import androidx.compose.ui.test.assertTextEquals
+import androidx.compose.ui.test.hasTestTag
+import androidx.compose.ui.test.hasText
+import androidx.compose.ui.test.performScrollToIndex
+import io.github.droidkaigi.confsched.contributors.ContributorsItemTestTagPrefix
+import io.github.droidkaigi.confsched.contributors.ContributorsScreen
+import io.github.droidkaigi.confsched.contributors.ContributorsTestTag
+import io.github.droidkaigi.confsched.contributors.component.ContributorsItemImageTestTagPrefix
+import io.github.droidkaigi.confsched.contributors.component.ContributorsUserNameTextTestTagPrefix
+import io.github.droidkaigi.confsched.model.Contributor
+import io.github.droidkaigi.confsched.model.fakes
+import io.github.droidkaigi.confsched.testing.utils.assertCountAtLeast
+import io.github.droidkaigi.confsched.testing.utils.hasTestTag
+import io.github.droidkaigi.confsched.ui.Inject
+
+class ContributorsScreenRobot @Inject constructor(
+    screenRobot: DefaultScreenRobot,
+    contributorsServerRobot: DefaultContributorsServerRobot,
+) : ScreenRobot by screenRobot,
+    ContributorsServerRobot by contributorsServerRobot {
+    fun setupScreenContent() {
+        robotTestRule.setContent {
+            ContributorsScreen(
+                onNavigationIconClick = { },
+                onContributorsItemClick = { },
+            )
+        }
+    }
+
+    fun scrollToIndex10() {
+        composeTestRule
+            .onNode(hasTestTag(ContributorsTestTag))
+            .performScrollToIndex(10)
+    }
+
+    fun checkRangeContributorItemsDisplayed(
+        fromTo: IntRange,
+    ) {
+        val contributorsList = Contributor.fakes().subList(fromTo.first, fromTo.last)
+        contributorsList.forEach { contributor ->
+            composeTestRule
+                .onNode(hasTestTag(ContributorsItemTestTagPrefix.plus(contributor.id)))
+                .assertExists()
+                .assertIsDisplayed()
+
+            composeTestRule
+                .onNode(
+                    matcher = hasTestTag(ContributorsItemImageTestTagPrefix.plus(contributor.username)),
+                    useUnmergedTree = true,
+                )
+                .assertExists()
+                .assertIsDisplayed()
+                .assertContentDescriptionEquals(contributor.username)
+
+            composeTestRule
+                .onNode(
+                    matcher = hasTestTag(ContributorsUserNameTextTestTagPrefix.plus(contributor.username)),
+                    useUnmergedTree = true,
+                )
+                .assertExists()
+                .assertIsDisplayed()
+                .assertTextEquals(contributor.username)
+        }
+    }
+
+    fun checkContributorItemsDisplayed() {
+        // Check there are two contributors
+        composeTestRule
+            .onAllNodes(hasTestTag(ContributorsItemTestTagPrefix, substring = true))
+            .assertCountAtLeast(2)
+    }
+
+    fun checkDoesNotFirstContributorItemDisplayed() {
+        val contributor = Contributor.fakes().first()
+        composeTestRule
+            .onNode(hasTestTag(ContributorsItemTestTagPrefix.plus(contributor.id)))
+            .assertDoesNotExist()
+
+        composeTestRule
+            .onNode(
+                matcher = hasTestTag(ContributorsItemImageTestTagPrefix.plus(contributor.username)),
+                useUnmergedTree = true,
+            )
+            .assertDoesNotExist()
+
+        composeTestRule
+            .onNode(
+                matcher = hasTestTag(ContributorsUserNameTextTestTagPrefix.plus(contributor.username)),
+                useUnmergedTree = true,
+            )
+            .assertDoesNotExist()
+    }
+
+    fun checkErrorSnackbarDisplayed() {
+        composeTestRule
+            .onNode(
+                hasText("Fake IO Exception"),
+                useUnmergedTree = true,
+            ).assertIsDisplayed()
+    }
+}

core/testing/src/main/java/io/github/droidkaigi/confsched/testing/utils/Matchers.kt

@@ -0,0 +1,60 @@
+package io.github.droidkaigi.confsched.testing.utils
+
+import androidx.compose.ui.semantics.SemanticsNode
+import androidx.compose.ui.semantics.SemanticsProperties
+import androidx.compose.ui.semantics.getOrNull
+import androidx.compose.ui.test.SemanticsMatcher
+import androidx.compose.ui.test.SemanticsNodeInteractionCollection
+
+fun hasTestTag(
+    testTag: String,
+    substring: Boolean = false,
+    ignoreCase: Boolean = false,
+): SemanticsMatcher {
+    return SemanticsMatcher(
+        "TestTag ${if (substring) "contains" else "is"} '$testTag' (ignoreCase: $ignoreCase)",
+    ) { node ->
+        val nodeTestTag: String? = node.config.getOrNull(SemanticsProperties.TestTag)
+        when {
+            nodeTestTag == null -> false
+            substring -> nodeTestTag.contains(testTag, ignoreCase)
+            else -> nodeTestTag.equals(testTag, ignoreCase)
+        }
+    }
+}
+
+fun SemanticsNodeInteractionCollection.assertCountAtLeast(
+    minimumExpectedSize: Int,
+): SemanticsNodeInteractionCollection {
+    val errorOnFail = "Failed to assert minimum count of nodes."
+    val matchedNodes = fetchSemanticsNodes(
+        atLeastOneRootRequired = minimumExpectedSize > 0,
+        errorOnFail,
+    )
+    if (matchedNodes.size < minimumExpectedSize) {
+        throw AssertionError(
+            buildErrorMessageForMinimumCountMismatch(
+                errorMessage = errorOnFail,
+                foundNodes = matchedNodes,
+                minimumExpectedCount = minimumExpectedSize,
+            ),
+        )
+    }
+    return this
+}
+
+private fun buildErrorMessageForMinimumCountMismatch(
+    errorMessage: String,
+    foundNodes: List<SemanticsNode>,
+    minimumExpectedCount: Int,
+): String {
+    return buildString {
+        appendLine(errorMessage)
+        appendLine("Expected at least: $minimumExpectedCount")
+        appendLine("Found: ${foundNodes.size}")
+        appendLine("Matched nodes:")
+        foundNodes.forEachIndexed { index, node ->
+            appendLine("$index: $node")
+        }
+    }
+}

feature/contributors/src/androidUnitTest/kotlin/io/github/droidkaigi/confsched/contributors/ContributorsScreenTest.kt

@@ -1,17 +1,12 @@
 package io.github.droidkaigi.confsched.contributors
 
-import androidx.compose.ui.test.assertIsDisplayed
-import androidx.compose.ui.test.hasTestTag
-import androidx.compose.ui.test.hasText
 import dagger.hilt.android.testing.BindValue
 import dagger.hilt.android.testing.HiltAndroidTest
 import io.github.droidkaigi.confsched.testing.DescribedBehavior
 import io.github.droidkaigi.confsched.testing.describeBehaviors
 import io.github.droidkaigi.confsched.testing.execute
+import io.github.droidkaigi.confsched.testing.robot.ContributorsScreenRobot
 import io.github.droidkaigi.confsched.testing.robot.ContributorsServerRobot
-import io.github.droidkaigi.confsched.testing.robot.DefaultContributorsServerRobot
-import io.github.droidkaigi.confsched.testing.robot.DefaultScreenRobot
-import io.github.droidkaigi.confsched.testing.robot.ScreenRobot
 import io.github.droidkaigi.confsched.testing.robot.runRobot
 import io.github.droidkaigi.confsched.testing.rules.RobotTestRule
 import org.junit.Rule
@@ -42,58 +37,54 @@ class ContributorsScreenTest(private val testCase: DescribedBehavior<Contributor
         @ParameterizedRobolectricTestRunner.Parameters(name = "{0}")
         fun behaviors(): List<DescribedBehavior<ContributorsScreenRobot>> {
             return describeBehaviors<ContributorsScreenRobot>(name = "ContributorsScreen") {
-                describe("when launch") {
+                describe("when server is operational") {
                     run {
                         setupContributorServer(ContributorsServerRobot.ServerStatus.Operational)
-                        setupScreenContent()
                     }
-                    itShould("show contributors list") {
-                        captureScreenWithChecks(
-                            checks = { checkContributorsDisplayed() },
-                        )
-                    }
-                }
-                describe("when launch with error") {
-                    run {
-                        setupContributorServer(ContributorsServerRobot.ServerStatus.Error)
-                        setupScreenContent()
+                    describe("when launch") {
+                        run {
+                            setupScreenContent()
+                        }
+                        itShould("show first and second contributors") {
+                            captureScreenWithChecks {
+                                checkRangeContributorItemsDisplayed(
+                                    fromTo = 0..2,
+                                )
+                            }
+                        }
+
+                        describe("when scroll to index 10") {
+                            run {
+                                scrollToIndex10()
+                            }
+                            itShould("show contributors") {
+                                captureScreenWithChecks {
+                                    checkContributorItemsDisplayed()
+                                }
+                            }
+                        }
                     }
-                    itShould("show error message") {
-                        captureScreenWithChecks(
-                            checks = { checkErrorSnackbarDisplayed() },
-                        )
+
+                    describe("when server is down") {
+                        run {
+                            setupContributorServer(ContributorsServerRobot.ServerStatus.Error)
+                        }
+                        describe("when launch") {
+                            run {
+                                setupScreenContent()
+                            }
+                            itShould("does not show contributor and show snackbar") {
+                                captureScreenWithChecks(
+                                    checks = {
+                                        checkDoesNotFirstContributorItemDisplayed()
+                                        checkErrorSnackbarDisplayed()
+                                    },
+                                )
+                            }
+                        }
                     }
                 }
             }
         }
     }
 }
-
-class ContributorsScreenRobot @Inject constructor(
-    screenRobot: DefaultScreenRobot,
-    contributorsServerRobot: DefaultContributorsServerRobot,
-) : ScreenRobot by screenRobot,
-    ContributorsServerRobot by contributorsServerRobot {
-    fun setupScreenContent() {
-        robotTestRule.setContent {
-            ContributorsScreen(
-                onNavigationIconClick = { },
-                onContributorsItemClick = { },
-            )
-        }
-    }
-
-    fun checkContributorsDisplayed() {
-        composeTestRule
-            .onNode(hasTestTag(ContributorsScreenTestTag))
-            .assertIsDisplayed()
-    }
-
-    fun checkErrorSnackbarDisplayed() {
-        composeTestRule
-            .onNode(
-                hasText("Fake IO Exception"),
-                useUnmergedTree = true,
-            ).assertIsDisplayed()
-    }
-}

feature/contributors/src/commonMain/kotlin/io/github/droidkaigi/confsched/contributors/ContributorsScreen.kt

@@ -35,6 +35,8 @@ import kotlinx.collections.immutable.PersistentList
 
 const val contributorsScreenRoute = "contributors"
 const val ContributorsScreenTestTag = "ContributorsScreenTestTag"
+const val ContributorsTestTag = "ContributorsTestTag"
+const val ContributorsItemTestTagPrefix = "ContributorsItemTestTag:"
 
 fun NavGraphBuilder.contributorsScreens(
     onNavigationIconClick: () -> Unit,
@@ -153,14 +155,16 @@ private fun Contributors(
     contentPadding: PaddingValues = PaddingValues(),
 ) {
     LazyColumn(
-        modifier = modifier,
+        modifier = modifier.testTag(ContributorsTestTag),
         contentPadding = contentPadding,
     ) {
         items(contributors) {
             ContributorsItem(
                 contributor = it,
                 onClick = onContributorsItemClick,
-                modifier = Modifier.fillMaxWidth(),
+                modifier = Modifier
+                    .fillMaxWidth()
+                    .testTag(ContributorsItemTestTagPrefix.plus(it.id)),
             )
         }
     }

feature/contributors/src/commonMain/kotlin/io/github/droidkaigi/confsched/contributors/component/ContributorsItem.kt

@@ -17,12 +17,16 @@ import androidx.compose.ui.Alignment
 import androidx.compose.ui.Modifier
 import androidx.compose.ui.draw.clip
 import androidx.compose.ui.graphics.vector.rememberVectorPainter
+import androidx.compose.ui.platform.testTag
 import androidx.compose.ui.text.style.TextOverflow
 import androidx.compose.ui.unit.dp
 import io.github.droidkaigi.confsched.model.Contributor
 import io.github.droidkaigi.confsched.ui.previewOverride
 import io.github.droidkaigi.confsched.ui.rememberAsyncImagePainter
 
+const val ContributorsItemImageTestTagPrefix = "ContributorsItemImageTestTag:"
+const val ContributorsUserNameTextTestTagPrefix = "ContributorsUserNameTextTestTag:"
+
 private val contributorIconShape = CircleShape
 
 @Composable
@@ -44,21 +48,23 @@ fun ContributorsItem(
             painter = previewOverride(previewPainter = { rememberVectorPainter(image = Icons.Default.Person) }) {
                 rememberAsyncImagePainter(contributor.iconUrl)
             },
-            contentDescription = null,
+            contentDescription = contributor.username,
             modifier = Modifier
                 .size(52.dp)
                 .clip(contributorIconShape)
                 .border(
                     width = 1.dp,
                     color = MaterialTheme.colorScheme.outline,
                     shape = contributorIconShape,
-                ),
+                )
+                .testTag(ContributorsItemImageTestTagPrefix.plus(contributor.username)),
         )
         Text(
             text = contributor.username,
             style = MaterialTheme.typography.bodyLarge,
             maxLines = 2,
             overflow = TextOverflow.Ellipsis,
+            modifier = Modifier.testTag(ContributorsUserNameTextTestTagPrefix.plus(contributor.username)),
         )
     }
 }