Fix jest tests

element-hq · Jan 12, 2025 · eeb1852 · eeb1852
1 parent 4eace4f
commit eeb1852
Show file tree

Hide file tree

Showing 5 changed files with 26 additions and 2 deletions.
diff --git 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
@@ -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
@@ -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/...ms/__snapshots__/RoomHeader-test.tsx.snap → ...er/__snapshots__/RoomHeader-test.tsx.snap b/...ms/__snapshots__/RoomHeader-test.tsx.snap → ...er/__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/...t-tests/components/views/rooms/RoomHeader/__snapshots__/VideoRoomChatButton-test.tsx.snap b/...t-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"