fix: keep the composed message when replying [WPB-982] (#2969)

wireapp · May 3, 2024 · a0ae53d · a0ae53d
1 parent 933883e
commit a0ae53d
Show file tree

Hide file tree

Showing 2 changed files with 17 additions and 4 deletions.
diff --git a/.../main/kotlin/com/wire/android/ui/home/messagecomposer/state/MessageComposerStateHolder.kt b/.../main/kotlin/com/wire/android/ui/home/messagecomposer/state/MessageComposerStateHolder.kt
@@ -109,7 +109,6 @@ class MessageComposerStateHolder(
     }
 
     fun toReply(message: UIMessage.Regular) {
-        messageCompositionHolder.clearMessage()
         messageCompositionHolder.setReply(message)
         messageCompositionInputStateHolder.toComposing()
     }

diff --git a/...rc/test/kotlin/com/wire/android/ui/home/messagecomposer/MessageComposerStateHolderTest.kt b/...rc/test/kotlin/com/wire/android/ui/home/messagecomposer/MessageComposerStateHolderTest.kt
@@ -116,9 +116,7 @@ class MessageComposerStateHolderTest {
         runTest {
             // given
             // when
-            state.toReply(
-                message = mockMessageWithText
-            )
+            state.toReply(mockMessageWithText)
 
             // then
             assertEquals(
@@ -133,6 +131,22 @@ class MessageComposerStateHolderTest {
             )
         }
 
+    @Test
+    fun `given some message was being composed, when setting toReply, then input continues with the current text`() = runTest {
+        // given
+        val currentTextField = TextFieldValue("Potato")
+        messageCompositionHolder.setMessageText(currentTextField, {}, {}, {})
+
+        // when
+        state.toReply(mockMessageWithText)
+
+        // then
+        assertEquals(
+            currentTextField.text,
+            messageCompositionHolder.messageComposition.value.messageTextFieldValue.text
+        )
+    }
+
     @Test
     fun `given state, when input focus change to false, then clear focus`() = runTest {
         // given