Protect call to getNotificationItem

libraries/matrix/api/src/main/kotlin/io/element/android/libraries/matrix/api/notification/NotificationService.kt

@@ -17,5 +17,5 @@
 package io.element.android.libraries.matrix.api.notification
 
 interface NotificationService {
-    fun getNotification(userId: String, roomId: String, eventId: String): NotificationData?
+    suspend fun getNotification(userId: String, roomId: String, eventId: String): Result<NotificationData?>
 }

libraries/matrix/impl/src/main/kotlin/io/element/android/libraries/matrix/impl/RustMatrixClient.kt

@@ -68,7 +68,7 @@ class RustMatrixClient constructor(
         client = client,
         dispatchers = dispatchers,
     )
-    private val notificationService = RustNotificationService(baseDirectory)
+    private val notificationService = RustNotificationService(baseDirectory, dispatchers)
     private var slidingSyncUpdateJob: Job? = null
 
     private val clientDelegate = object : ClientDelegate {

libraries/matrix/impl/src/main/kotlin/io/element/android/libraries/matrix/impl/notification/RustNotificationService.kt

@@ -16,23 +16,30 @@
 
 package io.element.android.libraries.matrix.impl.notification
 
+import io.element.android.libraries.core.coroutine.CoroutineDispatchers
 import io.element.android.libraries.matrix.api.notification.NotificationData
 import io.element.android.libraries.matrix.api.notification.NotificationService
+import kotlinx.coroutines.withContext
 import java.io.File
 
 class RustNotificationService(
     private val baseDirectory: File,
+    private val dispatchers: CoroutineDispatchers,
 ) : NotificationService {
     private val notificationMapper: NotificationMapper = NotificationMapper()
 
-    override fun getNotification(userId: String, roomId: String, eventId: String): NotificationData? {
+    override suspend fun getNotification(userId: String, roomId: String, eventId: String): Result<NotificationData?> {
-        return org.matrix.rustcomponents.sdk.NotificationService(
+        return withContext(dispatchers.io) {
-            basePath = File(baseDirectory, "sessions").absolutePath,
+            runCatching {
-            userId = userId
+                org.matrix.rustcomponents.sdk.NotificationService(
-        ).use {
+                    basePath = File(baseDirectory, "sessions").absolutePath,
-            // TODO Not implemented yet, see https://github.com/matrix-org/matrix-rust-sdk/issues/1628
+                    userId = userId
-            it.getNotificationItem(roomId, eventId)?.let { notificationItem ->
+                ).use {
-                notificationMapper.map(notificationItem)
+                    // TODO Not implemented yet, see https://github.com/matrix-org/matrix-rust-sdk/issues/1628
+                    it.getNotificationItem(roomId, eventId)?.let { notificationItem ->
+                        notificationMapper.map(notificationItem)
+                    }
+                }
             }
         }
     }

libraries/matrix/test/src/main/kotlin/io/element/android/libraries/matrix/test/notification/FakeNotificationService.kt

@@ -20,7 +20,7 @@ import io.element.android.libraries.matrix.api.notification.NotificationData
 import io.element.android.libraries.matrix.api.notification.NotificationService
 
 class FakeNotificationService : NotificationService {
-    override fun getNotification(userId: String, roomId: String, eventId: String): NotificationData? {
+    override suspend fun getNotification(userId: String, roomId: String, eventId: String): Result<NotificationData?> {
-        return null
+        return Result.success(null)
     }
 }