feat: handle federated case for user ids in QR code (WPB-10530)

app/src/main/kotlin/com/wire/android/ui/userprofile/qr/SelfQRCodeScreen.kt

@@ -280,7 +280,8 @@ fun PreviewSelfQRCodeContent() {
             SelfQRCodeState(
                 userId = UserId("userId", "wire.com"),
                 handle = "userid",
-                userProfileLink = "https://account.wire.com/user-profile/?id=aaaaaaa-222-3333-4444-55555555"
+                userProfileLink = "wire://user/wire.com/aaaaaaa-222-3333-4444-55555555",
+                userAccountProfileLink = "https://account.wire.com/user-profile/?id=aaaaaaa-222-3333-4444-55555555@wire.com"
             ),
             { "".toUri() },
             { }

app/src/main/kotlin/com/wire/android/ui/userprofile/qr/SelfQRCodeViewModel.kt

@@ -111,7 +111,7 @@ class SelfQRCodeViewModel @Inject constructor(
 
     private fun generateSelfUserUrls(accountsUrl: String): SelfQRCodeState =
         selfQRCodeState.copy(
-            userAccountProfileLink = String.format(BASE_USER_PROFILE_URL, accountsUrl, selfUserId.value),
+            userAccountProfileLink = String.format(BASE_USER_PROFILE_URL, accountsUrl, selfUserId),
             userProfileLink = String.format(DIRECT_BASE_USER_PROFILE_URL, selfUserId.domain, selfUserId.value)
         )
 

app/src/main/kotlin/com/wire/android/util/deeplink/DeepLinkProcessor.kt

@@ -145,22 +145,32 @@
         }
     }
 
+    /**
+     * Format of deeplink to parse: wire://user/domain/user-id
+     */
     private fun getConnectingUserProfile(uri: Uri, switchedAccount: Boolean, accountInfo: AccountInfo.Valid): DeepLinkResult {
-        // todo. handle with domain case, before lastPathSegment. format of deeplink wire://user/domain/user-id
-        return uri.lastPathSegment?.toDefaultQualifiedId(accountInfo.userId.domain)?.let {
-            DeepLinkResult.OpenOtherUserProfile(it, switchedAccount)
-        } ?: return DeepLinkResult.Unknown
+        val segments = uri.pathSegments
+        segments.takeLast(2).let { domainAndUserId ->
+            return if (domainAndUserId.size == 2) {
+                val userId = domainAndUserId[1]
+                val domain = domainAndUserId[0]
+                userId.toDefaultQualifiedId(domain).let {
+                    DeepLinkResult.OpenOtherUserProfile(it, switchedAccount)
+                }
+            } else {
+                uri.lastPathSegment?.toDefaultQualifiedId(accountInfo.userId.domain)?.let {
+                    DeepLinkResult.OpenOtherUserProfile(it, switchedAccount)
+                } ?: DeepLinkResult.Unknown
+            }
+        }
     }
 
     /**
-     * Converts the string to a [QualifiedID] with the current user domain or default, to preserve retro compatibility.
-     * When implementing Milestone 2 this should be replaced with a new qualifiedIdMapper, implementing wire://user/domain/user-id
-     *
-     * - new mapper should follow "domain/user-id" parsing.
+     * Converts the string to a [QualifiedID] with the current user domain or default.
+     * IMPORTANT! This also handles the special case where iOS is sending the ID in uppercase.
      */
     private fun String.toDefaultQualifiedId(currentUserDomain: String?): QualifiedID {
         val domain = currentUserDomain ?: "wire.com"
-        // TODO. This lowercase is important, since web/iOS is sending/handling this as uppercase!!
         return QualifiedID(this.lowercase(), domain)
     }
 

app/src/test/kotlin/com/wire/android/ui/userprofile/qr/SelfQRCodeViewModelTest.kt

@@ -39,7 +39,7 @@ class SelfQRCodeViewModelTest {
         )
 
         assertEquals(
-            expected = "${ServerConfig.STAGING.accounts}/user-profile/?id=${TestUser.SELF_USER.id.value}",
+            expected = "${ServerConfig.STAGING.accounts}/user-profile/?id=${TestUser.SELF_USER.id}",
             actual = viewModel.selfQRCodeState.userAccountProfileLink,
         )
     }

app/src/test/kotlin/com/wire/android/util/DeepLinkProcessorTest.kt

@@ -305,7 +305,7 @@ class DeepLinkProcessorTest {
         val conversationResult = deepLinkProcessor(arrangement.uri)
         assertInstanceOf(DeepLinkResult.OpenOtherUserProfile::class.java, conversationResult)
         assertEquals(
-            DeepLinkResult.OpenOtherUserProfile(UserId("other_user", "domain"), false),
+            DeepLinkResult.OpenOtherUserProfile(UserId("other_user", "other_domain"), false),
             conversationResult
         )
     }
@@ -387,7 +387,7 @@ class DeepLinkProcessorTest {
 
         fun withOtherUserProfileQRDeepLink(userIdToOpen: UserId = OTHER_USER_ID, userId: UserId = CURRENT_USER_ID) = apply {
             coEvery { uri.host } returns DeepLinkProcessor.OPEN_USER_PROFILE_DEEPLINK_HOST
-            coEvery { uri.lastPathSegment } returns userIdToOpen.value
+            coEvery { uri.pathSegments } returns listOf(userIdToOpen.domain, userIdToOpen.value)
             coEvery { uri.getQueryParameter(DeepLinkProcessor.USER_TO_USE_QUERY_PARAM) } returns userId.toString()
         }