From 48b3b83cae54bbd8a8f65572077ac065083f397f Mon Sep 17 00:00:00 2001
From: Harsh Shandilya <me@msfjarvis.dev>
Date: Tue, 8 Sep 2020 05:53:06 +0530
Subject: [PATCH] Set test tag on ItemAddDialog

Signed-off-by: Harsh Shandilya <me@msfjarvis.dev>
---
 app/src/main/java/dev/msfjarvis/todo/MainActivity.kt | 10 ++++++----
 1 file changed, 6 insertions(+), 4 deletions(-)

diff --git a/app/src/main/java/dev/msfjarvis/todo/MainActivity.kt b/app/src/main/java/dev/msfjarvis/todo/MainActivity.kt
index 56e1a3c5..0987226f 100644
--- a/app/src/main/java/dev/msfjarvis/todo/MainActivity.kt
+++ b/app/src/main/java/dev/msfjarvis/todo/MainActivity.kt
@@ -22,8 +22,7 @@ import androidx.compose.runtime.rememberCoroutineScope
 import androidx.compose.runtime.setValue
 import androidx.compose.ui.Modifier
 import androidx.compose.ui.platform.setContent
-import androidx.compose.ui.semantics.semantics
-import androidx.compose.ui.semantics.testTag
+import androidx.compose.ui.platform.testTag
 import androidx.compose.ui.text.input.TextFieldValue
 import androidx.compose.ui.unit.dp
 import androidx.ui.tooling.preview.Preview
@@ -64,6 +63,7 @@ fun TodoApp(
     ItemAddDialog(
       showingDialog = showingDialog,
       onAdd = onAdd,
+      modifier = Modifier.testTag("item_dialog")
     )
   }
 
@@ -73,7 +73,7 @@ fun TodoApp(
       FloatingActionButton(
         onClick = { showingDialog.value = true },
         elevation = 8.dp,
-        modifier = Modifier.semantics { testTag = "fab" }
+        modifier = Modifier.testTag("fab")
       ) {
         IconResource(
           resourceId = R.drawable.ic_exposure_plus_1_24dp,
@@ -96,6 +96,7 @@ fun TodoApp(
 fun ItemAddDialog(
   showingDialog: MutableState<Boolean>,
   onAdd: (item: TodoItem) -> Unit,
+  modifier: Modifier = Modifier,
 ) {
   var newItemName by mutableStateOf(TextFieldValue(""))
   val hideDialog = { showingDialog.value = false }
@@ -121,7 +122,8 @@ fun ItemAddDialog(
       ) {
         Text(text = "Add")
       }
-    }
+    },
+    modifier = Modifier then modifier,
   )
 }