Sulkta-OSS/element-x-ada
3
0
Fork 0
Code Issues 1 Pull requests 10 Projects Releases Packages Wiki Activity Actions

Fix regression on Preview.

Browse source
This commit is contained in:
Benoit Marty 2025-11-17 10:46:38 +01:00
parent 65ff881174
commit 6e66d63f53
1 changed files with 18 additions and 7 deletions
Download patch file Download diff file Expand all files Collapse all files

25
features/rolesandpermissions/impl/src/main/kotlin/io/element/android/features/rolesandpermissions/impl/roles/ChangeRolesStateProvider.kt
View file

@ -44,8 +44,14 @@ class ChangeRolesStateProvider : PreviewParameterProvider<ChangeRolesState> {
aChangeRolesStateWithSelectedUsers().copy(savingState = AsyncAction.Loading), aChangeRolesStateWithSelectedUsers().copy(savingState = AsyncAction.Loading),
aChangeRolesStateWithSelectedUsers().copy(savingState = AsyncAction.Success(true)), aChangeRolesStateWithSelectedUsers().copy(savingState = AsyncAction.Success(true)),
aChangeRolesStateWithSelectedUsers().copy(savingState = AsyncAction.Failure(Exception("boom"))), aChangeRolesStateWithSelectedUsers().copy(savingState = AsyncAction.Failure(Exception("boom"))),
aChangeRolesStateWithOwners(), aChangeRolesStateWithOwners(
aChangeRolesStateWithOwners().copy(role = RoomMember.Role.Owner(isCreator = false)), role = RoomMember.Role.Admin,
// Do not include the owners in the selectedUsers (the presenter will not do it), the View will add them
selectedUsers = listOf(
aMatrixUser(id = "@carol:server.org", displayName = "Carol"),
)
),
aChangeRolesStateWithOwners(role = RoomMember.Role.Owner(isCreator = false)),
) )
} }
@ -88,8 +94,15 @@ internal fun aChangeRolesStateWithSelectedUsers() = aChangeRolesState(
canRemoveMember = { it != UserId("@alice:server.org") }, canRemoveMember = { it != UserId("@alice:server.org") },
) )
internal fun aChangeRolesStateWithOwners() = aChangeRolesState( internal fun aChangeRolesStateWithOwners(
role = RoomMember.Role.Admin, role: RoomMember.Role = RoomMember.Role.Admin,
selectedUsers: List<MatrixUser> = listOf(
aMatrixUser(id = "@alice:server.org", displayName = "Alice"),
aMatrixUser(id = "@bob:server.org", displayName = "Bob"),
aMatrixUser(id = "@carol:server.org", displayName = "Carol"),
),
) = aChangeRolesState(
role = role,
searchResults = SearchBarResultState.Results( searchResults = SearchBarResultState.Results(
MembersByRole( MembersByRole(
members = persistentListOf( members = persistentListOf(
@ -125,7 +138,5 @@ internal fun aChangeRolesStateWithOwners() = aChangeRolesState(
else -> false else -> false
} }
}, },
selectedUsers = persistentListOf( selectedUsers = selectedUsers.toImmutableList(),
aMatrixUser(id = "@carol:server.org", displayName = "Carol"),
)
) )