Restore disabled tests with FFI fakes

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

@@ -23,11 +23,9 @@ import io.element.android.services.toolbox.test.systemclock.FakeSystemClock
 import io.element.android.tests.testutils.testCoroutineDispatchers
 import kotlinx.coroutines.test.TestScope
 import kotlinx.coroutines.test.runTest
-import org.junit.Ignore
 import org.junit.Test
 import java.io.File
 
-@Ignore("JNA direct mapping has broken unit tests with FFI fakes")
 class RustMatrixClientFactoryTest {
     @Test
     fun test() = runTest {

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

@@ -31,13 +31,11 @@ import kotlinx.coroutines.ExperimentalCoroutinesApi
 import kotlinx.coroutines.test.TestScope
 import kotlinx.coroutines.test.advanceUntilIdle
 import kotlinx.coroutines.test.runTest
-import org.junit.Ignore
 import org.junit.Test
 import org.matrix.rustcomponents.sdk.Client
 import org.matrix.rustcomponents.sdk.UserProfile
 import java.io.File
 
-@Ignore("JNA direct mapping has broken unit tests with FFI fakes")
 class RustMatrixClientTest {
     @Test
     fun `ensure that sessionId and deviceId can be retrieved from the client`() = runTest {

libraries/matrix/impl/src/test/kotlin/io/element/android/libraries/matrix/impl/auth/HomeserverDetailsKtTest.kt

@@ -11,10 +11,8 @@ package io.element.android.libraries.matrix.impl.auth
 import com.google.common.truth.Truth.assertThat
 import io.element.android.libraries.matrix.api.auth.MatrixHomeServerDetails
 import io.element.android.libraries.matrix.impl.fixtures.fakes.FakeFfiHomeserverLoginDetails
-import org.junit.Ignore
 import org.junit.Test
 
-@Ignore("JNA direct mapping has broken unit tests with FFI fakes")
 class HomeserverDetailsKtTest {
     @Test
     fun `map should be correct`() {

libraries/matrix/impl/src/test/kotlin/io/element/android/libraries/matrix/impl/auth/RustHomeserverLoginCompatibilityCheckerTest.kt

@@ -14,10 +14,8 @@ import io.element.android.libraries.matrix.impl.fixtures.fakes.FakeFfiClient
 import io.element.android.libraries.matrix.impl.fixtures.fakes.FakeFfiClientBuilder
 import io.element.android.libraries.matrix.impl.fixtures.fakes.FakeFfiHomeserverLoginDetails
 import kotlinx.coroutines.test.runTest
-import org.junit.Ignore
 import org.junit.Test
 
-@Ignore("JNA direct mapping has broken unit tests with FFI fakes")
 class RustHomeserverLoginCompatibilityCheckerTest {
     @Test
     fun `check - is valid if it supports OIDC login`() = runTest {

libraries/matrix/impl/src/test/kotlin/io/element/android/libraries/matrix/impl/auth/RustMatrixAuthenticationServiceTest.kt

@@ -23,12 +23,10 @@ import io.element.android.libraries.sessionstorage.test.InMemorySessionStore
 import io.element.android.tests.testutils.testCoroutineDispatchers
 import kotlinx.coroutines.test.TestScope
 import kotlinx.coroutines.test.runTest
-import org.junit.Ignore
 import org.junit.Test
 import java.io.File
 
 class RustMatrixAuthenticationServiceTest {
-    @Ignore("JNA direct mapping has broken unit tests with FFI fakes")
         @Test
     fun `setHomeserver is successful`() = runTest {
         val sut = createRustMatrixAuthenticationService(

libraries/matrix/impl/src/test/kotlin/io/element/android/libraries/matrix/impl/auth/qrlogin/SdkQrCodeLoginDataTest.kt

@@ -11,10 +11,8 @@ package io.element.android.libraries.matrix.impl.auth.qrlogin
 import com.google.common.truth.Truth.assertThat
 import io.element.android.libraries.matrix.impl.fixtures.fakes.FakeFfiQrCodeData
 import io.element.android.libraries.matrix.test.A_HOMESERVER_URL
-import org.junit.Ignore
 import org.junit.Test
 
-@Ignore("JNA direct mapping has broken unit tests with FFI fakes")
 class SdkQrCodeLoginDataTest {
     @Test
     fun `getServer reads the value from the Rust side, null case`() {

libraries/matrix/impl/src/test/kotlin/io/element/android/libraries/matrix/impl/notification/RustNotificationServiceTest.kt

@@ -29,14 +29,12 @@ import io.element.android.tests.testutils.lambda.lambdaRecorder
 import io.element.android.tests.testutils.testCoroutineDispatchers
 import kotlinx.coroutines.test.TestScope
 import kotlinx.coroutines.test.runTest
-import org.junit.Ignore
 import org.junit.Test
 import org.matrix.rustcomponents.sdk.NotificationClient
 import org.matrix.rustcomponents.sdk.NotificationStatus
 import org.matrix.rustcomponents.sdk.TimelineEventType
 
 class RustNotificationServiceTest {
-    @Ignore("JNA direct mapping has broken unit tests with FFI fakes")
         @Test
     fun test() = runTest {
         val notificationClient = FakeFfiNotificationClient(
@@ -58,7 +56,6 @@ class RustNotificationServiceTest {
         )
     }
 
-    @Ignore("JNA direct mapping has broken unit tests with FFI fakes")
         @Test
     fun `test mapping invalid item only drops that item`() = runTest {
         val error = IllegalStateException("This event type is not supported")
@@ -86,7 +83,6 @@ class RustNotificationServiceTest {
         assertThat(successfulResult?.isSuccess).isTrue()
     }
 
-    @Ignore("JNA direct mapping has broken unit tests with FFI fakes")
         @Test
     fun `test unable to resolve event`() = runTest {
         val notificationClient = FakeFfiNotificationClient(
@@ -99,7 +95,6 @@ class RustNotificationServiceTest {
         assertThat(exception).isInstanceOf(NotificationResolverException::class.java)
     }
 
-    @Ignore("JNA direct mapping has broken unit tests with FFI fakes")
         @Test
     fun `close should invoke the close method of the service`() = runTest {
         val closeResult = lambdaRecorder<Unit> { }

libraries/matrix/impl/src/test/kotlin/io/element/android/libraries/matrix/impl/notificationsettings/RustNotificationSettingsServiceTest.kt

@@ -16,12 +16,10 @@ import io.element.android.libraries.matrix.test.A_ROOM_ID
 import io.element.android.tests.testutils.testCoroutineDispatchers
 import kotlinx.coroutines.test.TestScope
 import kotlinx.coroutines.test.runTest
-import org.junit.Ignore
 import org.junit.Test
 import org.matrix.rustcomponents.sdk.NotificationSettings
 
 class RustNotificationSettingsServiceTest {
-    @Ignore("JNA direct mapping has broken unit tests with FFI fakes")
         @Test
     fun test() = runTest {
         val sut = createRustNotificationSettingsService()

libraries/matrix/impl/src/test/kotlin/io/element/android/libraries/matrix/impl/pushers/RustPushersServiceTest.kt

@@ -13,10 +13,8 @@ import io.element.android.libraries.matrix.api.pusher.UnsetHttpPusherData
 import io.element.android.libraries.matrix.impl.fixtures.fakes.FakeFfiClient
 import io.element.android.tests.testutils.testCoroutineDispatchers
 import kotlinx.coroutines.test.runTest
-import org.junit.Ignore
 import org.junit.Test
 
-@Ignore("JNA direct mapping has broken unit tests with FFI fakes")
 class RustPushersServiceTest {
     @Test
     fun `setPusher should invoke the client method`() = runTest {

libraries/matrix/impl/src/test/kotlin/io/element/android/libraries/matrix/impl/room/RoomInfoExtTest.kt

@@ -14,10 +14,8 @@ import io.element.android.libraries.matrix.api.user.MatrixUser
 import io.element.android.libraries.matrix.impl.fixtures.factories.aRustRoomHero
 import io.element.android.libraries.matrix.impl.fixtures.factories.aRustRoomInfo
 import io.element.android.libraries.matrix.test.A_USER_ID
-import org.junit.Ignore
 import org.junit.Test
 
-@Ignore("JNA direct mapping has broken unit tests with FFI fakes")
 class RoomInfoExtTest {
     @Test
     fun `get non empty element Heroes`() {

libraries/matrix/impl/src/test/kotlin/io/element/android/libraries/matrix/impl/room/RoomInfoMapperTest.kt

@@ -32,7 +32,6 @@ import io.element.android.libraries.matrix.test.room.defaultRoomPowerLevelValues
 import kotlinx.collections.immutable.persistentListOf
 import kotlinx.collections.immutable.persistentMapOf
 import kotlinx.collections.immutable.toImmutableList
-import org.junit.Ignore
 import org.junit.Test
 import org.matrix.rustcomponents.sdk.Membership
 import uniffi.matrix_sdk_base.EncryptionState
@@ -40,7 +39,6 @@ import org.matrix.rustcomponents.sdk.JoinRule as RustJoinRule
 import org.matrix.rustcomponents.sdk.RoomHistoryVisibility as RustRoomHistoryVisibility
 import org.matrix.rustcomponents.sdk.RoomNotificationMode as RustRoomNotificationMode
 
-@Ignore("JNA direct mapping has broken unit tests with FFI fakes")
 class RoomInfoMapperTest {
     @Test
     fun `mapping of RustRoomInfo should map all the fields`() {

libraries/matrix/impl/src/test/kotlin/io/element/android/libraries/matrix/impl/room/RustBaseRoomTest.kt

@@ -31,11 +31,9 @@ import kotlinx.coroutines.flow.shareIn
 import kotlinx.coroutines.isActive
 import kotlinx.coroutines.test.TestScope
 import kotlinx.coroutines.test.runTest
-import org.junit.Ignore
 import org.junit.Test
 import uniffi.matrix_sdk.RoomMemberRole
 
-@Ignore("JNA direct mapping has broken unit tests with FFI fakes")
 class RustBaseRoomTest {
     @Test
     fun `RustBaseRoom should cancel the room coroutine scope when it is destroyed`() = runTest {

libraries/matrix/impl/src/test/kotlin/io/element/android/libraries/matrix/impl/room/join/DefaultJoinRoomTest.kt

@@ -25,10 +25,8 @@ import io.element.android.services.analytics.test.FakeAnalyticsService
 import io.element.android.tests.testutils.lambda.lambdaRecorder
 import io.element.android.tests.testutils.lambda.value
 import kotlinx.coroutines.test.runTest
-import org.junit.Ignore
 import org.junit.Test
 
-@Ignore("JNA direct mapping has broken unit tests with FFI fakes")
 class DefaultJoinRoomTest {
     @Test
     fun `when using roomId and there is no server names, the classic join room API is used`() = runTest {

libraries/matrix/impl/src/test/kotlin/io/element/android/libraries/matrix/impl/room/member/RoomMemberListFetcherTest.kt

@@ -24,10 +24,8 @@ import io.element.android.libraries.matrix.test.A_USER_ID_3
 import io.element.android.libraries.matrix.test.A_USER_ID_4
 import kotlinx.coroutines.Dispatchers
 import kotlinx.coroutines.test.runTest
-import org.junit.Ignore
 import org.junit.Test
 
-@Ignore("JNA direct mapping has broken unit tests with FFI fakes")
 class RoomMemberListFetcherTest {
     @Test
     fun `fetchRoomMembers with CACHE source - emits cached members, if any`() = runTest {

libraries/matrix/impl/src/test/kotlin/io/element/android/libraries/matrix/impl/roomdirectory/RustBaseRoomDirectoryListTest.kt

@@ -19,12 +19,10 @@ import kotlinx.coroutines.test.StandardTestDispatcher
 import kotlinx.coroutines.test.TestScope
 import kotlinx.coroutines.test.runCurrent
 import kotlinx.coroutines.test.runTest
-import org.junit.Ignore
 import org.junit.Test
 import org.matrix.rustcomponents.sdk.RoomDirectorySearch
 import org.matrix.rustcomponents.sdk.RoomDirectorySearchEntryUpdate
 
-@Ignore("JNA direct mapping has broken unit tests with FFI fakes")
 @OptIn(ExperimentalCoroutinesApi::class)
 class RustBaseRoomDirectoryListTest {
     @Test

libraries/matrix/impl/src/test/kotlin/io/element/android/libraries/matrix/impl/roomdirectory/RustBaseRoomDirectoryServiceTest.kt

@@ -11,11 +11,9 @@ package io.element.android.libraries.matrix.impl.roomdirectory
 import io.element.android.libraries.matrix.impl.fixtures.fakes.FakeFfiClient
 import kotlinx.coroutines.test.StandardTestDispatcher
 import kotlinx.coroutines.test.runTest
-import org.junit.Ignore
 import org.junit.Test
 
 class RustBaseRoomDirectoryServiceTest {
-    @Ignore("JNA direct mapping has broken unit tests with FFI fakes")
         @Test
     fun test() = runTest {
         val client = FakeFfiClient()

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

@@ -12,12 +12,10 @@ import io.element.android.libraries.matrix.impl.fixtures.fakes.FakeFfiRoomList
 import io.element.android.libraries.matrix.impl.fixtures.fakes.FakeFfiRoomListService
 import io.element.android.services.analytics.test.FakeAnalyticsService
 import kotlinx.coroutines.test.runTest
-import org.junit.Ignore
 import org.junit.Test
 import kotlin.coroutines.EmptyCoroutineContext
 
 class RoomListFactoryTest {
-    @Ignore("JNA direct mapping has broken unit tests with FFI fakes")
         @Test
     fun `createRoomList should work`() = runTest {
         val sut = RoomListFactory(

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

@@ -21,14 +21,12 @@ import kotlinx.coroutines.flow.MutableStateFlow
 import kotlinx.coroutines.test.StandardTestDispatcher
 import kotlinx.coroutines.test.TestScope
 import kotlinx.coroutines.test.runTest
-import org.junit.Ignore
 import org.junit.Test
 import org.matrix.rustcomponents.sdk.RoomListEntriesUpdate
 
 class RoomSummaryListProcessorTest {
     private val summaries = MutableStateFlow<List<RoomSummary>>(emptyList())
 
-    @Ignore("JNA direct mapping has broken unit tests with FFI fakes")
         @Test
     fun `Append adds new entries at the end of the list`() = runTest {
         summaries.value = listOf(aRoomSummary())
@@ -41,7 +39,6 @@ class RoomSummaryListProcessorTest {
         assertThat(summaries.value.subList(1, 4).all { it.roomId == A_ROOM_ID_2 }).isTrue()
     }
 
-    @Ignore("JNA direct mapping has broken unit tests with FFI fakes")
         @Test
     fun `PushBack adds a new entry at the end of the list`() = runTest {
         summaries.value = listOf(aRoomSummary())
@@ -52,7 +49,6 @@ class RoomSummaryListProcessorTest {
         assertThat(summaries.value.last().roomId).isEqualTo(A_ROOM_ID_2)
     }
 
-    @Ignore("JNA direct mapping has broken unit tests with FFI fakes")
         @Test
     fun `PushFront inserts a new entry at the start of the list`() = runTest {
         summaries.value = listOf(aRoomSummary())
@@ -63,7 +59,6 @@ class RoomSummaryListProcessorTest {
         assertThat(summaries.value.first().roomId).isEqualTo(A_ROOM_ID_2)
     }
 
-    @Ignore("JNA direct mapping has broken unit tests with FFI fakes")
         @Test
     fun `Set replaces an entry at some index`() = runTest {
         summaries.value = listOf(aRoomSummary())
@@ -76,7 +71,6 @@ class RoomSummaryListProcessorTest {
         assertThat(summaries.value[index].roomId).isEqualTo(A_ROOM_ID_2)
     }
 
-    @Ignore("JNA direct mapping has broken unit tests with FFI fakes")
         @Test
     fun `Insert inserts a new entry at the provided index`() = runTest {
         summaries.value = listOf(aRoomSummary())
@@ -89,7 +83,6 @@ class RoomSummaryListProcessorTest {
         assertThat(summaries.value[index].roomId).isEqualTo(A_ROOM_ID_2)
     }
 
-    @Ignore("JNA direct mapping has broken unit tests with FFI fakes")
         @Test
     fun `Remove removes an entry at some index`() = runTest {
         summaries.value = listOf(
@@ -105,7 +98,6 @@ class RoomSummaryListProcessorTest {
         assertThat(summaries.value[index].roomId).isEqualTo(A_ROOM_ID_2)
     }
 
-    @Ignore("JNA direct mapping has broken unit tests with FFI fakes")
         @Test
     fun `PopBack removes an entry at the end of the list`() = runTest {
         summaries.value = listOf(
@@ -121,7 +113,6 @@ class RoomSummaryListProcessorTest {
         assertThat(summaries.value[index].roomId).isEqualTo(A_ROOM_ID)
     }
 
-    @Ignore("JNA direct mapping has broken unit tests with FFI fakes")
         @Test
     fun `PopFront removes an entry at the start of the list`() = runTest {
         summaries.value = listOf(
@@ -137,7 +128,6 @@ class RoomSummaryListProcessorTest {
         assertThat(summaries.value[index].roomId).isEqualTo(A_ROOM_ID_2)
     }
 
-    @Ignore("JNA direct mapping has broken unit tests with FFI fakes")
         @Test
     fun `Clear removes all the entries`() = runTest {
         summaries.value = listOf(
@@ -151,7 +141,6 @@ class RoomSummaryListProcessorTest {
         assertThat(summaries.value).isEmpty()
     }
 
-    @Ignore("JNA direct mapping has broken unit tests with FFI fakes")
         @Test
     fun `Truncate removes all entries after the provided length`() = runTest {
         summaries.value = listOf(
@@ -167,7 +156,6 @@ class RoomSummaryListProcessorTest {
         assertThat(summaries.value[index].roomId).isEqualTo(A_ROOM_ID)
     }
 
-    @Ignore("JNA direct mapping has broken unit tests with FFI fakes")
         @Test
     fun `Reset removes all entries and add the provided ones`() = runTest {
         summaries.value = listOf(

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

@@ -19,12 +19,10 @@ import kotlinx.coroutines.test.StandardTestDispatcher
 import kotlinx.coroutines.test.TestScope
 import kotlinx.coroutines.test.runCurrent
 import kotlinx.coroutines.test.runTest
-import org.junit.Ignore
 import org.junit.Test
 import org.matrix.rustcomponents.sdk.RoomListServiceSyncIndicator
 import org.matrix.rustcomponents.sdk.RoomListService as RustRoomListService
 
-@Ignore("JNA direct mapping has broken unit tests with FFI fakes")
 @OptIn(ExperimentalCoroutinesApi::class)
 class RustBaseRoomListServiceTest {
     @Test

libraries/matrix/impl/src/test/kotlin/io/element/android/libraries/matrix/impl/spaces/RoomSummaryListProcessorTest.kt

@@ -18,14 +18,12 @@ import io.element.android.libraries.previewutils.room.aSpaceRoom
 import kotlinx.coroutines.flow.MutableSharedFlow
 import kotlinx.coroutines.flow.MutableStateFlow
 import kotlinx.coroutines.test.runTest
-import org.junit.Ignore
 import org.junit.Test
 import org.matrix.rustcomponents.sdk.SpaceListUpdate
 
 class RoomSummaryListProcessorTest {
     private val spaceRoomsFlow = MutableStateFlow<List<SpaceRoom>>(emptyList())
 
-    @Ignore("JNA direct mapping has broken unit tests with FFI fakes")
         @Test
     fun `Append adds new entries at the end of the list`() = runTest {
         spaceRoomsFlow.value = listOf(aSpaceRoom())
@@ -38,7 +36,6 @@ class RoomSummaryListProcessorTest {
         assertThat(spaceRoomsFlow.value.subList(1, 4).all { it.roomId == A_ROOM_ID_2 }).isTrue()
     }
 
-    @Ignore("JNA direct mapping has broken unit tests with FFI fakes")
         @Test
     fun `PushBack adds a new entry at the end of the list`() = runTest {
         spaceRoomsFlow.value = listOf(aSpaceRoom())
@@ -49,7 +46,6 @@ class RoomSummaryListProcessorTest {
         assertThat(spaceRoomsFlow.value.last().roomId).isEqualTo(A_ROOM_ID_2)
     }
 
-    @Ignore("JNA direct mapping has broken unit tests with FFI fakes")
         @Test
     fun `PushFront inserts a new entry at the start of the list`() = runTest {
         spaceRoomsFlow.value = listOf(aSpaceRoom())
@@ -60,7 +56,6 @@ class RoomSummaryListProcessorTest {
         assertThat(spaceRoomsFlow.value.first().roomId).isEqualTo(A_ROOM_ID_2)
     }
 
-    @Ignore("JNA direct mapping has broken unit tests with FFI fakes")
         @Test
     fun `Set replaces an entry at some index`() = runTest {
         spaceRoomsFlow.value = listOf(aSpaceRoom())
@@ -73,7 +68,6 @@ class RoomSummaryListProcessorTest {
         assertThat(spaceRoomsFlow.value[index].roomId).isEqualTo(A_ROOM_ID_2)
     }
 
-    @Ignore("JNA direct mapping has broken unit tests with FFI fakes")
         @Test
     fun `Insert inserts a new entry at the provided index`() = runTest {
         spaceRoomsFlow.value = listOf(aSpaceRoom())
@@ -86,7 +80,6 @@ class RoomSummaryListProcessorTest {
         assertThat(spaceRoomsFlow.value[index].roomId).isEqualTo(A_ROOM_ID_2)
     }
 
-    @Ignore("JNA direct mapping has broken unit tests with FFI fakes")
         @Test
     fun `Remove removes an entry at some index`() = runTest {
         spaceRoomsFlow.value = listOf(
@@ -102,7 +95,6 @@ class RoomSummaryListProcessorTest {
         assertThat(spaceRoomsFlow.value[index].roomId).isEqualTo(A_ROOM_ID_2)
     }
 
-    @Ignore("JNA direct mapping has broken unit tests with FFI fakes")
         @Test
     fun `PopBack removes an entry at the end of the list`() = runTest {
         spaceRoomsFlow.value = listOf(
@@ -118,7 +110,6 @@ class RoomSummaryListProcessorTest {
         assertThat(spaceRoomsFlow.value[index].roomId).isEqualTo(A_ROOM_ID)
     }
 
-    @Ignore("JNA direct mapping has broken unit tests with FFI fakes")
         @Test
     fun `PopFront removes an entry at the start of the list`() = runTest {
         spaceRoomsFlow.value = listOf(
@@ -134,7 +125,6 @@ class RoomSummaryListProcessorTest {
         assertThat(spaceRoomsFlow.value[index].roomId).isEqualTo(A_ROOM_ID_2)
     }
 
-    @Ignore("JNA direct mapping has broken unit tests with FFI fakes")
         @Test
     fun `Clear removes all the entries`() = runTest {
         spaceRoomsFlow.value = listOf(
@@ -148,7 +138,6 @@ class RoomSummaryListProcessorTest {
         assertThat(spaceRoomsFlow.value).isEmpty()
     }
 
-    @Ignore("JNA direct mapping has broken unit tests with FFI fakes")
         @Test
     fun `Truncate removes all entries after the provided length`() = runTest {
         spaceRoomsFlow.value = listOf(
@@ -164,7 +153,6 @@ class RoomSummaryListProcessorTest {
         assertThat(spaceRoomsFlow.value[index].roomId).isEqualTo(A_ROOM_ID)
     }
 
-    @Ignore("JNA direct mapping has broken unit tests with FFI fakes")
         @Test
     fun `Reset removes all entries and add the provided ones`() = runTest {
         spaceRoomsFlow.value = listOf(
@@ -180,7 +168,6 @@ class RoomSummaryListProcessorTest {
         assertThat(spaceRoomsFlow.value[index].roomId).isEqualTo(A_ROOM_ID_3)
     }
 
-    @Ignore("JNA direct mapping has broken unit tests with FFI fakes")
         @Test
     fun `When there is no replay cache SpaceListUpdateProcessor starts with an empty list`() = runTest {
         val spaceRoomsSharedFlow = MutableSharedFlow<List<SpaceRoom>>(replay = 1)

libraries/matrix/impl/src/test/kotlin/io/element/android/libraries/matrix/impl/spaces/RustSpaceRoomListTest.kt

@@ -23,14 +23,12 @@ import kotlinx.coroutines.ExperimentalCoroutinesApi
 import kotlinx.coroutines.test.TestScope
 import kotlinx.coroutines.test.runCurrent
 import kotlinx.coroutines.test.runTest
-import org.junit.Ignore
 import org.junit.Test
 import org.matrix.rustcomponents.sdk.SpaceListUpdate
 import uniffi.matrix_sdk_ui.SpaceRoomListPaginationState
 import org.matrix.rustcomponents.sdk.SpaceRoomList as InnerSpaceRoomList
 
 class RustSpaceRoomListTest {
-    @Ignore("JNA direct mapping has broken unit tests with FFI fakes")
         @Test
     fun `paginationStatusFlow emits values`() = runTest {
         val innerSpaceRoomList = FakeFfiSpaceRoomList(
@@ -53,7 +51,6 @@ class RustSpaceRoomListTest {
         }
     }
 
-    @Ignore("JNA direct mapping has broken unit tests with FFI fakes")
         @Test
     fun `spaceRoomsFlow emits values`() = runTest {
         val innerSpaceRoomList = FakeFfiSpaceRoomList(
@@ -76,7 +73,6 @@ class RustSpaceRoomListTest {
         }
     }
 
-    @Ignore("JNA direct mapping has broken unit tests with FFI fakes")
         @Test
     fun `paginate invokes paginate on the inner class`() = runTest {
         val paginateResult = lambdaRecorder<Unit> { }

libraries/matrix/impl/src/test/kotlin/io/element/android/libraries/matrix/impl/timeline/MatrixTimelineDiffProcessorTest.kt

@@ -21,7 +21,6 @@ import kotlinx.coroutines.flow.MutableSharedFlow
 import kotlinx.coroutines.flow.MutableStateFlow
 import kotlinx.coroutines.test.TestScope
 import kotlinx.coroutines.test.runTest
-import org.junit.Ignore
 import org.junit.Test
 import org.matrix.rustcomponents.sdk.TimelineDiff
 
@@ -31,7 +30,6 @@ class MatrixTimelineDiffProcessorTest {
     private val anEvent = MatrixTimelineItem.Event(A_UNIQUE_ID, anEventTimelineItem())
     private val anEvent2 = MatrixTimelineItem.Event(A_UNIQUE_ID_2, anEventTimelineItem())
 
-    @Ignore("JNA direct mapping has broken unit tests with FFI fakes")
         @Test
     fun `Append adds new entries at the end of the list`() = runTest {
         timelineItems.value = listOf(anEvent)
@@ -44,7 +42,6 @@ class MatrixTimelineDiffProcessorTest {
         )
     }
 
-    @Ignore("JNA direct mapping has broken unit tests with FFI fakes")
         @Test
     fun `PushBack adds a new entry at the end of the list`() = runTest {
         timelineItems.value = listOf(anEvent)
@@ -57,7 +54,6 @@ class MatrixTimelineDiffProcessorTest {
         )
     }
 
-    @Ignore("JNA direct mapping has broken unit tests with FFI fakes")
         @Test
     fun `PushFront inserts a new entry at the start of the list`() = runTest {
         timelineItems.value = listOf(anEvent)
@@ -70,7 +66,6 @@ class MatrixTimelineDiffProcessorTest {
         )
     }
 
-    @Ignore("JNA direct mapping has broken unit tests with FFI fakes")
         @Test
     fun `Set replaces an entry at some index`() = runTest {
         timelineItems.value = listOf(anEvent, anEvent2)
@@ -83,7 +78,6 @@ class MatrixTimelineDiffProcessorTest {
         )
     }
 
-    @Ignore("JNA direct mapping has broken unit tests with FFI fakes")
         @Test
     fun `Insert inserts a new entry at the provided index`() = runTest {
         timelineItems.value = listOf(anEvent, anEvent2)
@@ -97,7 +91,6 @@ class MatrixTimelineDiffProcessorTest {
         )
     }
 
-    @Ignore("JNA direct mapping has broken unit tests with FFI fakes")
         @Test
     fun `Remove removes an entry at some index`() = runTest {
         timelineItems.value = listOf(anEvent, MatrixTimelineItem.Other, anEvent2)
@@ -110,7 +103,6 @@ class MatrixTimelineDiffProcessorTest {
         )
     }
 
-    @Ignore("JNA direct mapping has broken unit tests with FFI fakes")
         @Test
     fun `PopBack removes an entry at the end of the list`() = runTest {
         timelineItems.value = listOf(anEvent, anEvent2)
@@ -122,7 +114,6 @@ class MatrixTimelineDiffProcessorTest {
         )
     }
 
-    @Ignore("JNA direct mapping has broken unit tests with FFI fakes")
         @Test
     fun `PopFront removes an entry at the start of the list`() = runTest {
         timelineItems.value = listOf(anEvent, anEvent2)
@@ -134,7 +125,6 @@ class MatrixTimelineDiffProcessorTest {
         )
     }
 
-    @Ignore("JNA direct mapping has broken unit tests with FFI fakes")
         @Test
     fun `Clear removes all the entries`() = runTest {
         timelineItems.value = listOf(anEvent, anEvent2)
@@ -143,7 +133,6 @@ class MatrixTimelineDiffProcessorTest {
         assertThat(timelineItems.value).isEmpty()
     }
 
-    @Ignore("JNA direct mapping has broken unit tests with FFI fakes")
         @Test
     fun `Truncate removes all entries after the provided length`() = runTest {
         timelineItems.value = listOf(anEvent, MatrixTimelineItem.Other, anEvent2)
@@ -155,7 +144,6 @@ class MatrixTimelineDiffProcessorTest {
         )
     }
 
-    @Ignore("JNA direct mapping has broken unit tests with FFI fakes")
         @Test
     fun `Reset removes all entries and add the provided ones`() = runTest {
         timelineItems.value = listOf(anEvent, MatrixTimelineItem.Other, anEvent2)

libraries/matrix/impl/src/test/kotlin/io/element/android/libraries/matrix/impl/timeline/RustTimelineTest.kt

@@ -30,13 +30,11 @@ import kotlinx.coroutines.ExperimentalCoroutinesApi
 import kotlinx.coroutines.test.TestScope
 import kotlinx.coroutines.test.runCurrent
 import kotlinx.coroutines.test.runTest
-import org.junit.Ignore
 import org.junit.Test
 import org.matrix.rustcomponents.sdk.TimelineDiff
 import uniffi.matrix_sdk.RoomPaginationStatus
 import org.matrix.rustcomponents.sdk.Timeline as InnerTimeline
 
-@Ignore("JNA direct mapping has broken unit tests with FFI fakes")
 class RustTimelineTest {
     @Test
     fun `ensure that the timeline emits new loading item when pagination does not bring new events`() = runTest {

libraries/matrix/impl/src/test/kotlin/io/element/android/libraries/matrix/impl/timeline/TimelineItemsSubscriberTest.kt

@@ -20,7 +20,6 @@ import kotlinx.coroutines.test.StandardTestDispatcher
 import kotlinx.coroutines.test.TestScope
 import kotlinx.coroutines.test.runCurrent
 import kotlinx.coroutines.test.runTest
-import org.junit.Ignore
 import org.junit.Test
 import org.matrix.rustcomponents.sdk.Timeline
 import org.matrix.rustcomponents.sdk.TimelineDiff
@@ -28,7 +27,6 @@ import uniffi.matrix_sdk_ui.EventItemOrigin
 
 @OptIn(ExperimentalCoroutinesApi::class)
 class TimelineItemsSubscriberTest {
-    @Ignore("JNA direct mapping has broken unit tests with FFI fakes")
         @Test
     fun `when timeline emits an empty list of items, the flow must emits an empty list`() = runTest {
         val timelineItems: MutableSharedFlow<List<MatrixTimelineItem>> =
@@ -52,7 +50,6 @@ class TimelineItemsSubscriberTest {
         }
     }
 
-    @Ignore("JNA direct mapping has broken unit tests with FFI fakes")
         @Test
     fun `when timeline emits a non empty list of items, the flow must emits a non empty list`() = runTest {
         val timelineItems: MutableSharedFlow<List<MatrixTimelineItem>> =
@@ -76,7 +73,6 @@ class TimelineItemsSubscriberTest {
         }
     }
 
-    @Ignore("JNA direct mapping has broken unit tests with FFI fakes")
         @Test
     fun `when timeline emits an item with SYNC origin`() = runTest {
         val timelineItems: MutableSharedFlow<List<MatrixTimelineItem>> =
@@ -108,7 +104,6 @@ class TimelineItemsSubscriberTest {
         }
     }
 
-    @Ignore("JNA direct mapping has broken unit tests with FFI fakes")
         @Test
     fun `multiple subscriptions does not have side effect`() = runTest {
         val timelineItemsSubscriber = createTimelineItemsSubscriber()