Include the Members and Add Rooms screens in the spaces UI tests. (#4981)

* Expand the space flow UI tests to include the add rooms screen.

Also fixes some failures in these tests due to the room/space title now being a button.

* Expand the space flow UI tests to include the space members screen.

* Reset AppSettings before running preview tests.

ElementX/Sources/Mocks/JoinedRoomProxyMock.swift

@@ -203,3 +203,14 @@ private extension RoomHero {
                   avatarUrl: memberProxy.avatarURL?.absoluteString)
     }
 }
+
+@MainActor
+extension Array where Element == JoinedRoomProxyProtocol {
+    static var mockRooms: [JoinedRoomProxyProtocol] {
+        [
+            JoinedRoomProxyMock(.init(id: "1", name: "Room Name", canonicalAlias: "#room-name:example.com")),
+            JoinedRoomProxyMock(.init(id: "2", name: "Room Name", canonicalAlias: "#room-name:example.com")),
+            JoinedRoomProxyMock(.init(id: "3", name: "Room Name", canonicalAlias: "#room-name:example.com"))
+        ]
+    }
+}

ElementX/Sources/Other/AccessibilityIdentifiers.swift

@@ -43,8 +43,11 @@ enum A11yIdentifiers {
     static let notificationSettingsEditScreen = NotificationSettingsEditScreen()
     static let pollFormScreen = PollFormScreen()
     static let roomPollsHistoryScreen = RoomPollsHistoryScreen()
+    static let roomMembersListScreen = RoomMembersListScreen()
     static let manageRoomMemberSheet = ManageRoomMemberSheet()
     static let spacesScreen = SpacesScreen()
+    static let spaceScreen = SpaceScreen()
+    static let spaceAddRoomsScreen = SpaceAddRoomsScreen()
     static let linkNewDeviceScreen = LinkNewDeviceScreen()
     
     struct AlertInfo {
@@ -303,6 +306,10 @@ enum A11yIdentifiers {
         let loadMore = "room_polls_history_screen-load_more"
     }
     
+    struct RoomMembersListScreen {
+        let invite = "room_members_list_screen-invite"
+    }
+    
     struct ManageRoomMemberSheet {
         let viewProfile = "manage_room_member_sheet-view_profile"
     }
@@ -316,6 +323,16 @@ enum A11yIdentifiers {
         }
     }
     
+    struct SpaceScreen {
+        let moreMenu = "space_screen-more_menu"
+        let addExistingRooms = "space_screen-add_existing_rooms"
+        let viewMembers = "space_screen-view_members"
+    }
+    
+    struct SpaceAddRoomsScreen {
+        let cancel = "space_add_rooms_screen-cancel"
+    }
+    
     struct LinkNewDeviceScreen {
         let cancel = "link_new_device_screen-cancel"
         let mobileDevice = "link_new_device_screen-mobile_device"

ElementX/Sources/Screens/RoomMemberListScreen/View/RoomMembersListScreen.swift

@@ -102,6 +102,7 @@ struct RoomMembersListScreen: View {
                 Button(L10n.actionInvite) {
                     context.send(viewAction: .invite)
                 }
+                .accessibilityIdentifier(A11yIdentifiers.roomMembersListScreen.invite)
             }
         }
     }

ElementX/Sources/Screens/RoomScreen/View/RoomScreen.swift

