diff --git a/app/src/main/java/org/groundplatform/android/ui/components/MapFloatingActionButton.kt b/app/src/main/java/org/groundplatform/android/ui/components/MapFloatingActionButton.kt index 69660dd0ab..6b8039a54f 100644 --- a/app/src/main/java/org/groundplatform/android/ui/components/MapFloatingActionButton.kt +++ b/app/src/main/java/org/groundplatform/android/ui/components/MapFloatingActionButton.kt @@ -30,6 +30,11 @@ import androidx.compose.ui.unit.dp import org.groundplatform.android.R import org.groundplatform.android.ui.theme.AppTheme +const val OPEN_NAV_DRAWER_TEST_TAG = "open_nav_drawer" +const val CHOOSE_MAP_TYPE_TEST_TAG = "choose_map_type" +const val LOCATION_LOCKED_TEST_TAG = "location_locked" +const val LOCATION_NOT_LOCKED_TEST_TAG = "location_not_locked" + @Composable fun MapFloatingActionButton( modifier: Modifier = Modifier, @@ -55,10 +60,13 @@ sealed class MapFloatingActionButtonType( val testTag: String, ) { data object OpenNavDrawer : - MapFloatingActionButtonType(iconRes = R.drawable.baseline_menu_24, testTag = "open_nav_drawer") + MapFloatingActionButtonType( + iconRes = R.drawable.baseline_menu_24, + testTag = OPEN_NAV_DRAWER_TEST_TAG, + ) data object MapType : - MapFloatingActionButtonType(iconRes = R.drawable.map_layers, testTag = "choose_map_type") + MapFloatingActionButtonType(iconRes = R.drawable.map_layers, testTag = CHOOSE_MAP_TYPE_TEST_TAG) // TODO: Consider adding another icon for representing "GPS disabled" state. // Issue URL: https://github.com/google/ground-android/issues/1789 @@ -66,13 +74,13 @@ sealed class MapFloatingActionButtonType( MapFloatingActionButtonType( iconRes = R.drawable.ic_gps_lock, iconTintRes = iconTintRes, - testTag = "location_locked", + testTag = LOCATION_LOCKED_TEST_TAG, ) data object LocationNotLocked : MapFloatingActionButtonType( iconRes = R.drawable.ic_gps_lock_not_fixed, - testTag = "location_not_locked", + testTag = LOCATION_NOT_LOCKED_TEST_TAG, ) } diff --git a/app/src/main/java/org/groundplatform/android/ui/datacollection/tasks/multiplechoice/MultipleChoiceItemView.kt b/app/src/main/java/org/groundplatform/android/ui/datacollection/tasks/multiplechoice/MultipleChoiceItemView.kt index e61fbd2904..09c7abe87c 100644 --- a/app/src/main/java/org/groundplatform/android/ui/datacollection/tasks/multiplechoice/MultipleChoiceItemView.kt +++ b/app/src/main/java/org/groundplatform/android/ui/datacollection/tasks/multiplechoice/MultipleChoiceItemView.kt @@ -52,6 +52,7 @@ import org.groundplatform.android.ui.theme.AppTheme const val MULTIPLE_CHOICE_ITEM_TEST_TAG = "multiple choice item test tag" const val OTHER_INPUT_TEXT_TEST_TAG = "other input test tag" const val SELECT_MULTIPLE_RADIO_TEST_TAG = "select multiple radio test tag" +const val SELECT_MULTIPLE_CHECKBOX_TEST_TAG = "select multiple checkbox test tag" /** * A composable function that displays a single item in a multiple-choice list. @@ -95,7 +96,11 @@ fun MultipleChoiceItemView( } MultipleChoice.Cardinality.SELECT_MULTIPLE -> { - Checkbox(checked = item.isSelected, onCheckedChange = { toggleItem(item) }) + Checkbox( + modifier = Modifier.testTag(SELECT_MULTIPLE_CHECKBOX_TEST_TAG), + checked = item.isSelected, + onCheckedChange = { toggleItem(item) }, + ) } } diff --git a/app/src/main/res/layout/date_task_frag.xml b/app/src/main/res/layout/date_task_frag.xml index a502a1dd5a..c683645d87 100644 --- a/app/src/main/res/layout/date_task_frag.xml +++ b/app/src/main/res/layout/date_task_frag.xml @@ -31,7 +31,7 @@ android:layout_width="wrap_content" android:layout_height="wrap_content"> (job, task) - onView(withId(R.id.user_response_text)) + onView(withId(R.id.user_date_response_text)) .check(matches(withText(""))) .check(matches(isDisplayed())) .check(matches(isEnabled())) @@ -89,7 +89,7 @@ class DateTaskFragmentTest : BaseTaskFragmentTest(job, task) - onView(withId(R.id.user_response_text)) + onView(withId(R.id.user_time_response_text)) .check(matches(withText(""))) .check(matches(isDisplayed())) .check(matches(isEnabled())) @@ -86,7 +86,7 @@ class TimeTaskFragmentTest : BaseTaskFragmentTest