Fix the issue where errors are ignored and occur when changing the size during GIF recording.

README.md

@@ -264,7 +264,7 @@ class ManualTest {
       .captureRoboImage()
 
     // Capture Jetpack Compose Node
-    composeTestRule.onNodeWithTag("MyComposeButton")
+    composeTestRule.onNodeWithTag("AddBoxButton")
       .onParent()
       .captureRoboImage("build/compose.png")
   }
@@ -280,7 +280,7 @@ Roborazzi supports the following APIs.
 </td><td>
 
 ```kotlin
-composeTestRule.onNodeWithTag("MyComposeButton")
+composeTestRule.onNodeWithTag("AddBoxButton")
   .captureRoboImage()
 ```
 
@@ -635,7 +635,7 @@ fun SampleComposableFunction() {
   ) {
     Box(
       Modifier
-        .testTag("MyComposeButton")
+        .testTag("AddBoxButton")
         .size(50.dp)
         .clickable {
           count++
@@ -676,7 +676,7 @@ class ComposeTest {
     }
     (0 until 3).forEach { _ ->
       composeTestRule
-        .onNodeWithTag("MyComposeButton")
+        .onNodeWithTag("AddBoxButton")
         .performClick()
     }
   }

roborazzi-painter/src/main/java/com/github/takahirom/roborazzi/DifferBufferedImage.kt

@@ -11,6 +11,10 @@ internal class DifferBufferedImage(private val bufferedImage: BufferedImage) : I
     get() = bufferedImage.width
 
   override fun getPixel(x: Int, y: Int): Color {
+    if (width <= x || height <= y) {
+      // Waiting for dropbox differs next release to support size difference
+      return Color(0, 0, 0, 0)
+    }
     return Color(bufferedImage.getRGB(x, y))
   }
 }

roborazzi/src/main/java/com/github/takahirom/roborazzi/Roborazzi.kt

@@ -28,7 +28,7 @@ import org.hamcrest.Matcher
 import org.hamcrest.Matchers
 import org.hamcrest.core.IsEqual
 import java.io.File
-import java.util.Locale
+import java.util.*
 
 fun ViewInteraction.captureRoboImage(
   filePath: String = DefaultFileNameGenerator.generateFilePath("png"),
@@ -246,6 +246,7 @@ fun SemanticsNodeInteraction.captureRoboGif(
   ).apply {
     saveGif(fileWithRecordFilePathStrategy(filePath))
     clear()
+    result.getOrThrow()
   }
 }
 
@@ -260,6 +261,7 @@ fun SemanticsNodeInteraction.captureRoboGif(
   captureComposeNode(composeRule, roborazziOptions, block).apply {
     saveGif(file)
     clear()
+    result.getOrThrow()
   }
 }
 

sample-android/src/main/java/com/github/takahirom/roborazzi/sample/FirstFragment.kt

@@ -9,11 +9,8 @@ import androidx.compose.foundation.clickable
 import androidx.compose.foundation.layout.Box
 import androidx.compose.foundation.layout.Column
 import androidx.compose.foundation.layout.size
-import androidx.compose.runtime.Composable
-import androidx.compose.runtime.getValue
-import androidx.compose.runtime.mutableStateOf
-import androidx.compose.runtime.remember
-import androidx.compose.runtime.setValue
+import androidx.compose.material3.Text
+import androidx.compose.runtime.*
 import androidx.compose.ui.Modifier
 import androidx.compose.ui.graphics.Color
 import androidx.compose.ui.platform.testTag
@@ -61,27 +58,40 @@ class FirstFragment : Fragment() {
 
 @Composable
 fun SampleComposableFunction() {
-  var count by remember { mutableStateOf(0) }
+  var count by remember { mutableStateOf(2) }
   Column(
     Modifier
-      .size(300.dp)
+      .testTag("MyColumn")
   ) {
-    Box(
+    Text(
+      text = "Add",
       Modifier
-        .testTag("MyComposeButton")
+        .testTag("AddBoxButton")
         .background(Color.Gray)
-        .size(50.dp)
         .clickable {
           count++
         }
     )
-    (0 until count).forEach {
+    Text(
+      text = "Sub",
+      Modifier
+        .testTag("SubBoxButton")
+        .background(Color.Gray)
+        .clickable {
+          count--
+        }
+    )
+    (0 until count).forEach { index ->
       Box(
         Modifier
           .background(Color.Red)
-          .testTag("child:$it")
-          .size(30.dp)
-      )
+          .testTag("child:$index")
+          .size(25.dp)
+      ) {
+        Text(
+          text = "$index/$count"
+        )
+      }
     }
   }
 }

sample-android/src/main/res/layout/fragment_first.xml

@@ -21,8 +21,9 @@
     android:id="@+id/compose"
     android:layout_width="wrap_content"
     android:layout_height="wrap_content"
-    app:layout_constraintEnd_toEndOf="parent"
+    app:layout_constraintTop_toTopOf="parent"
     app:layout_constraintBottom_toBottomOf="parent"
+    app:layout_constraintStart_toStartOf="parent"
     />
 
   <Button

sample-android/src/test/java/com/github/takahirom/roborazzi/sample/ComposeSetContentTest.kt

@@ -55,7 +55,7 @@ class ComposeSetContentTest {
       }
       (0 until 3).forEach { _ ->
         composeTestRule
-          .onNodeWithTag("MyComposeButton")
+          .onNodeWithTag("AddBoxButton")
           .performClick()
       }
     } catch (e: Exception) {
@@ -73,7 +73,7 @@ fun SampleComposableFunction() {
   ) {
     Box(
       Modifier
-        .testTag("MyComposeButton")
+        .testTag("AddBoxButton")
         .size(50.dp)
         .clickable {
           count++

sample-android/src/test/java/com/github/takahirom/roborazzi/sample/ComposeTest.kt

@@ -54,7 +54,7 @@ class ComposeTest {
     }
     (0 until 3).forEach { _ ->
       composeTestRule
-        .onNodeWithTag("MyComposeButton")
+        .onNodeWithTag("AddBoxButton")
         .performClick()
     }
   }

sample-android/src/test/java/com/github/takahirom/roborazzi/sample/ManualTest.kt

@@ -8,7 +8,6 @@ import androidx.compose.material3.Text
 import androidx.compose.ui.test.junit4.createAndroidComposeRule
 import androidx.compose.ui.test.onNodeWithTag
 import androidx.compose.ui.test.onParent
-import androidx.compose.ui.test.onRoot
 import androidx.compose.ui.test.performClick
 import androidx.core.graphics.applyCanvas
 import androidx.core.graphics.createBitmap
@@ -68,7 +67,7 @@ class ManualTest {
 
   @Test
   fun captureComposeImage() {
-    composeTestRule.onNodeWithTag("MyComposeButton")
+    composeTestRule.onNodeWithTag("AddBoxButton")
       .onParent()
       .captureRoboImage()
   }
@@ -134,10 +133,10 @@ class ManualTest {
         )
       )
 
-    composeTestRule.onNodeWithTag("MyComposeButton")
+    composeTestRule.onNodeWithTag("AddBoxButton")
       .performClick()
 
-    composeTestRule.onNodeWithTag("MyComposeButton")
+    composeTestRule.onNodeWithTag("AddBoxButton")
       .performClick()
     composeTestRule.waitForIdle()
 
@@ -220,16 +219,16 @@ class ManualTest {
     qualifiers = "w150dp-h200dp",
   )
   fun captureRoboGifSampleCompose() {
-    composeTestRule.onRoot(false)
+    composeTestRule.onNodeWithTag("MyColumn")
       .captureRoboGif(
         composeTestRule,
         "$DEFAULT_ROBORAZZI_OUTPUT_DIR_PATH/manual_captureRoboGifSampleCompose.gif"
       ) {
-        composeTestRule.onNodeWithTag("MyComposeButton")
+        composeTestRule.onNodeWithTag("SubBoxButton")
           .performClick()
-        composeTestRule.onNodeWithTag("MyComposeButton")
+        composeTestRule.onNodeWithTag("AddBoxButton")
           .performClick()
-        composeTestRule.onNodeWithTag("MyComposeButton")
+        composeTestRule.onNodeWithTag("AddBoxButton")
           .performClick()
       }
   }