Add test on RoomListFactory

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

@@ -25,21 +25,21 @@ import org.matrix.rustcomponents.sdk.RoomListEntriesDynamicFilterKind
 import org.matrix.rustcomponents.sdk.RoomListLoadingState
 import org.matrix.rustcomponents.sdk.RoomListService
 import kotlin.coroutines.CoroutineContext
-import kotlin.coroutines.EmptyCoroutineContext
 import org.matrix.rustcomponents.sdk.RoomList as InnerRoomList
 
 internal class RoomListFactory(
     private val innerRoomListService: RoomListService,
     private val sessionCoroutineScope: CoroutineScope,
-    private val roomSummaryDetailsFactory: RoomSummaryDetailsFactory = RoomSummaryDetailsFactory(),
 ) {
+    private val roomSummaryDetailsFactory: RoomSummaryDetailsFactory = RoomSummaryDetailsFactory()
+
     /**
      * Creates a room list that can be used to load more rooms and filter them dynamically.
      */
     fun createRoomList(
         pageSize: Int,
+        coroutineContext: CoroutineContext,
         coroutineScope: CoroutineScope = sessionCoroutineScope,
-        coroutineContext: CoroutineContext = EmptyCoroutineContext,
         initialFilter: RoomListFilter = RoomListFilter.all(),
         innerProvider: suspend () -> InnerRoomList
     ): DynamicRoomList {

libraries/matrix/impl/src/test/kotlin/io/element/android/libraries/matrix/impl/roomlist/RoomListFactoryTest.kt

@@ -0,0 +1,30 @@
+/*
+ * Copyright 2024 New Vector Ltd.
+ *
+ * SPDX-License-Identifier: AGPL-3.0-only
+ * Please see LICENSE in the repository root for full details.
+ */
+
+package io.element.android.libraries.matrix.impl.roomlist
+
+import io.element.android.libraries.matrix.impl.fixtures.fakes.FakeRustRoomList
+import io.element.android.libraries.matrix.impl.fixtures.fakes.FakeRustRoomListService
+import io.element.android.tests.testutils.runCancellableScopeTest
+import org.junit.Test
+import kotlin.coroutines.EmptyCoroutineContext
+
+class RoomListFactoryTest {
+    @Test
+    fun `createRoomList should work`() = runCancellableScopeTest {
+        val sut = RoomListFactory(
+            innerRoomListService = FakeRustRoomListService(),
+            sessionCoroutineScope = it,
+        )
+        sut.createRoomList(
+            pageSize = 10,
+            coroutineContext = EmptyCoroutineContext,
+        ) {
+            FakeRustRoomList()
+        }
+    }
+}