@@ -226,10 +226,6 @@ struct RoomScreen_Previews: PreviewProvider, TestablePreview {
     }
     
     static func makeViewModels(canSendMessage: Bool = true, hasSuccessor: Bool = false) -> ViewModels {
-        // TimelineItemBubbledStylerView sets enableKeyShareOnInvite to true which if run
-        // before these tests, ends up adding a banner to the snapshots.
-        AppSettings.resetAllSettings()
-        
         let roomProxyMock = JoinedRoomProxyMock(.init(id: "stable_id",
                                                       name: "Preview room",
                                                       hasOngoingCall: true,

ElementX/Sources/Screens/Spaces/SpaceAddRoomsScreen/View/SpaceAddRoomsScreen.swift

@@ -115,6 +115,7 @@ struct SpaceAddRoomsScreen: View {
             ToolbarButton(role: .cancel) {
                 context.send(viewAction: .cancel)
             }
+            .accessibilityIdentifier(A11yIdentifiers.spaceAddRoomsScreen.cancel)
         }
         
         ToolbarItem(placement: .confirmationAction) {
@@ -186,11 +187,7 @@ struct SpaceAddRoomsScreen_Previews: PreviewProvider, TestablePreview {
         let spaceRoomListProxy = SpaceRoomListProxyMock(.init(spaceServiceRoom: SpaceServiceRoomMock(.init(isSpace: true))))
         
         let clientProxy = ClientProxyMock(.init())
-        clientProxy.recentlyVisitedRoomsFilterReturnValue = [
-            JoinedRoomProxyMock(.init(id: "1", name: "Room Name", canonicalAlias: "#room-name:example.com")),
-            JoinedRoomProxyMock(.init(id: "2", name: "Room Name", canonicalAlias: "#room-name:example.com")),
-            JoinedRoomProxyMock(.init(id: "3", name: "Room Name", canonicalAlias: "#room-name:example.com"))
-        ]
+        clientProxy.recentlyVisitedRoomsFilterReturnValue = .mockRooms
         
         let viewModel = SpaceAddRoomsScreenViewModel(spaceRoomListProxy: spaceRoomListProxy,
                                                      userSession: UserSessionMock(.init(clientProxy: clientProxy)),

ElementX/Sources/Screens/Spaces/SpaceScreen/View/SpaceScreen.swift

@@ -100,6 +100,8 @@ struct SpaceScreen: View {
                             Button { context.send(viewAction: .addExistingRooms) } label: {
                                 Label(L10n.actionAddExistingRooms, icon: \.room)
                             }
+                            .accessibilityIdentifier(A11yIdentifiers.spaceScreen.addExistingRooms)
+                            
                             Button { context.send(viewAction: .manageChildren) } label: {
                                 Label(L10n.actionManageRooms, icon: \.edit)
                             }
@@ -111,7 +113,9 @@ struct SpaceScreen: View {
                             Button { context.send(viewAction: .displayMembers(roomProxy: roomProxy)) } label: {
                                 Label(L10n.screenSpaceMenuActionMembers, icon: \.user)
                             }
+                            .accessibilityIdentifier(A11yIdentifiers.spaceScreen.viewMembers)
                         }
+                        
                         if let permalink = context.viewState.permalink {
                             ShareLink(item: permalink) {
                                 Label(L10n.actionShare, icon: \.shareIos)
@@ -135,6 +139,7 @@ struct SpaceScreen: View {
                     // Use an SF Symbol to match what ToolbarItemGroup(placement: .secondaryAction) would give us.
                     Image(systemSymbol: .ellipsis)
                 }
+                .accessibilityIdentifier(A11yIdentifiers.spaceScreen.moreMenu)
             }
         }
     }

ElementX/Sources/UITests/UITestsAppCoordinator.swift

@@ -596,8 +596,9 @@ class MockScreen: Identifiable {
             let clientProxy = ClientProxyMock(.init(userID: "@mock:client.com",
                                                     deviceID: "MOCKCLIENT",
                                                     roomSummaryProvider: RoomSummaryProviderMock(.init(state: .loaded(roomSummaries))),
-                                                    spaceServiceConfiguration: .init(topLevelSpaces: .mockSingleRoom),
+                                                    spaceServiceConfiguration: .init(topLevelSpaces: .mockSpaceList.filter(\.isSpace) + .mockSingleRoom),
                                                     roomPreviews: [SpaceServiceRoomProtocol].mockSpaceList.map(RoomPreviewProxyMock.init)))
+            clientProxy.recentlyVisitedRoomsFilterReturnValue = .mockRooms
             
             // The tab bar remains hidden for the non-spaces tests as we don't supply any mock spaces.
             let spaceServiceProxy = SpaceServiceProxyMock(id == .userSessionSpacesFlow ? .populated : .init())