Update text and element test IDs

CrisisCleanup · Feb 20, 2024 · 8846fd4 · 8846fd4
1 parent 9cac72a
commit 8846fd4
Show file tree

Hide file tree

Showing 3 changed files with 11 additions and 7 deletions.
diff --git a/app/build.gradle.kts b/app/build.gradle.kts
@@ -13,7 +13,7 @@ plugins {
 
 android {
     defaultConfig {
-        val buildVersion = 187
+        val buildVersion = 188
         applicationId = "com.crisiscleanup"
         versionCode = buildVersion
         versionName = "0.9.${buildVersion - 168}"

diff --git a/app/src/main/java/com/crisiscleanup/MainActivityViewModel.kt b/app/src/main/java/com/crisiscleanup/MainActivityViewModel.kt
@@ -286,7 +286,7 @@ class MainActivityViewModel @Inject constructor(
                     accountDataRefresher.updateAcceptedTerms()
                 } else {
                     val errorMessage = if (networkMonitor.isOnline.first()) {
-                        translator("~~Something went wrong. Please try again later")
+                        translator("~~Something went wrong. Please try again later.")
                     } else {
                         translator("~~Connect to the internet and try again.")
                     }

diff --git a/app/src/main/java/com/crisiscleanup/ui/AcceptTermsView.kt b/app/src/main/java/com/crisiscleanup/ui/AcceptTermsView.kt
@@ -55,13 +55,13 @@ internal fun AcceptTermsView(
                 // TODO Common dimensions
                 Modifier
                     .padding(16.dp)
-                    .testTag("acceptTermsAgreeText"),
+                    .testTag("acceptTermsAgreeAgreementText"),
             )
 
             AndroidView(
                 modifier = Modifier
                     .weight(1f)
-                    .testTag("acceptTermsWebView"),
+                    .testTag("acceptTermsAgreement"),
                 factory = { context ->
                     WebView(context).apply {
                         settings.javaScriptEnabled = true
@@ -75,7 +75,7 @@ internal fun AcceptTermsView(
             if (errorMessage.isNotBlank()) {
                 Text(
                     errorMessage,
-                    listItemModifier,
+                    listItemModifier.testTag("acceptTermsErrorMessage"),
                     color = primaryRedColor,
                 )
             }
@@ -87,11 +87,15 @@ internal fun AcceptTermsView(
                 Checkbox(
                     checked = isAcceptingTerms,
                     onCheckedChange = setAcceptingTerms,
+                    modifier = Modifier.testTag("acceptTermsAcceptToggle"),
                     enabled = enable,
                 )
-                val acceptText = t("~~I accept the terms of service from {terms_url} ?")
+                val acceptText = t("~~I accept the terms of service from {terms_url}")
                     .replace("{terms_url}", termsOfServiceUrl)
-                LinkifyUrlText(acceptText)
+                LinkifyUrlText(
+                    acceptText,
+                    Modifier.testTag("acceptTermsAcceptText"),
+                )
             }
 
             FlowRow(