From 6be06de1536c0551a066f5280b2f0ea955f545d6 Mon Sep 17 00:00:00 2001
From: Timo K <toger5@hotmail.de>
Date: Thu, 9 Apr 2026 17:12:21 +0200
Subject: [PATCH] fix tests

---
 src/components/InCallFooter.tsx               |  5 ++--
 src/room/InCallView.test.tsx                  |  2 +-
 src/room/InCallView.tsx                       |  4 ++-
 .../__snapshots__/InCallView.test.tsx.snap    | 27 +++++++++++++++++--
 4 files changed, 32 insertions(+), 6 deletions(-)

diff --git a/src/components/InCallFooter.tsx b/src/components/InCallFooter.tsx
index f5ca656c..8a25bde3 100644
--- a/src/components/InCallFooter.tsx
+++ b/src/components/InCallFooter.tsx
@@ -93,7 +93,7 @@ export const InCallFooter: FC<InCallFooterProps> = ({
       key="audio"
       enabled={audioEnabled}
       onClick={toggleAudio ?? undefined}
-      disabled={toggleAudio === null}
+      disabled={toggleAudio === undefined}
       data-testid="incall_mute"
     />,
     <VideoButton
@@ -101,7 +101,7 @@ export const InCallFooter: FC<InCallFooterProps> = ({
       key="video"
       enabled={videoEnabled}
       onClick={toggleVideo ?? undefined}
-      disabled={toggleVideo === null}
+      disabled={toggleVideo === undefined}
       data-testid="incall_videomute"
     />,
   );
@@ -137,6 +137,7 @@ export const InCallFooter: FC<InCallFooterProps> = ({
     return (
       <LoudspeakerButton
         size={buttonSize}
+        onClick={() => audioOutputSwitcher.switch()}
         isEarpieceTarget={audioOutputSwitcher.targetOutput === "earpiece"}
       />
     );
diff --git a/src/room/InCallView.test.tsx b/src/room/InCallView.test.tsx
index 0c3be63b..2f797c65 100644
--- a/src/room/InCallView.test.tsx
+++ b/src/room/InCallView.test.tsx
@@ -197,7 +197,7 @@ describe("InCallView", () => {
       const { getByRole } = createInCallView({ mediaDevices });
       // The button should be visible. When current output is "speaker",
       // the switcher targets "earpiece", so the tooltip label is "Handset".
-      const audioOutputBtn = getByRole("button", { name: "Handset" });
+      const audioOutputBtn = getByRole("switch", { name: "Handset" });
       expect(audioOutputBtn).toBeVisible();
 
       await user.click(audioOutputBtn);
diff --git a/src/room/InCallView.tsx b/src/room/InCallView.tsx
index 12374d05..56cc02e1 100644
--- a/src/room/InCallView.tsx
+++ b/src/room/InCallView.tsx
@@ -565,12 +565,14 @@ export const InCallView: FC<InCallViewProps> = ({
     <SettingsButton key="settings" onClick={openSettings} />,
   );
 
+  const footerNotNeeded =
+    showControls === false && headerStyle === HeaderStyle.None;
   const footer = (
     <InCallFooter
       ref={footerRef}
       asOverlay={windowMode === "flat"}
       // TODO this should be computed in the view model!
-      showFooter={!showFooter || (!showControls && headerStyle === "none")}
+      showFooter={showFooter && !footerNotNeeded}
       showControls={showControls}
       showLogo={headerStyle !== HeaderStyle.None}
       showSettingsButton={headerStyle !== HeaderStyle.AppBar}
diff --git a/src/room/__snapshots__/InCallView.test.tsx.snap b/src/room/__snapshots__/InCallView.test.tsx.snap
index ddd85fe2..81c9c344 100644
--- a/src/room/__snapshots__/InCallView.test.tsx.snap
+++ b/src/room/__snapshots__/InCallView.test.tsx.snap
@@ -345,7 +345,30 @@ exports[`InCallView > rendering > renders 1`] = `
           </svg>
         </button>
         <button
+          aria-checked="false"
           aria-labelledby="_r_n_"
+          class="_button_13vu4_8 shareScreen _has-icon_13vu4_60 _icon-only_13vu4_53"
+          data-kind="secondary"
+          data-size="lg"
+          data-testid="incall_screenshare"
+          role="switch"
+          tabindex="0"
+        >
+          <svg
+            aria-hidden="true"
+            fill="currentColor"
+            height="24"
+            viewBox="0 0 24 24"
+            width="24"
+            xmlns="http://www.w3.org/2000/svg"
+          >
+            <path
+              d="M1.288 20.713Q1.575 21 2 21h20q.424 0 .712-.288A.97.97 0 0 0 23 20a.97.97 0 0 0-.288-.712A.97.97 0 0 0 22 19H2a.97.97 0 0 0-.712.288A.97.97 0 0 0 1 20q0 .424.288.712m1.3-3.299A1.93 1.93 0 0 1 2 16V5q0-.824.587-1.412A1.93 1.93 0 0 1 4 3h16q.824 0 1.413.587Q22 4.176 22 5v11q0 .824-.587 1.413A1.93 1.93 0 0 1 20 18H4q-.824 0-1.412-.587m10.12-10.12a1 1 0 0 0-1.415 0l-2.5 2.5a1 1 0 0 0 1.414 1.414l.793-.793V13a1 1 0 1 0 2 0v-2.586l.793.793a1 1 0 0 0 1.414-1.414z"
+            />
+          </svg>
+        </button>
+        <button
+          aria-labelledby="_r_s_"
           class="_button_13vu4_8 endCall _has-icon_13vu4_60 _icon-only_13vu4_53 _destructive_13vu4_110"
           data-kind="primary"
           data-size="lg"
@@ -371,7 +394,7 @@ exports[`InCallView > rendering > renders 1`] = `
         class="toggle layout"
       >
         <input
-          aria-labelledby="_r_s_"
+          aria-labelledby="_r_11_"
           name="layout"
           type="radio"
           value="spotlight"
@@ -389,7 +412,7 @@ exports[`InCallView > rendering > renders 1`] = `
           />
         </svg>
         <input
-          aria-labelledby="_r_11_"
+          aria-labelledby="_r_16_"
           checked=""
           name="layout"
           type="radio"