diff --git a/features/home/impl/src/main/kotlin/io/element/android/features/home/impl/HomeFlowNode.kt b/features/home/impl/src/main/kotlin/io/element/android/features/home/impl/HomeFlowNode.kt index 94f243b634..cb205960fd 100644 --- a/features/home/impl/src/main/kotlin/io/element/android/features/home/impl/HomeFlowNode.kt +++ b/features/home/impl/src/main/kotlin/io/element/android/features/home/impl/HomeFlowNode.kt @@ -87,8 +87,10 @@ class HomeFlowNode( analyticsService.screen(MobileScreen(screenName = MobileScreen.ScreenName.Home)) } ) - whenChildAttached { commonLifecycle: Lifecycle, - changeRoomMemberRolesNode: ChangeRoomMemberRolesEntryPoint.NodeProxy -> + whenChildAttached { + commonLifecycle: Lifecycle, + changeRoomMemberRolesNode: ChangeRoomMemberRolesEntryPoint.NodeProxy, + -> commonLifecycle.coroutineScope.launch { changeRoomMemberRolesNode.waitForRoleChanged() withContext(NonCancellable) { diff --git a/features/invite/impl/src/test/kotlin/io/element/android/features/invite/impl/DefaultDeclineInviteTest.kt b/features/invite/impl/src/test/kotlin/io/element/android/features/invite/impl/DefaultDeclineInviteTest.kt index cb07bd5191..d740d6065c 100644 --- a/features/invite/impl/src/test/kotlin/io/element/android/features/invite/impl/DefaultDeclineInviteTest.kt +++ b/features/invite/impl/src/test/kotlin/io/element/android/features/invite/impl/DefaultDeclineInviteTest.kt @@ -34,14 +34,14 @@ class DefaultDeclineInviteTest { private val notificationCleaner = FakeNotificationCleaner(clearMembershipNotificationForRoomLambda = clearMembershipNotificationForRoomLambda) - private val successLeaveRoomLambda = lambdaRecorder> { -> Result.success(Unit) } + private val successLeaveRoomLambda = lambdaRecorder> { Result.success(Unit) } private val successIgnoreUserLambda = lambdaRecorder> { _ -> Result.success(Unit) } private val successReportRoomLambda = lambdaRecorder> { _ -> Result.success(Unit) } private val failureLeaveRoomLambda = - lambdaRecorder> { -> Result.failure(Exception("Leave room error")) } + lambdaRecorder> { Result.failure(Exception("Leave room error")) } private val failureIgnoreUserLambda = lambdaRecorder> { _ -> Result.failure(Exception("Ignore user error")) } private val failureReportRoomLambda = diff --git a/features/reportroom/impl/src/test/kotlin/io/element/android/features/reportroom/impl/DefaultReportRoomTest.kt b/features/reportroom/impl/src/test/kotlin/io/element/android/features/reportroom/impl/DefaultReportRoomTest.kt index fc410ab08f..c196d7a29e 100644 --- a/features/reportroom/impl/src/test/kotlin/io/element/android/features/reportroom/impl/DefaultReportRoomTest.kt +++ b/features/reportroom/impl/src/test/kotlin/io/element/android/features/reportroom/impl/DefaultReportRoomTest.kt @@ -19,12 +19,12 @@ import org.junit.Test class DefaultReportRoomTest { private val roomId = A_ROOM_ID - private val successLeaveRoomLambda = lambdaRecorder> { -> Result.success(Unit) } + private val successLeaveRoomLambda = lambdaRecorder> { Result.success(Unit) } private val successReportRoomLambda = lambdaRecorder> { _ -> Result.success(Unit) } private val failureLeaveRoomLambda = - lambdaRecorder> { -> Result.failure(Exception("Leave room error")) } + lambdaRecorder> { Result.failure(Exception("Leave room error")) } private val failureReportRoomLambda = lambdaRecorder> { _ -> Result.failure(Exception("Report room error")) } diff --git a/features/roomdetails/impl/src/main/kotlin/io/element/android/features/roomdetails/impl/RoomDetailsFlowNode.kt b/features/roomdetails/impl/src/main/kotlin/io/element/android/features/roomdetails/impl/RoomDetailsFlowNode.kt index a72f0394b4..b0540626b9 100644 --- a/features/roomdetails/impl/src/main/kotlin/io/element/android/features/roomdetails/impl/RoomDetailsFlowNode.kt +++ b/features/roomdetails/impl/src/main/kotlin/io/element/android/features/roomdetails/impl/RoomDetailsFlowNode.kt @@ -148,9 +148,11 @@ class RoomDetailsFlowNode( override fun onBuilt() { super.onBuilt() - whenChildrenAttached { commonLifecycle: Lifecycle, - roomDetailsNode: RoomDetailsNode, - changeRoomMemberRolesNode: ChangeRoomMemberRolesEntryPoint.NodeProxy -> + whenChildrenAttached { + commonLifecycle: Lifecycle, + roomDetailsNode: RoomDetailsNode, + changeRoomMemberRolesNode: ChangeRoomMemberRolesEntryPoint.NodeProxy, + -> commonLifecycle.coroutineScope.launch { changeRoomMemberRolesNode.waitForRoleChanged() withContext(NonCancellable) { diff --git a/features/roomdirectory/impl/src/test/kotlin/io/element/android/features/roomdirectory/impl/root/RoomDirectoryPresenterTest.kt b/features/roomdirectory/impl/src/test/kotlin/io/element/android/features/roomdirectory/impl/root/RoomDirectoryPresenterTest.kt index 1e450297ae..eedfe03b02 100644 --- a/features/roomdirectory/impl/src/test/kotlin/io/element/android/features/roomdirectory/impl/root/RoomDirectoryPresenterTest.kt +++ b/features/roomdirectory/impl/src/test/kotlin/io/element/android/features/roomdirectory/impl/root/RoomDirectoryPresenterTest.kt @@ -105,9 +105,7 @@ class RoomDirectoryPresenterTest { @Test fun `present - emit load more event`() = runTest { - val loadMoreLambda = lambdaRecorder { -> - Result.success(Unit) - } + val loadMoreLambda = lambdaRecorder> { Result.success(Unit) } val roomDirectoryList = FakeRoomDirectoryList(loadMoreLambda = loadMoreLambda) val roomDirectoryService = FakeRoomDirectoryService { roomDirectoryList } val presenter = createRoomDirectoryPresenter(roomDirectoryService = roomDirectoryService) diff --git a/libraries/designsystem/src/main/kotlin/io/element/android/libraries/designsystem/components/media/WaveFormSamples.kt b/libraries/designsystem/src/main/kotlin/io/element/android/libraries/designsystem/components/media/WaveFormSamples.kt index 6fbb217392..18efa2d3f3 100644 --- a/libraries/designsystem/src/main/kotlin/io/element/android/libraries/designsystem/components/media/WaveFormSamples.kt +++ b/libraries/designsystem/src/main/kotlin/io/element/android/libraries/designsystem/components/media/WaveFormSamples.kt @@ -14,6 +14,7 @@ import kotlinx.collections.immutable.toPersistentList object WaveFormSamples { val allRangeWaveForm = List(100) { it.toFloat() / 100 }.toImmutableList() + @Suppress("ktlint:standard:argument-list-wrapping") val realisticWaveForm = persistentListOf( 0.000f, 0.000f, 0.000f, 0.003f, 0.354f, 0.353f, 0.365f, 0.790f, 0.787f, 0.167f, diff --git a/libraries/maplibre-compose/src/main/kotlin/io/element/android/libraries/maplibre/compose/MapLibreMap.kt b/libraries/maplibre-compose/src/main/kotlin/io/element/android/libraries/maplibre/compose/MapLibreMap.kt index f1ef0f0459..478c7db499 100644 --- a/libraries/maplibre-compose/src/main/kotlin/io/element/android/libraries/maplibre/compose/MapLibreMap.kt +++ b/libraries/maplibre-compose/src/main/kotlin/io/element/android/libraries/maplibre/compose/MapLibreMap.kt @@ -71,10 +71,7 @@ public fun MapLibreMap( uiSettings: MapUiSettings = DefaultMapUiSettings, symbolManagerSettings: MapSymbolManagerSettings = DefaultMapSymbolManagerSettings, locationSettings: MapLocationSettings = DefaultMapLocationSettings, - content: ( - @Composable @MapLibreMapComposable - () -> Unit - )? = null, + content: (@Composable @MapLibreMapComposable () -> Unit)? = null, ) { // When in preview, early return a Box with the received modifier preserving layout if (LocalInspectionMode.current) { diff --git a/libraries/matrix/impl/src/main/kotlin/io/element/android/libraries/matrix/impl/timeline/RustTimeline.kt b/libraries/matrix/impl/src/main/kotlin/io/element/android/libraries/matrix/impl/timeline/RustTimeline.kt index a488341060..ecf37dd9bf 100644 --- a/libraries/matrix/impl/src/main/kotlin/io/element/android/libraries/matrix/impl/timeline/RustTimeline.kt +++ b/libraries/matrix/impl/src/main/kotlin/io/element/android/libraries/matrix/impl/timeline/RustTimeline.kt @@ -207,10 +207,12 @@ class RustTimeline( backwardPaginationStatus, forwardPaginationStatus, joinedRoom.roomInfoFlow.map { it.creators to it.isDm }.distinctUntilChanged(), - ) { timelineItems, + ) { + timelineItems, backwardPaginationStatus, forwardPaginationStatus, - (roomCreators, isDm) -> + (roomCreators, isDm), + -> withContext(dispatcher) { timelineItems .let { items -> diff --git a/libraries/push/impl/src/test/kotlin/io/element/android/libraries/push/impl/notifications/fake/FakeNotificationCreator.kt b/libraries/push/impl/src/test/kotlin/io/element/android/libraries/push/impl/notifications/fake/FakeNotificationCreator.kt index 491b0bac33..1cc4468b15 100644 --- a/libraries/push/impl/src/test/kotlin/io/element/android/libraries/push/impl/notifications/fake/FakeNotificationCreator.kt +++ b/libraries/push/impl/src/test/kotlin/io/element/android/libraries/push/impl/notifications/fake/FakeNotificationCreator.kt @@ -34,7 +34,7 @@ class FakeNotificationCreator( var createFallbackNotificationResult: LambdaOneParamRecorder = lambdaRecorder { _ -> A_NOTIFICATION }, var createSummaryListNotificationResult: LambdaFourParamsRecorder = lambdaRecorder { _, _, _, _ -> A_NOTIFICATION }, - var createDiagnosticNotificationResult: LambdaNoParamRecorder = lambdaRecorder { -> A_NOTIFICATION }, + var createDiagnosticNotificationResult: LambdaNoParamRecorder = lambdaRecorder { A_NOTIFICATION }, ) : NotificationCreator { override suspend fun createMessagesListNotification( roomInfo: RoomEventGroupInfo, diff --git a/libraries/voiceplayer/impl/src/test/kotlin/io/element/android/libraries/voiceplayer/impl/DefaultVoiceMessagePlayerTest.kt b/libraries/voiceplayer/impl/src/test/kotlin/io/element/android/libraries/voiceplayer/impl/DefaultVoiceMessagePlayerTest.kt index cd8a3f6c6b..a3e2938734 100644 --- a/libraries/voiceplayer/impl/src/test/kotlin/io/element/android/libraries/voiceplayer/impl/DefaultVoiceMessagePlayerTest.kt +++ b/libraries/voiceplayer/impl/src/test/kotlin/io/element/android/libraries/voiceplayer/impl/DefaultVoiceMessagePlayerTest.kt @@ -114,7 +114,8 @@ class DefaultVoiceMessagePlayerTest { assertThat(player1.prepare().isSuccess).isTrue() matchReadyState(1_000L) player1.play() - awaitItem().let { // it plays until the end. + awaitItem().let { + // it plays until the end. assertThat(it.isReady).isFalse() assertThat(it.isPlaying).isFalse() assertThat(it.isEnded).isTrue() @@ -127,14 +128,16 @@ class DefaultVoiceMessagePlayerTest { player2.state.test { matchInitialState() assertThat(player2.prepare().isSuccess).isTrue() - awaitItem().let { // Additional spurious state due to MediaPlayer owner change. + awaitItem().let { + // Additional spurious state due to MediaPlayer owner change. assertThat(it.isReady).isFalse() assertThat(it.isPlaying).isFalse() assertThat(it.isEnded).isTrue() assertThat(it.currentPosition).isEqualTo(1000) assertThat(it.duration).isEqualTo(1000) } - awaitItem().let { // Additional spurious state due to MediaPlayer owner change. + awaitItem().let { + // Additional spurious state due to MediaPlayer owner change. assertThat(it.isReady).isFalse() assertThat(it.isPlaying).isFalse() assertThat(it.isEnded).isFalse() @@ -143,7 +146,8 @@ class DefaultVoiceMessagePlayerTest { } matchReadyState(1_000L) player2.play() - awaitItem().let { // it plays until the end. + awaitItem().let { + // it plays until the end. assertThat(it.isReady).isFalse() assertThat(it.isPlaying).isFalse() assertThat(it.isEnded).isTrue() @@ -154,7 +158,8 @@ class DefaultVoiceMessagePlayerTest { // Play player1 again. player1.state.test { - awaitItem().let { // Last previous state/ + awaitItem().let { + // Last previous state/ assertThat(it.isReady).isFalse() assertThat(it.isPlaying).isFalse() assertThat(it.isEnded).isTrue() @@ -162,7 +167,8 @@ class DefaultVoiceMessagePlayerTest { assertThat(it.duration).isEqualTo(1000) } assertThat(player1.prepare().isSuccess).isTrue() - awaitItem().let { // Additional spurious state due to MediaPlayer owner change. + awaitItem().let { + // Additional spurious state due to MediaPlayer owner change. assertThat(it.isReady).isFalse() assertThat(it.isPlaying).isFalse() assertThat(it.isEnded).isFalse() @@ -171,7 +177,8 @@ class DefaultVoiceMessagePlayerTest { } matchReadyState(1_000L) player1.play() - awaitItem().let { // it played again until the end. + awaitItem().let { + // it played again until the end. assertThat(it.isReady).isFalse() assertThat(it.isPlaying).isFalse() assertThat(it.isEnded).isTrue() @@ -189,7 +196,8 @@ class DefaultVoiceMessagePlayerTest { assertThat(player.prepare().isSuccess).isTrue() matchReadyState() player.play() - skipItems(1) // skip play state + // skip play state + skipItems(1) player.pause() awaitItem().let { assertThat(it.isPlaying).isFalse() @@ -206,9 +214,11 @@ class DefaultVoiceMessagePlayerTest { assertThat(player.prepare().isSuccess).isTrue() matchReadyState() player.play() - skipItems(1) // skip play state + // skip play state + skipItems(1) player.pause() - skipItems(1) // skip pause state + // skip pause state + skipItems(1) player.play() awaitItem().let { assertThat(it.isPlaying).isTrue()