Update regarding SDK change on verification data

features/verifysession/impl/src/main/kotlin/io/element/android/features/verifysession/impl/VerifySelfSessionPresenter.kt

@@ -97,7 +97,7 @@ class VerifySelfSessionPresenter @Inject constructor(
                     is StateMachineState.Verifying.Replying -> Async.Loading()
                     else -> Async.Uninitialized
                 }
-                VerifySelfSessionState.VerificationStep.Verifying(machineState.emojis, async)
+                VerifySelfSessionState.VerificationStep.Verifying(machineState.data, async)
             }
 
             StateMachineState.Completed -> {
@@ -116,7 +116,7 @@ class VerifySelfSessionPresenter @Inject constructor(
                     stateMachine.dispatch(VerifySelfSessionStateMachine.Event.DidStartSasVerification)
                 }
                 is VerificationFlowState.ReceivedVerificationData -> {
-                    stateMachine.dispatch(VerifySelfSessionStateMachine.Event.DidReceiveChallenge(verificationAttemptState.emoji))
+                    stateMachine.dispatch(VerifySelfSessionStateMachine.Event.DidReceiveChallenge(verificationAttemptState.data))
                 }
                 VerificationFlowState.Finished -> {
                     stateMachine.dispatch(VerifySelfSessionStateMachine.Event.DidAcceptChallenge)

features/verifysession/impl/src/main/kotlin/io/element/android/features/verifysession/impl/VerifySelfSessionState.kt

@@ -19,7 +19,7 @@ package io.element.android.features.verifysession.impl
 import androidx.compose.runtime.Immutable
 import androidx.compose.runtime.Stable
 import io.element.android.libraries.architecture.Async
-import io.element.android.libraries.matrix.api.verification.VerificationEmoji
+import io.element.android.libraries.matrix.api.verification.SessionVerificationData
 
 @Immutable
 data class VerifySelfSessionState(
@@ -33,7 +33,7 @@ data class VerifySelfSessionState(
         data object Canceled : VerificationStep
         data object AwaitingOtherDeviceResponse : VerificationStep
         data object Ready : VerificationStep
-        data class Verifying(val emojiList: List<VerificationEmoji>, val state: Async<Unit>) : VerificationStep
+        data class Verifying(val data: SessionVerificationData, val state: Async<Unit>) : VerificationStep
         data object Completed : VerificationStep
     }
 }

features/verifysession/impl/src/main/kotlin/io/element/android/features/verifysession/impl/VerifySelfSessionStateMachine.kt

@@ -20,8 +20,8 @@
 package io.element.android.features.verifysession.impl
 
 import com.freeletics.flowredux.dsl.FlowReduxStateMachine
+import io.element.android.libraries.matrix.api.verification.SessionVerificationData
 import io.element.android.libraries.matrix.api.verification.SessionVerificationService
-import io.element.android.libraries.matrix.api.verification.VerificationEmoji
 import kotlinx.coroutines.ExperimentalCoroutinesApi
 import javax.inject.Inject
 import com.freeletics.flowredux.dsl.State as MachineState
@@ -70,15 +70,15 @@ class VerifySelfSessionStateMachine @Inject constructor(
             }
             inState<State.SasVerificationStarted> {
                 on { event: Event.DidReceiveChallenge, state: MachineState<State.SasVerificationStarted> ->
-                    state.override { State.Verifying.ChallengeReceived(event.emojis) }
+                    state.override { State.Verifying.ChallengeReceived(event.data) }
                 }
             }
             inState<State.Verifying.ChallengeReceived> {
                 on { _: Event.AcceptChallenge, state: MachineState<State.Verifying.ChallengeReceived> ->
-                    state.override { State.Verifying.Replying(state.snapshot.emojis, accept = true) }
+                    state.override { State.Verifying.Replying(state.snapshot.data, accept = true) }
                 }
                 on { _: Event.DeclineChallenge, state: MachineState<State.Verifying.ChallengeReceived> ->
-                    state.override { State.Verifying.Replying(state.snapshot.emojis, accept = false) }
+                    state.override { State.Verifying.Replying(state.snapshot.data, accept = false) }
                 }
             }
             inState<State.Verifying.Replying> {
@@ -139,12 +139,12 @@ class VerifySelfSessionStateMachine @Inject constructor(
         /** A SaS verification flow has been started. */
         data object SasVerificationStarted : State
 
-        sealed class Verifying(open val emojis: List<VerificationEmoji>) : State {
+        sealed class Verifying(open val data: SessionVerificationData) : State {
             /** Verification accepted and emojis received. */
-            data class ChallengeReceived(override val emojis: List<VerificationEmoji>) : Verifying(emojis)
+            data class ChallengeReceived(override val data: SessionVerificationData) : Verifying(data)
 
             /** Replying to a verification challenge. */
-            data class Replying(override val emojis: List<VerificationEmoji>, val accept: Boolean) : Verifying(emojis)
+            data class Replying(override val data: SessionVerificationData, val accept: Boolean) : Verifying(data)
         }
 
         /** The verification is being canceled. */
@@ -170,8 +170,8 @@ class VerifySelfSessionStateMachine @Inject constructor(
         /** Started a SaS verification flow. */
         data object DidStartSasVerification : Event
 
-        /** Has received emojis. */
-        data class DidReceiveChallenge(val emojis: List<VerificationEmoji>) : Event
+        /** Has received data. */
+        data class DidReceiveChallenge(val data: SessionVerificationData) : Event
 
         /** Emojis match. */
         data object AcceptChallenge : Event

features/verifysession/impl/src/main/kotlin/io/element/android/features/verifysession/impl/VerifySelfSessionStateProvider.kt

@@ -18,6 +18,7 @@ package io.element.android.features.verifysession.impl
 
 import androidx.compose.ui.tooling.preview.PreviewParameterProvider
 import io.element.android.libraries.architecture.Async
+import io.element.android.libraries.matrix.api.verification.SessionVerificationData
 import io.element.android.libraries.matrix.api.verification.VerificationEmoji
 
 open class VerifySelfSessionStateProvider : PreviewParameterProvider<VerifySelfSessionState> {
@@ -28,10 +29,10 @@ open class VerifySelfSessionStateProvider : PreviewParameterProvider<VerifySelfS
                 verificationFlowStep = VerifySelfSessionState.VerificationStep.AwaitingOtherDeviceResponse
             ),
             aVerifySelfSessionState().copy(
-                verificationFlowStep = VerifySelfSessionState.VerificationStep.Verifying(aVerificationEmojiList(), Async.Uninitialized)
+                verificationFlowStep = VerifySelfSessionState.VerificationStep.Verifying(aSessionVerificationData(), Async.Uninitialized)
             ),
             aVerifySelfSessionState().copy(
-                verificationFlowStep = VerifySelfSessionState.VerificationStep.Verifying(aVerificationEmojiList(), Async.Loading())
+                verificationFlowStep = VerifySelfSessionState.VerificationStep.Verifying(aSessionVerificationData(), Async.Loading())
             ),
             aVerifySelfSessionState().copy(
                 verificationFlowStep = VerifySelfSessionState.VerificationStep.Canceled
@@ -43,17 +44,28 @@ open class VerifySelfSessionStateProvider : PreviewParameterProvider<VerifySelfS
         )
 }
 
-fun aVerifySelfSessionState() = VerifySelfSessionState(
+private fun aSessionVerificationData(
+    emojiList: List<VerificationEmoji> = aVerificationEmojiList(),
+    decimals: List<Int> = emptyList(),
+): SessionVerificationData {
+    return if (emojiList.isEmpty()) {
+        SessionVerificationData.Decimals(decimals)
+    } else {
+        SessionVerificationData.Emojis(emojiList)
+    }
+}
+
+private fun aVerifySelfSessionState() = VerifySelfSessionState(
     verificationFlowStep = VerifySelfSessionState.VerificationStep.Initial,
     eventSink = {},
 )
 
-fun aVerificationEmojiList() = listOf(
-    VerificationEmoji(27, "Pizza"),
-    VerificationEmoji(54, "Rocket"),
-    VerificationEmoji(54, "Rocket"),
-    VerificationEmoji(42, "Book"),
-    VerificationEmoji(48, "Hammer"),
-    VerificationEmoji(48, "Hammer"),
-    VerificationEmoji(63, "Pin"),
+private fun aVerificationEmojiList() = listOf(
+    VerificationEmoji(number = 27, emoji = "🍕", description = "Pizza"),
+    VerificationEmoji(number = 54, emoji = "🚀", description = "Rocket"),
+    VerificationEmoji(number = 54, emoji = "🚀", description = "Rocket"),
+    VerificationEmoji(number = 42, emoji = "📕", description = "Book"),
+    VerificationEmoji(number = 48, emoji = "🔨", description = "Hammer"),
+    VerificationEmoji(number = 48, emoji = "🔨", description = "Hammer"),
+    VerificationEmoji(number = 63, emoji = "📌", description = "Pin"),
 )

features/verifysession/impl/src/main/kotlin/io/element/android/features/verifysession/impl/VerifySelfSessionView.kt

@@ -52,6 +52,7 @@ import io.element.android.libraries.designsystem.theme.components.Button
 import io.element.android.libraries.designsystem.theme.components.CircularProgressIndicator
 import io.element.android.libraries.designsystem.theme.components.Text
 import io.element.android.libraries.designsystem.theme.components.TextButton
+import io.element.android.libraries.matrix.api.verification.SessionVerificationData
 import io.element.android.libraries.matrix.api.verification.VerificationEmoji
 import io.element.android.libraries.ui.strings.CommonStrings
 import io.element.android.features.verifysession.impl.VerifySelfSessionState.VerificationStep as FlowStep
@@ -141,17 +142,22 @@ private fun ContentWaiting(modifier: Modifier = Modifier) {
 
 @Composable
 private fun ContentVerifying(verificationFlowStep: FlowStep.Verifying, modifier: Modifier = Modifier) {
-    // We want each row to have up to 4 emojis
-    val rows = verificationFlowStep.emojiList.chunked(4)
-    Column(modifier = modifier.fillMaxWidth()) {
-        for ((rowIndex, emojis) in rows.withIndex()) {
-            // Vertical spacing between rows
-            if (rowIndex > 0) {
-                Spacer(modifier = Modifier.height(40.dp))
-            }
-            Row(modifier = Modifier.fillMaxWidth(), horizontalArrangement = Arrangement.SpaceEvenly) {
-                for (emoji in emojis) {
-                    EmojiItemView(emoji = emoji, modifier = Modifier.widthIn(max = 60.dp))
+    when (verificationFlowStep.data) {
+        is SessionVerificationData.Decimals -> Unit // TODO Render decimals
+        is SessionVerificationData.Emojis -> {
+            // We want each row to have up to 4 emojis
+            val rows = verificationFlowStep.data.emojis.chunked(4)
+            Column(modifier = modifier.fillMaxWidth()) {
+                for ((rowIndex, emojis) in rows.withIndex()) {
+                    // Vertical spacing between rows
+                    if (rowIndex > 0) {
+                        Spacer(modifier = Modifier.height(40.dp))
+                    }
+                    Row(modifier = Modifier.fillMaxWidth(), horizontalArrangement = Arrangement.SpaceEvenly) {
+                        for (emoji in emojis) {
+                            EmojiItemView(emoji = emoji, modifier = Modifier.widthIn(max = 60.dp))
+                        }
+                    }
                 }
             }
         }
@@ -160,7 +166,7 @@ private fun ContentVerifying(verificationFlowStep: FlowStep.Verifying, modifier:
 
 @Composable
 private fun EmojiItemView(emoji: VerificationEmoji, modifier: Modifier = Modifier) {
-    val emojiResource = emoji.code.toEmojiResource()
+    val emojiResource = emoji.number.toEmojiResource()
     Column(horizontalAlignment = Alignment.CenterHorizontally, modifier = modifier) {
         Image(
             modifier = Modifier.size(48.dp),