Avoid rendering view-message link in RoomKnocksBar unnecessarily

Signed-off-by: Charly Nguyen <[email protected]>
matrix-org · Sep 11, 2023 · 09b8f39 · 09b8f39
1 parent 3c6d8a2
commit 09b8f39
Show file tree

Hide file tree

Showing 2 changed files with 23 additions and 8 deletions.
diff --git a/src/components/views/rooms/RoomKnocksBar.tsx b/src/components/views/rooms/RoomKnocksBar.tsx
@@ -112,7 +112,7 @@ export const RoomKnocksBar: VFC<{ room: Room }> = ({ room }) => {
                 </>
             );
             names = `${knockMembers[0].name} (${knockMembers[0].userId})`;
-            link = (
+            link = knockMembers[0].events.member?.getContent().reason && (
                 <AccessibleButton
                     className="mx_RoomKnocksBar_link"
                     element="a"

diff --git a/test/components/views/rooms/RoomKnocksBar-test.tsx b/test/components/views/rooms/RoomKnocksBar-test.tsx
@@ -165,13 +165,28 @@ describe("RoomKnocksBar", () => {
                 expect(screen.getByRole("paragraph")).toHaveTextContent(`${bob.name} (${bob.userId})`);
             });
 
-            it("renders a link to open the room settings people tab", () => {
-                getComponent(room);
-                fireEvent.click(getButton("View message"));
-                expect(dis.dispatch).toHaveBeenCalledWith({
-                    action: "open_room_settings",
-                    initial_tab_id: RoomSettingsTab.People,
-                    room_id: roomId,
+            describe("when a knock reason is not provided", () => {
+                it("does not render a link to open the room settings people tab", () => {
+                    getComponent(room);
+                    expect(screen.queryByRole("button", { name: "View message" })).not.toBeInTheDocument();
+                });
+            });
+
+            describe("when a knock reason is provided", () => {
+                it("renders a link to open the room settings people tab", () => {
+                    bob.setMembershipEvent(
+                        new MatrixEvent({
+                            content: { displayname: "Bob", membership: "knock", reason: "some reason" },
+                            type: EventType.RoomMember,
+                        }),
+                    );
+                    getComponent(room);
+                    fireEvent.click(getButton("View message"));
+                    expect(dis.dispatch).toHaveBeenCalledWith({
+                        action: "open_room_settings",
+                        initial_tab_id: RoomSettingsTab.People,
+                        room_id: roomId,
+                    });
                 });
             });