Skip to content

Commit

Permalink
fix: Update legal hold status class path (#2496)
Browse files Browse the repository at this point in the history
  • Loading branch information
ohassine committed Dec 4, 2023
1 parent f3108ca commit 7e67653
Show file tree
Hide file tree
Showing 13 changed files with 29 additions and 19 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -45,7 +45,7 @@ class GlobalObserversManager @Inject constructor(
dispatcherProvider: DispatcherProvider,
@KaliumCoreLogic private val coreLogic: CoreLogic,
private val notificationManager: WireNotificationManager,
private val notificationChannelsManager: NotificationChannelsManager,
private val notificationChannelsManager: NotificationChannelsManager
) {
private val scope = CoroutineScope(SupervisorJob() + dispatcherProvider.io())

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -107,7 +107,8 @@ class MigrationMapper @Inject constructor() {
archived = false,
archivedDateTime = null,
mlsVerificationStatus = Conversation.VerificationStatus.NOT_VERIFIED,
proteusVerificationStatus = Conversation.VerificationStatus.NOT_VERIFIED
proteusVerificationStatus = Conversation.VerificationStatus.NOT_VERIFIED,
legalHoldStatus = Conversation.LegalHoldStatus.DISABLED
)
}
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -31,7 +31,7 @@ import com.wire.android.util.CurrentScreen
import com.wire.android.util.CurrentScreenManager
import com.wire.kalium.logic.CoreLogic
import com.wire.kalium.logic.data.call.Call
import com.wire.kalium.logic.data.conversation.LegalHoldStatus
import com.wire.kalium.logic.data.user.LegalHoldStatus
import com.wire.kalium.logic.data.sync.SyncState
import com.wire.kalium.logic.data.user.UserId
import com.wire.kalium.logic.feature.legalhold.ObserveLegalHoldRequestUseCaseResult
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -53,7 +53,7 @@ import com.wire.kalium.logic.data.conversation.ConversationDetails.Connection
import com.wire.kalium.logic.data.conversation.ConversationDetails.Group
import com.wire.kalium.logic.data.conversation.ConversationDetails.OneOne
import com.wire.kalium.logic.data.conversation.ConversationDetails.Self
import com.wire.kalium.logic.data.conversation.LegalHoldStatus
import com.wire.kalium.logic.data.user.LegalHoldStatus
import com.wire.kalium.logic.data.conversation.MutedConversationStatus
import com.wire.kalium.logic.data.conversation.UnreadEventCount
import com.wire.kalium.logic.data.id.ConversationId
Expand Down
15 changes: 10 additions & 5 deletions app/src/test/kotlin/com/wire/android/framework/TestConversation.kt
Original file line number Diff line number Diff line change
Expand Up @@ -53,7 +53,8 @@ object TestConversation {
archived = false,
archivedDateTime = null,
mlsVerificationStatus = Conversation.VerificationStatus.NOT_VERIFIED,
proteusVerificationStatus = Conversation.VerificationStatus.NOT_VERIFIED
proteusVerificationStatus = Conversation.VerificationStatus.NOT_VERIFIED,
legalHoldStatus = Conversation.LegalHoldStatus.DISABLED
)
val SELF = Conversation(
ID.copy(value = "SELF ID"),
Expand All @@ -75,7 +76,8 @@ object TestConversation {
archived = false,
archivedDateTime = null,
mlsVerificationStatus = Conversation.VerificationStatus.NOT_VERIFIED,
proteusVerificationStatus = Conversation.VerificationStatus.NOT_VERIFIED
proteusVerificationStatus = Conversation.VerificationStatus.NOT_VERIFIED,
legalHoldStatus = Conversation.LegalHoldStatus.DISABLED
)

fun GROUP(protocolInfo: ProtocolInfo = ProtocolInfo.Proteus) = Conversation(
Expand All @@ -98,7 +100,8 @@ object TestConversation {
archived = false,
archivedDateTime = null,
mlsVerificationStatus = Conversation.VerificationStatus.NOT_VERIFIED,
proteusVerificationStatus = Conversation.VerificationStatus.NOT_VERIFIED
proteusVerificationStatus = Conversation.VerificationStatus.NOT_VERIFIED,
legalHoldStatus = Conversation.LegalHoldStatus.DISABLED
)

fun one_on_one(convId: ConversationId) = Conversation(
Expand All @@ -121,7 +124,8 @@ object TestConversation {
archived = false,
archivedDateTime = null,
mlsVerificationStatus = Conversation.VerificationStatus.NOT_VERIFIED,
proteusVerificationStatus = Conversation.VerificationStatus.NOT_VERIFIED
proteusVerificationStatus = Conversation.VerificationStatus.NOT_VERIFIED,
legalHoldStatus = Conversation.LegalHoldStatus.DISABLED
)

val USER_1 = UserId("member1", "domainMember")
Expand Down Expand Up @@ -151,6 +155,7 @@ object TestConversation {
archived = false,
archivedDateTime = null,
mlsVerificationStatus = Conversation.VerificationStatus.NOT_VERIFIED,
proteusVerificationStatus = Conversation.VerificationStatus.NOT_VERIFIED
proteusVerificationStatus = Conversation.VerificationStatus.NOT_VERIFIED,
legalHoldStatus = Conversation.LegalHoldStatus.DISABLED
)
}
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,7 @@ package com.wire.android.framework
import com.wire.kalium.logic.data.conversation.Conversation
import com.wire.kalium.logic.data.conversation.Conversation.ProtocolInfo
import com.wire.kalium.logic.data.conversation.ConversationDetails
import com.wire.kalium.logic.data.conversation.LegalHoldStatus
import com.wire.kalium.logic.data.user.LegalHoldStatus
import com.wire.kalium.logic.data.user.type.UserType

object TestConversationDetails {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -45,7 +45,7 @@ import com.wire.kalium.logic.CoreFailure
import com.wire.kalium.logic.configuration.FileSharingStatus
import com.wire.kalium.logic.data.conversation.Conversation
import com.wire.kalium.logic.data.conversation.ConversationDetails
import com.wire.kalium.logic.data.conversation.LegalHoldStatus
import com.wire.kalium.logic.data.user.LegalHoldStatus
import com.wire.kalium.logic.data.id.ConversationId
import com.wire.kalium.logic.data.message.SelfDeletionTimer
import com.wire.kalium.logic.data.sync.SyncState
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -35,7 +35,7 @@ import com.wire.android.ui.home.conversationslist.model.DialogState
import com.wire.android.ui.navArgs
import com.wire.kalium.logic.data.conversation.Conversation
import com.wire.kalium.logic.data.conversation.ConversationDetails
import com.wire.kalium.logic.data.conversation.LegalHoldStatus
import com.wire.kalium.logic.data.user.LegalHoldStatus
import com.wire.kalium.logic.data.conversation.MutedConversationStatus
import com.wire.kalium.logic.data.id.ConversationId
import com.wire.kalium.logic.data.id.TeamId
Expand Down Expand Up @@ -593,7 +593,8 @@ class GroupConversationDetailsViewModelTest {
archived = false,
archivedDateTime = null,
mlsVerificationStatus = Conversation.VerificationStatus.NOT_VERIFIED,
proteusVerificationStatus = Conversation.VerificationStatus.NOT_VERIFIED
proteusVerificationStatus = Conversation.VerificationStatus.NOT_VERIFIED,
legalHoldStatus = Conversation.LegalHoldStatus.DISABLED
),
legalHoldStatus = LegalHoldStatus.DISABLED,
hasOngoingCall = false,
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,7 @@ import com.wire.android.framework.TestUser
import com.wire.android.ui.home.conversations.ConversationNavArgs
import com.wire.android.ui.navArgs
import com.wire.kalium.logic.data.conversation.ConversationDetails
import com.wire.kalium.logic.data.conversation.LegalHoldStatus
import com.wire.kalium.logic.data.user.LegalHoldStatus
import com.wire.kalium.logic.data.id.ConversationId
import com.wire.kalium.logic.data.user.type.UserType
import com.wire.kalium.logic.feature.conversation.ObserveConversationDetailsUseCase
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -36,7 +36,7 @@ import com.wire.kalium.logic.CoreFailure
import com.wire.kalium.logic.data.conversation.Conversation
import com.wire.kalium.logic.data.conversation.ConversationDetails
import com.wire.kalium.logic.data.conversation.ConversationDetails.OneOne
import com.wire.kalium.logic.data.conversation.LegalHoldStatus
import com.wire.kalium.logic.data.user.LegalHoldStatus
import com.wire.kalium.logic.data.conversation.MutedConversationStatus.AllAllowed
import com.wire.kalium.logic.data.id.QualifiedID
import com.wire.kalium.logic.data.user.ConnectionState
Expand Down Expand Up @@ -306,7 +306,8 @@ class MediaGalleryViewModelTest {
archived = false,
archivedDateTime = null,
mlsVerificationStatus = Conversation.VerificationStatus.NOT_VERIFIED,
proteusVerificationStatus = Conversation.VerificationStatus.NOT_VERIFIED
proteusVerificationStatus = Conversation.VerificationStatus.NOT_VERIFIED,
legalHoldStatus = Conversation.LegalHoldStatus.DISABLED
),
otherUser = OtherUser(
QualifiedID("other-user-id", "domain-id"),
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -160,7 +160,8 @@ internal class NewConversationViewModelArrangement {
archived = false,
archivedDateTime = null,
mlsVerificationStatus = Conversation.VerificationStatus.NOT_VERIFIED,
proteusVerificationStatus = Conversation.VerificationStatus.NOT_VERIFIED
proteusVerificationStatus = Conversation.VerificationStatus.NOT_VERIFIED,
legalHoldStatus = Conversation.LegalHoldStatus.DISABLED
)

val PUBLIC_USER = OtherUser(
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -248,7 +248,8 @@ class OtherUserProfileScreenViewModelTest {
archived = false,
archivedDateTime = null,
mlsVerificationStatus = Conversation.VerificationStatus.NOT_VERIFIED,
proteusVerificationStatus = Conversation.VerificationStatus.NOT_VERIFIED
proteusVerificationStatus = Conversation.VerificationStatus.NOT_VERIFIED,
legalHoldStatus = Conversation.LegalHoldStatus.DISABLED
)
val CONVERSATION_ROLE_DATA = ConversationRoleData(
"some_name",
Expand Down
2 changes: 1 addition & 1 deletion kalium
Submodule kalium updated 45 files
+1 βˆ’1 .github/workflows/codestyle.yml
+1 βˆ’1 .github/workflows/gradle-android-instrumented-tests.yml
+1 βˆ’1 .github/workflows/gradle-android-unit-tests.yml
+7 βˆ’2 .github/workflows/gradle-ios-tests.yml
+1 βˆ’1 .github/workflows/gradle-jvm-tests.yml
+2 βˆ’1 logic/src/commonMain/kotlin/com/wire/kalium/logic/data/connection/ConnectionRepository.kt
+6 βˆ’2 logic/src/commonMain/kotlin/com/wire/kalium/logic/data/conversation/Conversation.kt
+18 βˆ’5 logic/src/commonMain/kotlin/com/wire/kalium/logic/data/conversation/ConversationMapper.kt
+25 βˆ’0 logic/src/commonMain/kotlin/com/wire/kalium/logic/data/conversation/ConversationRepository.kt
+1 βˆ’0 logic/src/commonMain/kotlin/com/wire/kalium/logic/data/conversation/LegalHoldStatusMapper.kt
+1 βˆ’1 logic/src/commonMain/kotlin/com/wire/kalium/logic/data/team/TeamRepository.kt
+1 βˆ’1 logic/src/commonMain/kotlin/com/wire/kalium/logic/data/user/LegalHoldStatus.kt
+3 βˆ’1 logic/src/commonMain/kotlin/com/wire/kalium/logic/data/web/WebMappers.kt
+2 βˆ’1 logic/src/commonMain/kotlin/com/wire/kalium/logic/feature/UserSessionScope.kt
+10 βˆ’1 logic/src/commonMain/kotlin/com/wire/kalium/logic/feature/conversation/ConversationScope.kt
+65 βˆ’0 logic/src/commonMain/kotlin/com/wire/kalium/logic/feature/conversation/NotifyConversationIsOpenUseCase.kt
+1 βˆ’1 ...src/commonMain/kotlin/com/wire/kalium/logic/feature/legalhold/FetchLegalHoldForSelfUserFromRemoteUseCase.kt
+1 βˆ’1 logic/src/commonTest/kotlin/com/wire/kalium/logic/data/call/CallRepositoryTest.kt
+34 βˆ’0 logic/src/commonTest/kotlin/com/wire/kalium/logic/data/conversation/ConversationRepositoryTest.kt
+1 βˆ’1 logic/src/commonTest/kotlin/com/wire/kalium/logic/data/team/TeamRepositoryTest.kt
+3 βˆ’2 .../src/commonTest/kotlin/com/wire/kalium/logic/feature/call/usecase/EndCallOnConversationChangeUseCaseTest.kt
+104 βˆ’0 logic/src/commonTest/kotlin/com/wire/kalium/logic/feature/conversation/NotifyConversationIsOpenUseCaseTest.kt
+1 βˆ’3 ...c/src/commonTest/kotlin/com/wire/kalium/logic/feature/conversation/ObserveConversationDetailsUseCaseTest.kt
+1 βˆ’1 ...c/commonTest/kotlin/com/wire/kalium/logic/feature/conversation/ObserveConversationListDetailsUseCaseTest.kt
+2 βˆ’1 logic/src/commonTest/kotlin/com/wire/kalium/logic/feature/conversation/UpdateConversationAccessUseCaseTest.kt
+1 βˆ’1 ...commonTest/kotlin/com/wire/kalium/logic/feature/legalhold/FetchLegalHoldForSelfUserFromRemoteUseCaseTest.kt
+16 βˆ’7 logic/src/commonTest/kotlin/com/wire/kalium/logic/framework/TestConversation.kt
+1 βˆ’1 logic/src/commonTest/kotlin/com/wire/kalium/logic/framework/TestConversationDetails.kt
+1 βˆ’1 logic/src/commonTest/kotlin/com/wire/kalium/logic/sync/slow/SlowSyncWorkerTest.kt
+16 βˆ’5 persistence/src/commonMain/db_user/com/wire/kalium/persistence/Conversations.sq
+2 βˆ’1 persistence/src/commonMain/db_user/com/wire/kalium/persistence/Members.sq
+112 βˆ’0 persistence/src/commonMain/db_user/migrations/66.sqm
+2 βˆ’0 persistence/src/commonMain/kotlin/com/wire/kalium/persistence/dao/conversation/ConversationDAO.kt
+13 βˆ’0 persistence/src/commonMain/kotlin/com/wire/kalium/persistence/dao/conversation/ConversationDAOImpl.kt
+3 βˆ’1 persistence/src/commonMain/kotlin/com/wire/kalium/persistence/dao/conversation/ConversationEntity.kt
+6 βˆ’3 persistence/src/commonMain/kotlin/com/wire/kalium/persistence/dao/conversation/ConversationMapper.kt
+2 βˆ’2 persistence/src/commonMain/kotlin/com/wire/kalium/persistence/dao/conversation/ConversationViewEntity.kt
+2 βˆ’1 persistence/src/commonMain/kotlin/com/wire/kalium/persistence/db/TableMapper.kt
+4 βˆ’2 persistence/src/commonTest/kotlin/com/wire/kalium/persistence/backup/DatabaseImporterTest.kt
+12 βˆ’6 persistence/src/commonTest/kotlin/com/wire/kalium/persistence/backup/UserDatabaseDataGenerator.kt
+13 βˆ’6 persistence/src/commonTest/kotlin/com/wire/kalium/persistence/dao/ConversationDAOTest.kt
+2 βˆ’1 persistence/src/commonTest/kotlin/com/wire/kalium/persistence/dao/ConversationMetaDataDAOTest.kt
+2 βˆ’1 persistence/src/commonTest/kotlin/com/wire/kalium/persistence/dao/client/ClientDAOTest.kt
+4 βˆ’2 persistence/src/commonTest/kotlin/com/wire/kalium/persistence/utils/stubs/ConversationStubs.kt
+10 βˆ’5 persistence/src/commonTest/kotlin/com/wire/kalium/persistence/utils/stubs/TestStubs.kt

0 comments on commit 7e67653

Please sign in to comment.