From f83093cc44586b881d0918e4e4dee60d3263d44b Mon Sep 17 00:00:00 2001
From: David Langley <davidl@element.io>
Date: Wed, 27 Aug 2025 16:22:53 +0100
Subject: [PATCH] try fix RoomHeader tests again by also mocking
 useParticipants

---
 src/hooks/useCall.ts                          |  3 +-
 .../rooms/RoomHeader/RoomHeader-test.tsx      | 42 +++++++++++++------
 2 files changed, 31 insertions(+), 14 deletions(-)

diff --git a/src/hooks/useCall.ts b/src/hooks/useCall.ts
index 9f1fb010cb..cc32b46bc8 100644
--- a/src/hooks/useCall.ts
+++ b/src/hooks/useCall.ts
@@ -42,11 +42,10 @@ export const useConnectionState = (call: Call | null): ConnectionState =>
     );
 
 export const useParticipants = (call: Call | null): Map<RoomMember, Set<string>> => {
-    const participants = call?.participants;
     return useTypedEventEmitterState(
         call ?? undefined,
         CallEvent.Participants,
-        useCallback((state) => state ?? participants ?? new Map(), [participants]),
+        useCallback((state) => state ?? call?.participants ?? new Map(), [call]),
     );
 };
 
diff --git a/test/unit-tests/components/views/rooms/RoomHeader/RoomHeader-test.tsx b/test/unit-tests/components/views/rooms/RoomHeader/RoomHeader-test.tsx
index a9f02532c8..da9add1b54 100644
--- a/test/unit-tests/components/views/rooms/RoomHeader/RoomHeader-test.tsx
+++ b/test/unit-tests/components/views/rooms/RoomHeader/RoomHeader-test.tsx
@@ -554,7 +554,9 @@ describe("RoomHeader", () => {
         });
 
         it("join button is shown if there is an ongoing call", async () => {
-            mockRoomMembers(room, 3);
+            const members = mockRoomMembers(room, 3);
+            const participants = mockParticipants(members);
+            jest.spyOn(UseCall, "useParticipants").mockReturnValue(participants);
             jest.spyOn(UseCall, "useParticipantCount").mockReturnValue(3);
             render(<RoomHeader room={room} />, getWrapper());
             const joinButton = getByLabelText(document.body, "Join");
@@ -562,7 +564,9 @@ describe("RoomHeader", () => {
         });
 
         it("join button is disabled if there is an other ongoing call", async () => {
-            mockRoomMembers(room, 3);
+            const members = mockRoomMembers(room, 3);
+            const participants = mockParticipants(members);
+            jest.spyOn(UseCall, "useParticipants").mockReturnValue(participants);
             jest.spyOn(UseCall, "useParticipantCount").mockReturnValue(3);
             jest.spyOn(CallStore.prototype, "connectedCalls", "get").mockReturnValue(
                 new Set([{ roomId: "some_other_room" } as Call]),
@@ -582,7 +586,9 @@ describe("RoomHeader", () => {
         });
 
         it("close lobby button is shown if there is an ongoing call but we are viewing the lobby", async () => {
-            mockRoomMembers(room, 3);
+            const members = mockRoomMembers(room, 3);
+            const participants = mockParticipants(members);
+            jest.spyOn(UseCall, "useParticipants").mockReturnValue(participants);
             jest.spyOn(UseCall, "useParticipantCount").mockReturnValue(3);
             jest.spyOn(SdkContextClass.instance.roomViewStore, "isViewingCall").mockReturnValue(true);
 
@@ -792,20 +798,32 @@ describe("RoomHeader", () => {
 /**
  *
  * @param count the number of users to create
+ * @returns the created members
  */
 function mockRoomMembers(room: Room, count: number) {
     const members = Array(count)
         .fill(0)
-        .map((_, index) => ({
-            userId: `@user-${index}:example.org`,
-            name: `Member ${index}`,
-            rawDisplayName: `Member ${index}`,
-            roomId: room.roomId,
-            membership: KnownMembership.Join,
-            getAvatarUrl: () => `mxc://avatar.url/user-${index}.png`,
-            getMxcAvatarUrl: () => `mxc://avatar.url/user-${index}.png`,
-        }));
+        .map((_, index) => {
+            const userId = `@user-${index}:example.org`;
+            const member = new RoomMember(room.roomId, userId);
+            member.name = `Member ${index}`;
+            member.rawDisplayName = `Member ${index}`;
+            member.membership = KnownMembership.Join;
+            member.getAvatarUrl = () => `mxc://avatar.url/user-${index}.png`;
+            member.getMxcAvatarUrl = () => `mxc://avatar.url/user-${index}.png`;
+            return member;
+        });
 
     room.currentState.setJoinedMemberCount(members.length);
     room.getJoinedMembers = jest.fn().mockReturnValue(members);
+    return members;
+}
+
+/**
+ * Creates a participants map from room members
+ * @param members the room members to convert to participants
+ * @returns Map of participants with device sets
+ */
+function mockParticipants(members: RoomMember[]): Map<RoomMember, Set<string>> {
+    return new Map(members.map((member, index) => [member, new Set([`device${index + 1}`])]));
 }