Revert "fix(migration): select proteus conversationId for mls migrati…

…on in al…" (#3008) This reverts commit 5cad6c8.
wireapp · Sep 14, 2024 · 9eafc77 · 9eafc77
1 parent 28a9dc3
commit 9eafc77
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 22 deletions.
diff --git a/persistence/src/commonMain/db_user/com/wire/kalium/persistence/Conversations.sq b/persistence/src/commonMain/db_user/com/wire/kalium/persistence/Conversations.sq
@@ -323,8 +323,7 @@ WHERE qualifiedId = (SELECT active_one_on_one_conversation_id FROM User WHERE qu
 selectOneOnOneConversationIdsByProtocol:
 SELECT Member.conversation FROM Member
 JOIN Conversation ON Conversation.qualified_id = Member.conversation
-WHERE Conversation.type = 'ONE_ON_ONE' AND Conversation.protocol = :protocol AND Member.user = :user
-ORDER BY Member.conversation ASC;
+WHERE Conversation.type = 'ONE_ON_ONE' AND Conversation.protocol = :protocol AND Member.user = :user;
 
 getConversationIdByGroupId:
 SELECT qualified_id FROM Conversation WHERE mls_group_id = ?;

diff --git a/persistence/src/commonTest/kotlin/com/wire/kalium/persistence/dao/ConversationDAOTest.kt b/persistence/src/commonTest/kotlin/com/wire/kalium/persistence/dao/ConversationDAOTest.kt
@@ -1259,26 +1259,6 @@ class ConversationDAOTest : BaseDatabaseTest() {
         )
     }
 
-    @Test
-    fun givenMultipleOneOnOneProteusConversationExisting_whenGettingOneOnOneConversationId_thenShouldReturnAlphabeticallyOrderedConversation() =
-        runTest {
-            // given
-            val conversationA = conversationEntity1.copy(id = QualifiedIDEntity("a", "wire.com"))
-            val conversationB = conversationEntity1.copy(id = QualifiedIDEntity("b", "wire.com"))
-
-            userDAO.upsertUser(user1)
-            conversationDAO.insertConversation(conversationB)
-            conversationDAO.insertConversation(conversationA)
-            memberDAO.insertMember(member1, conversationB.id)
-            memberDAO.insertMember(member1, conversationA.id)
-
-            // then
-            assertEquals(
-                conversationA.id,
-                conversationDAO.getOneOnOneConversationIdsWithOtherUser(user1.id, protocol = ConversationEntity.Protocol.PROTEUS).first()
-            )
-        }
-
     @Test
     fun givenNoMLSConversationExistsForGivenClients_whenGettingE2EIClientInfoByClientId_thenReturnsNull() = runTest {
         // given