diff --git a/test/ui-test/src/screens/StatusChatScreen.py b/test/ui-test/src/screens/StatusChatScreen.py index e0bcf20b3f..10a6e4df77 100644 --- a/test/ui-test/src/screens/StatusChatScreen.py +++ b/test/ui-test/src/screens/StatusChatScreen.py @@ -196,8 +196,8 @@ class StatusChatScreen: self.send_message(message) def edit_message_at_index(self, index: int, message: str): - message_object_to_edit = get_obj(ChatComponents.CHAT_LOG.value).itemAtIndex(int(index)) - hover_obj(message_object_to_edit) + message_object_to_edit = wait_and_get_obj(ChatComponents.CHAT_LOG.value).itemAtIndex(int(index)) + move_mouse_over_object(message_object_to_edit) click_obj_by_name(ChatComponents.EDIT_MESSAGE_BUTTON.value) wait_for_object_and_type(ChatComponents.EDIT_MESSAGE_TEXTAREA.value, "") type(ChatComponents.EDIT_MESSAGE_TEXTAREA.value, message) @@ -340,6 +340,10 @@ class StatusChatScreen: verify_values_equal(str(last_message_reply_details_obj.replyDetails.sender.id), str(last_message_obj.senderId), "Message sender ID doesn't match reply message sender ID") + def verify_last_message_is_edited(self, message: str): + last_message_obj = self.get_message_at_index(0) + verify(bool(last_message_obj.isEdited), "Message is not marked as edited") + def verify_last_message_sent(self, message: str): # Get the message text # We don't search for StatusTextMessage_chatText directly, because there're 2 instances of it in a reply message diff --git a/test/ui-test/testSuites/suite_messaging/shared/steps/chatSteps.py b/test/ui-test/testSuites/suite_messaging/shared/steps/chatSteps.py index fd75033549..e9db4b39ff 100644 --- a/test/ui-test/testSuites/suite_messaging/shared/steps/chatSteps.py +++ b/test/ui-test/testSuites/suite_messaging/shared/steps/chatSteps.py @@ -194,8 +194,8 @@ def step(context, reply, message): @Then("the chat message \"|any|\" is displayed as an edited one") def step(context, message): - # TODO: Check last message is an edited one. _statusChat.verify_last_message_sent(message) + _statusChat.verify_last_message_is_edited(message) @Then("the last message displayed is not \"|any|\"") def step(context, message): diff --git a/test/ui-test/testSuites/suite_messaging/tst_ChatFlow/test.feature b/test/ui-test/testSuites/suite_messaging/tst_ChatFlow/test.feature index b7041ef35a..c5fdeaf9b1 100644 --- a/test/ui-test/testSuites/suite_messaging/tst_ChatFlow/test.feature +++ b/test/ui-test/testSuites/suite_messaging/tst_ChatFlow/test.feature @@ -39,7 +39,6 @@ Feature: Status Desktop Chat Basic Flows Scenario Outline: The user can edit a message Given the user sends a chat message "Edit me" When the user edits the message at index 0 and changes it to "" - # TODO: Check last message is an edited one, now just checking the last message is the expected one but could not be an edited one. Then the chat message "" is displayed as an edited one Examples: | edited |