From 765df10b93f58e1ed7f7ae2df6907b1930c92738 Mon Sep 17 00:00:00 2001
From: Pablo Garcia <pablo.garcianunez@telefonica.com>
Date: Fri, 31 May 2024 12:17:53 +0200
Subject: [PATCH] Fix tests screenshot names

---
 .../telefonica/mistica/compose/button/ButtonKtTest.kt | 11 +++++++----
 1 file changed, 7 insertions(+), 4 deletions(-)

diff --git a/library/src/test/java/com/telefonica/mistica/compose/button/ButtonKtTest.kt b/library/src/test/java/com/telefonica/mistica/compose/button/ButtonKtTest.kt
index 350b0e236..ff8c8b5dc 100644
--- a/library/src/test/java/com/telefonica/mistica/compose/button/ButtonKtTest.kt
+++ b/library/src/test/java/com/telefonica/mistica/compose/button/ButtonKtTest.kt
@@ -77,9 +77,12 @@ internal class ButtonKtTest(
         darkTheme: Boolean,
         loadingText: String,
     ) {
-        val extra: MutableList<String> = mutableListOf()
-        icon.takeIf { it }?.let { extra.add("icon") }
-        loadingText.takeIf { it.isNotEmpty() }?.let { extra.add("loadingText") }
+        val extra: String? = mutableListOf<String>().apply {
+            icon.takeIf { it }?.let { add("icon") }
+            loadingText.takeIf { it.isNotEmpty() }?.let { add("loadingText") }
+        }.takeIf {
+            it.isNotEmpty()
+        }?.joinToString(separator = "_")
 
         compareScreenshot(
             node = composeTestRule.onRoot(),
@@ -87,7 +90,7 @@ internal class ButtonKtTest(
             style = style.toString(),
             brand = brand,
             darkTheme = darkTheme,
-            extra = extra.joinToString(separator = "_")
+            extra = extra
         )
     }