From 2c24ab25a687cab59d9b6b3385704edc5ddea74f Mon Sep 17 00:00:00 2001
From: Xavier Molloy <xavi@dhis2.org>
Date: Fri, 5 Jul 2024 09:37:20 +0200
Subject: [PATCH] fix: [ANDROAPP-6124] change phone number input keyboard to
 phone (#271)

---
 .../component/InputPhoneNumber.kt             |  6 ++--
 .../component/InputPhoneNumberTest.kt         | 34 +++++++++++++++++--
 2 files changed, 35 insertions(+), 5 deletions(-)

diff --git a/designsystem/src/commonMain/kotlin/org/hisp/dhis/mobile/ui/designsystem/component/InputPhoneNumber.kt b/designsystem/src/commonMain/kotlin/org/hisp/dhis/mobile/ui/designsystem/component/InputPhoneNumber.kt
index 2d246ff82..e93608022 100644
--- a/designsystem/src/commonMain/kotlin/org/hisp/dhis/mobile/ui/designsystem/component/InputPhoneNumber.kt
+++ b/designsystem/src/commonMain/kotlin/org/hisp/dhis/mobile/ui/designsystem/component/InputPhoneNumber.kt
@@ -20,6 +20,7 @@ import org.hisp.dhis.mobile.ui.designsystem.component.internal.RegExValidations
  * @param onCallActionClicked: callback to when call phone number button is clicked.
  * @param modifier: allows a modifier to be passed externally.
  * @param maxLength: number of characters/digits that can be entered.
+ * @param minLength: number of characters/digits that needs be entered to enable phone button.
  * @param state: Manages the InputShell state.
  * @param inputStyle: manages the InputShell style.
  * @param legendData: manages the legendComponent.
@@ -42,6 +43,7 @@ fun InputPhoneNumber(
     onCallActionClicked: () -> Unit,
     modifier: Modifier = Modifier,
     maxLength: Int = 12,
+    minLength: Int = 4,
     state: InputShellState,
     inputStyle: InputStyle = InputStyle.DataInputStyle(),
     legendData: LegendData? = null,
@@ -56,7 +58,7 @@ fun InputPhoneNumber(
     supportingText: List<SupportingTextData>? = emptyList(),
     allowedCharacters: RegExValidations = RegExValidations.PHONE_NUMBER,
 ) {
-    val hasMinimumPhoneNumberInput = inputTextFieldValue?.text.orEmpty().length > 2
+    val hasMinimumPhoneNumberInput = inputTextFieldValue?.text.orEmpty().length >= minLength
     BasicTextInput(
         title = title,
         state = state,
@@ -75,7 +77,7 @@ fun InputPhoneNumber(
         },
         keyboardOptions = KeyboardOptions(
             imeAction = imeAction,
-            keyboardType = KeyboardType.Number,
+            keyboardType = KeyboardType.Phone,
         ),
         allowedCharacters = allowedCharacters.regex,
         modifier = modifier,
diff --git a/designsystem/src/desktopTest/kotlin/org/hisp/dhis/mobile/ui/designsystem/component/InputPhoneNumberTest.kt b/designsystem/src/desktopTest/kotlin/org/hisp/dhis/mobile/ui/designsystem/component/InputPhoneNumberTest.kt
index 3af2ba0d7..c04191e5f 100644
--- a/designsystem/src/desktopTest/kotlin/org/hisp/dhis/mobile/ui/designsystem/component/InputPhoneNumberTest.kt
+++ b/designsystem/src/desktopTest/kotlin/org/hisp/dhis/mobile/ui/designsystem/component/InputPhoneNumberTest.kt
@@ -10,6 +10,7 @@ import androidx.compose.ui.test.assertTextEquals
 import androidx.compose.ui.test.junit4.createComposeRule
 import androidx.compose.ui.test.onNodeWithTag
 import androidx.compose.ui.test.performClick
+import androidx.compose.ui.test.performTextClearance
 import androidx.compose.ui.test.performTextInput
 import androidx.compose.ui.text.input.TextFieldValue
 import org.junit.Rule
@@ -47,14 +48,38 @@ class InputPhoneNumberTest {
     }
 
     @Test
-    fun shouldEnableCallActionButtonAfterInputTextReachesCharacterLimit() {
+    fun shouldAllowAddAndParenthesisInput() {
         rule.setContent {
             var inputValue by remember { mutableStateOf(TextFieldValue()) }
 
             InputPhoneNumber(
                 title = "Phone Number",
                 inputTextFieldValue = inputValue,
-                maxLength = 10,
+                onValueChanged = {
+                    if (it != null) {
+                        inputValue = it
+                    }
+                },
+                onCallActionClicked = {
+                    // no-op
+                },
+                state = InputShellState.UNFOCUSED,
+            )
+        }
+        rule.onNodeWithTag("INPUT_PHONE_NUMBER_FIELD").assertTextEquals("")
+        rule.onNodeWithTag("INPUT_PHONE_NUMBER_FIELD").performTextInput("(+91)-9876543210")
+        rule.onNodeWithTag("INPUT_PHONE_NUMBER_FIELD").assertTextEquals("(+91)-9876543210")
+    }
+
+    @Test
+    fun shouldEnableCallActionButtonWhenTextLengthIsEqualOrGreaterThanMinCharacter() {
+        rule.setContent {
+            var inputValue by remember { mutableStateOf(TextFieldValue()) }
+
+            InputPhoneNumber(
+                title = "Phone Number",
+                inputTextFieldValue = inputValue,
+                minLength = 10,
                 onValueChanged = {
                     if (it != null) {
                         inputValue = it
@@ -67,8 +92,11 @@ class InputPhoneNumberTest {
             )
         }
         rule.onNodeWithTag("CALL_PHONE_NUMBER_BUTTON").assertIsNotEnabled()
-        rule.onNodeWithTag("INPUT_PHONE_NUMBER_FIELD").performTextInput("1111111111")
+        rule.onNodeWithTag("INPUT_PHONE_NUMBER_FIELD").performTextInput("9876543210")
         rule.onNodeWithTag("CALL_PHONE_NUMBER_BUTTON").assertIsEnabled()
+        rule.onNodeWithTag("INPUT_PHONE_NUMBER_FIELD").performTextClearance()
+        rule.onNodeWithTag("INPUT_PHONE_NUMBER_FIELD").performTextInput("987654321")
+        rule.onNodeWithTag("CALL_PHONE_NUMBER_BUTTON").assertIsNotEnabled()
     }
 
     @Test