From 48b7f054ee00b961230e15ee275d039bb3fde206 Mon Sep 17 00:00:00 2001
From: Benoit Marty <benoit@matrix.org>
Date: Wed, 7 Feb 2024 10:58:31 +0100
Subject: [PATCH] Introduce function `aContextMenuShown` to create
 `RoomListState.ContextMenu.Shown`

---
 .../roomlist/impl/RoomListContextMenu.kt      | 14 ++-----------
 .../roomlist/impl/RoomListStateProvider.kt    | 20 +++++++++++--------
 2 files changed, 14 insertions(+), 20 deletions(-)

diff --git a/features/roomlist/impl/src/main/kotlin/io/element/android/features/roomlist/impl/RoomListContextMenu.kt b/features/roomlist/impl/src/main/kotlin/io/element/android/features/roomlist/impl/RoomListContextMenu.kt
index f605b71a10..15213ad29e 100644
--- a/features/roomlist/impl/src/main/kotlin/io/element/android/features/roomlist/impl/RoomListContextMenu.kt
+++ b/features/roomlist/impl/src/main/kotlin/io/element/android/features/roomlist/impl/RoomListContextMenu.kt
@@ -164,12 +164,7 @@ private fun RoomListModalBottomSheetContent(
 @Composable
 internal fun RoomListModalBottomSheetContentPreview() = ElementPreview {
     RoomListModalBottomSheetContent(
-        contextMenu = RoomListState.ContextMenu.Shown(
-            roomId = RoomId(value = "!aRoom:aDomain"),
-            roomName = "aRoom",
-            isDm = false,
-            hasNewContent = true,
-        ),
+        contextMenu = aContextMenuShown(hasNewContent = true),
         onRoomMarkReadClicked = {},
         onRoomMarkUnreadClicked = {},
         onRoomSettingsClicked = {},
@@ -181,12 +176,7 @@ internal fun RoomListModalBottomSheetContentPreview() = ElementPreview {
 @Composable
 internal fun RoomListModalBottomSheetContentForDmPreview() = ElementPreview {
     RoomListModalBottomSheetContent(
-        contextMenu = RoomListState.ContextMenu.Shown(
-            roomId = RoomId(value = "!aRoom:aDomain"),
-            roomName = "aRoom",
-            isDm = true,
-            hasNewContent = false,
-        ),
+        contextMenu = aContextMenuShown(isDm = true),
         onRoomMarkReadClicked = {},
         onRoomMarkUnreadClicked = {},
         onRoomSettingsClicked = {},
diff --git a/features/roomlist/impl/src/main/kotlin/io/element/android/features/roomlist/impl/RoomListStateProvider.kt b/features/roomlist/impl/src/main/kotlin/io/element/android/features/roomlist/impl/RoomListStateProvider.kt
index f56359c5d1..64f75c85dc 100644
--- a/features/roomlist/impl/src/main/kotlin/io/element/android/features/roomlist/impl/RoomListStateProvider.kt
+++ b/features/roomlist/impl/src/main/kotlin/io/element/android/features/roomlist/impl/RoomListStateProvider.kt
@@ -43,14 +43,7 @@ open class RoomListStateProvider : PreviewParameterProvider<RoomListState> {
             aRoomListState().copy(invitesState = InvitesState.NewInvites),
             aRoomListState().copy(displaySearchResults = true, filter = "", filteredRoomList = persistentListOf()),
             aRoomListState().copy(displaySearchResults = true),
-            aRoomListState().copy(
-                contextMenu = RoomListState.ContextMenu.Shown(
-                    roomId = RoomId("!aRoom:aDomain"),
-                    roomName = "A nice room name",
-                    isDm = false,
-                    hasNewContent = false,
-                )
-            ),
+            aRoomListState().copy(contextMenu = aContextMenuShown(roomName = "A nice room name")),
             aRoomListState().copy(displayRecoveryKeyPrompt = true),
             aRoomListState().copy(roomList = AsyncData.Success(persistentListOf())),
             aRoomListState().copy(roomList = AsyncData.Loading(prevData = RoomListRoomSummaryFactory.createFakeList())),
@@ -104,3 +97,14 @@ internal fun aRoomListRoomSummaryList(): ImmutableList<RoomListRoomSummary> {
         ),
     )
 }
+
+internal fun aContextMenuShown(
+    roomName: String = "aRoom",
+    isDm: Boolean = false,
+    hasNewContent: Boolean = false,
+) = RoomListState.ContextMenu.Shown(
+    roomId = RoomId("!aRoom:aDomain"),
+    roomName = roomName,
+    isDm = isDm,
+    hasNewContent = hasNewContent,
+)