From eeb18521c9a197f01988a77fd0347e978ed5418f Mon Sep 17 00:00:00 2001
From: R Midhun Suresh <hi@midhun.dev>
Date: Mon, 13 Jan 2025 02:51:54 +0530
Subject: [PATCH] Fix jest tests

---
 .../rooms/RoomHeader/highlight/useToggled.tsx     |  2 +-
 .../__snapshots__/RoomView-test.tsx.snap          | 15 +++++++++++++++
 .../views/rooms/RoomHeader/RoomHeader-test.tsx    |  8 +++++++-
 .../__snapshots__/RoomHeader-test.tsx.snap        |  2 ++
 .../VideoRoomChatButton-test.tsx.snap             |  1 +
 5 files changed, 26 insertions(+), 2 deletions(-)
 rename test/unit-tests/components/views/rooms/{ => RoomHeader}/__snapshots__/RoomHeader-test.tsx.snap (99%)

diff --git a/src/components/views/rooms/RoomHeader/highlight/useToggled.tsx b/src/components/views/rooms/RoomHeader/highlight/useToggled.tsx
index 28f36fe0f98..61908edcc67 100644
--- a/src/components/views/rooms/RoomHeader/highlight/useToggled.tsx
+++ b/src/components/views/rooms/RoomHeader/highlight/useToggled.tsx
@@ -16,7 +16,7 @@ import { CurrentRightPanelPhaseContext } from "../../../../../contexts/CurrentRi
 export function useToggled(phase: RightPanelPhases): boolean {
     const context = useContext(CurrentRightPanelPhaseContext);
     if (!context) {
-        throw new Error("Context is null, did you forget to use CurrentRightPanelPhaseContextProvider?");
+        return false;
     }
     const { currentPhase, isPanelOpen } = context;
     return !!(isPanelOpen && currentPhase === phase);
diff --git a/test/unit-tests/components/structures/__snapshots__/RoomView-test.tsx.snap b/test/unit-tests/components/structures/__snapshots__/RoomView-test.tsx.snap
index 65a755058bf..eb775bdcbec 100644
--- a/test/unit-tests/components/structures/__snapshots__/RoomView-test.tsx.snap
+++ b/test/unit-tests/components/structures/__snapshots__/RoomView-test.tsx.snap
@@ -110,6 +110,7 @@ exports[`RoomView for a local room in state CREATING should match the snapshot 1
           style="--cpd-icon-button-size: 100%;"
         >
           <svg
+            class=""
             fill="currentColor"
             height="1em"
             viewBox="0 0 24 24"
@@ -135,6 +136,7 @@ exports[`RoomView for a local room in state CREATING should match the snapshot 1
           style="--cpd-icon-button-size: 100%;"
         >
           <svg
+            class=""
             fill="currentColor"
             height="1em"
             viewBox="0 0 24 24"
@@ -323,6 +325,7 @@ exports[`RoomView for a local room in state ERROR should match the snapshot 1`]
           style="--cpd-icon-button-size: 100%;"
         >
           <svg
+            class=""
             fill="currentColor"
             height="1em"
             viewBox="0 0 24 24"
@@ -348,6 +351,7 @@ exports[`RoomView for a local room in state ERROR should match the snapshot 1`]
           style="--cpd-icon-button-size: 100%;"
         >
           <svg
+            class=""
             fill="currentColor"
             height="1em"
             viewBox="0 0 24 24"
@@ -621,6 +625,7 @@ exports[`RoomView for a local room in state NEW should match the snapshot 1`] =
           style="--cpd-icon-button-size: 100%;"
         >
           <svg
+            class=""
             fill="currentColor"
             height="1em"
             viewBox="0 0 24 24"
@@ -646,6 +651,7 @@ exports[`RoomView for a local room in state NEW should match the snapshot 1`] =
           style="--cpd-icon-button-size: 100%;"
         >
           <svg
+            class=""
             fill="currentColor"
             height="1em"
             viewBox="0 0 24 24"
@@ -996,6 +1002,7 @@ exports[`RoomView for a local room in state NEW that is encrypted should match t
           style="--cpd-icon-button-size: 100%;"
         >
           <svg
+            class=""
             fill="currentColor"
             height="1em"
             viewBox="0 0 24 24"
@@ -1021,6 +1028,7 @@ exports[`RoomView for a local room in state NEW that is encrypted should match t
           style="--cpd-icon-button-size: 100%;"
         >
           <svg
+            class=""
             fill="currentColor"
             height="1em"
             viewBox="0 0 24 24"
@@ -1379,6 +1387,7 @@ exports[`RoomView should not display the timeline when the room encryption is lo
               style="--cpd-icon-button-size: 100%;"
             >
               <svg
+                class=""
                 fill="currentColor"
                 height="1em"
                 viewBox="0 0 24 24"
@@ -1404,6 +1413,7 @@ exports[`RoomView should not display the timeline when the room encryption is lo
               style="--cpd-icon-button-size: 100%;"
             >
               <svg
+                class=""
                 fill="currentColor"
                 height="1em"
                 viewBox="0 0 24 24"
@@ -1585,6 +1595,7 @@ exports[`RoomView should not display the timeline when the room encryption is lo
               style="--cpd-icon-button-size: 100%;"
             >
               <svg
+                class=""
                 fill="currentColor"
                 height="1em"
                 viewBox="0 0 24 24"
@@ -1610,6 +1621,7 @@ exports[`RoomView should not display the timeline when the room encryption is lo
               style="--cpd-icon-button-size: 100%;"
             >
               <svg
+                class=""
                 fill="currentColor"
                 height="1em"
                 viewBox="0 0 24 24"
@@ -1912,6 +1924,7 @@ exports[`RoomView video rooms should render joined video room view 1`] = `
               style="--cpd-icon-button-size: 100%;"
             >
               <svg
+                class="mx_RoomHeader_toggled"
                 fill="currentColor"
                 height="1em"
                 viewBox="0 0 24 24"
@@ -1937,6 +1950,7 @@ exports[`RoomView video rooms should render joined video room view 1`] = `
               style="--cpd-icon-button-size: 100%;"
             >
               <svg
+                class=""
                 fill="currentColor"
                 height="1em"
                 viewBox="0 0 24 24"
@@ -1962,6 +1976,7 @@ exports[`RoomView video rooms should render joined video room view 1`] = `
               style="--cpd-icon-button-size: 100%;"
             >
               <svg
+                class=""
                 fill="currentColor"
                 height="1em"
                 viewBox="0 0 24 24"
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 481f8501cda..9a752eee1e3 100644
--- a/test/unit-tests/components/views/rooms/RoomHeader/RoomHeader-test.tsx
+++ b/test/unit-tests/components/views/rooms/RoomHeader/RoomHeader-test.tsx
@@ -57,7 +57,12 @@ import { SdkContextClass } from "../../../../../../src/contexts/SDKContext";
 import WidgetStore, { IApp } from "../../../../../../src/stores/WidgetStore";
 import { UIFeature } from "../../../../../../src/settings/UIFeature";
 
-jest.mock("../../../../../src/utils/ShieldUtils");
+jest.mock("../../../../../../src/utils/ShieldUtils");
+jest.mock("../../../../../../src/hooks/right-panel/useCurrentPhase", () => ({
+    useCurrentPhase: () => {
+        return { currentPhase: "foo", isOpen: false };
+    },
+}));
 
 function getWrapper(): RenderOptions {
     return {
@@ -67,6 +72,7 @@ function getWrapper(): RenderOptions {
     };
 }
 
+
 describe("RoomHeader", () => {
     filterConsole(
         "[getType] Room !1:example.org does not have an m.room.create event",
diff --git a/test/unit-tests/components/views/rooms/__snapshots__/RoomHeader-test.tsx.snap b/test/unit-tests/components/views/rooms/RoomHeader/__snapshots__/RoomHeader-test.tsx.snap
similarity index 99%
rename from test/unit-tests/components/views/rooms/__snapshots__/RoomHeader-test.tsx.snap
rename to test/unit-tests/components/views/rooms/RoomHeader/__snapshots__/RoomHeader-test.tsx.snap
index 3db3fb67fbf..6d0c2dc3e4a 100644
--- a/test/unit-tests/components/views/rooms/__snapshots__/RoomHeader-test.tsx.snap
+++ b/test/unit-tests/components/views/rooms/RoomHeader/__snapshots__/RoomHeader-test.tsx.snap
@@ -105,6 +105,7 @@ exports[`RoomHeader dm does not show the face pile for DMs 1`] = `
         style="--cpd-icon-button-size: 100%;"
       >
         <svg
+          class=""
           fill="currentColor"
           height="1em"
           viewBox="0 0 24 24"
@@ -130,6 +131,7 @@ exports[`RoomHeader dm does not show the face pile for DMs 1`] = `
         style="--cpd-icon-button-size: 100%;"
       >
         <svg
+          class=""
           fill="currentColor"
           height="1em"
           viewBox="0 0 24 24"
diff --git a/test/unit-tests/components/views/rooms/RoomHeader/__snapshots__/VideoRoomChatButton-test.tsx.snap b/test/unit-tests/components/views/rooms/RoomHeader/__snapshots__/VideoRoomChatButton-test.tsx.snap
index 83f023e99b2..ee5571747a9 100644
--- a/test/unit-tests/components/views/rooms/RoomHeader/__snapshots__/VideoRoomChatButton-test.tsx.snap
+++ b/test/unit-tests/components/views/rooms/RoomHeader/__snapshots__/VideoRoomChatButton-test.tsx.snap
@@ -16,6 +16,7 @@ exports[`<VideoRoomChatButton /> renders button with an unread marker when room
     style="--cpd-icon-button-size: 100%;"
   >
     <svg
+      class=""
       fill="currentColor"
       height="1em"
       viewBox="0 0 24 24"