From 4d116f57fd5fbc9f2439ec9a3f7875dcaaa8d609 Mon Sep 17 00:00:00 2001
From: Michael Telatynski <7t3chguy@gmail.com>
Date: Fri, 29 Dec 2023 10:40:21 +0000
Subject: [PATCH 01/12] Fix CSS stacking contexts for Dialogs &
 PersistedElement

Signed-off-by: Michael Telatynski <7t3chguy@gmail.com>
---
 res/css/_common.pcss                               | 6 ++++++
 src/components/views/elements/PersistedElement.tsx | 1 +
 2 files changed, 7 insertions(+)

diff --git a/res/css/_common.pcss b/res/css/_common.pcss
index 29bb165e7b1..63a6f989831 100644
--- a/res/css/_common.pcss
+++ b/res/css/_common.pcss
@@ -56,6 +56,12 @@ limitations under the License.
     /* This is required to ensure Compound tooltips correctly draw where they should with z-index: auto */
     contain: strict;
 }
+.mx_PersistedElement_container,
+#mx_Dialog_Container,
+#mx_Dialog_StaticContainer {
+    /* This is required to ensure Compound tooltips correctly draw where they should with z-index: auto */
+    isolation: isolate;
+}
 
 /**
  * We need to increase the specificity of the selector to override the
diff --git a/src/components/views/elements/PersistedElement.tsx b/src/components/views/elements/PersistedElement.tsx
index 2d4c763eb20..28f876d967f 100644
--- a/src/components/views/elements/PersistedElement.tsx
+++ b/src/components/views/elements/PersistedElement.tsx
@@ -39,6 +39,7 @@ function getOrCreateContainer(containerId: string): HTMLDivElement {
     if (!container) {
         container = document.createElement("div");
         container.id = containerId;
+        container.className = "mx_PersistedElement_container";
         document.body.appendChild(container);
     }
 

From 53e30e21055f28fe1c6e50e80b24007743e93047 Mon Sep 17 00:00:00 2001
From: Michael Telatynski <7t3chguy@gmail.com>
Date: Fri, 29 Dec 2023 11:22:29 +0000
Subject: [PATCH 02/12] Switch to PersistedElement sharing a CSS stacking
 context for z-index to continue functioning

Signed-off-by: Michael Telatynski <7t3chguy@gmail.com>
---
 res/css/_common.pcss                             |  2 +-
 .../views/elements/PersistedElement.tsx          | 16 ++++++++++++++--
 2 files changed, 15 insertions(+), 3 deletions(-)

diff --git a/res/css/_common.pcss b/res/css/_common.pcss
index 63a6f989831..6a95408a1cb 100644
--- a/res/css/_common.pcss
+++ b/res/css/_common.pcss
@@ -56,7 +56,7 @@ limitations under the License.
     /* This is required to ensure Compound tooltips correctly draw where they should with z-index: auto */
     contain: strict;
 }
-.mx_PersistedElement_container,
+#mx_PersistedElement_container,
 #mx_Dialog_Container,
 #mx_Dialog_StaticContainer {
     /* This is required to ensure Compound tooltips correctly draw where they should with z-index: auto */
diff --git a/src/components/views/elements/PersistedElement.tsx b/src/components/views/elements/PersistedElement.tsx
index 28f876d967f..99730ec3443 100644
--- a/src/components/views/elements/PersistedElement.tsx
+++ b/src/components/views/elements/PersistedElement.tsx
@@ -29,6 +29,19 @@ export const getPersistKey = (appId: string): string => "widget_" + appId;
 // of doing reusable widgets like dialog boxes & menus where we go and
 // pass in a custom control as the actual body.
 
+// We contain all persisted elements within a master container to allow them all to be within the same
+// CSS stacking context, and thus be able to control their z-indexes relative to each other.
+function getOrCreateMasterContainer(): HTMLDivElement {
+    let container = getContainer("mx_PersistedElement_container");
+    if (!container) {
+        container = document.createElement("div");
+        container.id = "mx_PersistedElement_container";
+        document.body.appendChild(container);
+    }
+
+    return container;
+}
+
 function getContainer(containerId: string): HTMLDivElement {
     return document.getElementById(containerId) as HTMLDivElement;
 }
@@ -39,8 +52,7 @@ function getOrCreateContainer(containerId: string): HTMLDivElement {
     if (!container) {
         container = document.createElement("div");
         container.id = containerId;
-        container.className = "mx_PersistedElement_container";
-        document.body.appendChild(container);
+        getOrCreateMasterContainer().appendChild(container);
     }
 
     return container;

From c9783dea406359c6eb66313cd3125e9dd0ccc305 Mon Sep 17 00:00:00 2001
From: Michael Telatynski <7t3chguy@gmail.com>
Date: Fri, 29 Dec 2023 11:26:09 +0000
Subject: [PATCH 03/12] Fix Widget PIP overlay being under the widget and
 dragging being broken

Signed-off-by: Michael Telatynski <7t3chguy@gmail.com>
---
 res/css/components/views/pips/_WidgetPip.pcss | 19 +++++--
 src/components/views/elements/AppTile.tsx     | 27 +++++-----
 .../views/elements/PersistentApp.tsx          |  1 +
 src/components/views/pips/WidgetPip.tsx       | 49 ++++++++++---------
 4 files changed, 59 insertions(+), 37 deletions(-)

diff --git a/res/css/components/views/pips/_WidgetPip.pcss b/res/css/components/views/pips/_WidgetPip.pcss
index f6bf5a2a63e..db619bd4727 100644
--- a/res/css/components/views/pips/_WidgetPip.pcss
+++ b/res/css/components/views/pips/_WidgetPip.pcss
@@ -14,15 +14,25 @@ See the License for the specific language governing permissions and
 limitations under the License.
 */
 
+$width: 320px;
+$height: 220px;
+
 .mx_WidgetPip {
-    width: 320px;
-    height: 220px;
+    width: $width;
+    height: $height;
     border-radius: 8px;
     contain: paint;
     color: $call-primary-content;
     cursor: pointer;
 }
 
+.mx_WidgetPip_overlay {
+    width: $width;
+    height: $height;
+    position: absolute;
+    top: 0;
+}
+
 .mx_WidgetPip_header,
 .mx_WidgetPip_footer {
     position: absolute;
@@ -31,8 +41,11 @@ limitations under the License.
     width: 100%;
     box-sizing: border-box;
     transition: opacity ease 0.15s;
+}
 
-    .mx_WidgetPip:not(:hover) > & {
+.mx_WidgetPip_overlay:not(:hover) {
+    .mx_WidgetPip_header,
+    .mx_WidgetPip_footer {
         opacity: 0;
     }
 }
diff --git a/src/components/views/elements/AppTile.tsx b/src/components/views/elements/AppTile.tsx
index 56a1969c1b0..c33cfc571c3 100644
--- a/src/components/views/elements/AppTile.tsx
+++ b/src/components/views/elements/AppTile.tsx
@@ -93,6 +93,8 @@ interface IProps {
     showLayoutButtons?: boolean;
     // Handle to manually notify the PersistedElement that it needs to move
     movePersistedElement?: MutableRefObject<(() => void) | undefined>;
+    // An element to render after the iframe as an overlay
+    overlay?: ReactNode;
 }
 
 interface IState {
@@ -663,17 +665,20 @@ export default class AppTile extends React.Component<IProps, IState> {
                 );
             } else {
                 appTileBody = (
-                    <div className={appTileBodyClass} style={appTileBodyStyles}>
-                        {this.state.loading && loadingElement}
-                        <iframe
-                            title={widgetTitle}
-                            allow={iframeFeatures}
-                            ref={this.iframeRefChange}
-                            src={this.sgWidget.embedUrl}
-                            allowFullScreen={true}
-                            sandbox={sandboxFlags}
-                        />
-                    </div>
+                    <>
+                        <div className={appTileBodyClass} style={appTileBodyStyles}>
+                            {this.state.loading && loadingElement}
+                            <iframe
+                                title={widgetTitle}
+                                allow={iframeFeatures}
+                                ref={this.iframeRefChange}
+                                src={this.sgWidget.embedUrl}
+                                allowFullScreen={true}
+                                sandbox={sandboxFlags}
+                            />
+                        </div>
+                        {this.props.overlay}
+                    </>
                 );
 
                 if (!this.props.userWidget) {
diff --git a/src/components/views/elements/PersistentApp.tsx b/src/components/views/elements/PersistentApp.tsx
index b48002bd158..812d3432498 100644
--- a/src/components/views/elements/PersistentApp.tsx
+++ b/src/components/views/elements/PersistentApp.tsx
@@ -58,6 +58,7 @@ export default class PersistentApp extends React.Component<IProps> {
                 showMenubar={false}
                 pointerEvents={this.props.pointerEvents}
                 movePersistedElement={this.props.movePersistedElement}
+                overlay={this.props.children}
             />
         );
     }
diff --git a/src/components/views/pips/WidgetPip.tsx b/src/components/views/pips/WidgetPip.tsx
index 8a9bae1cdec..4a0d5e66180 100644
--- a/src/components/views/pips/WidgetPip.tsx
+++ b/src/components/views/pips/WidgetPip.tsx
@@ -107,34 +107,37 @@ export const WidgetPip: FC<Props> = ({ widgetId, room, viewingRoom, onStartMovin
 
     return (
         <div className="mx_WidgetPip" onMouseDown={onStartMoving} onClick={onBackClick}>
-            <Toolbar className="mx_WidgetPip_header">
-                <RovingAccessibleButton
-                    onClick={onBackClick}
-                    className="mx_WidgetPip_backButton"
-                    aria-label={_t("action|back")}
-                >
-                    <BackIcon className="mx_Icon mx_Icon_16" />
-                    {roomName}
-                </RovingAccessibleButton>
-            </Toolbar>
             <PersistentApp
                 persistentWidgetId={widgetId}
                 persistentRoomId={room.roomId}
                 pointerEvents="none"
                 movePersistedElement={movePersistedElement}
-            />
-            {(call !== null || WidgetType.JITSI.matches(widget?.type)) && (
-                <Toolbar className="mx_WidgetPip_footer">
-                    <RovingAccessibleTooltipButton
-                        onClick={onLeaveClick}
-                        tooltip={_t("action|leave")}
-                        aria-label={_t("action|leave")}
-                        alignment={Alignment.Top}
-                    >
-                        <HangupIcon className="mx_Icon mx_Icon_24" />
-                    </RovingAccessibleTooltipButton>
-                </Toolbar>
-            )}
+            >
+                <div onMouseDown={onStartMoving} className="mx_WidgetPip_overlay">
+                    <Toolbar className="mx_WidgetPip_header">
+                        <RovingAccessibleButton
+                            onClick={onBackClick}
+                            className="mx_WidgetPip_backButton"
+                            aria-label={_t("action|back")}
+                        >
+                            <BackIcon className="mx_Icon mx_Icon_16" />
+                            {roomName}
+                        </RovingAccessibleButton>
+                    </Toolbar>
+                    {(call !== null || WidgetType.JITSI.matches(widget?.type)) && (
+                        <Toolbar className="mx_WidgetPip_footer">
+                            <RovingAccessibleTooltipButton
+                                onClick={onLeaveClick}
+                                tooltip={_t("action|leave")}
+                                aria-label={_t("action|leave")}
+                                alignment={Alignment.Top}
+                            >
+                                <HangupIcon className="mx_Icon mx_Icon_24" />
+                            </RovingAccessibleTooltipButton>
+                        </Toolbar>
+                    )}
+                </div>
+            </PersistentApp>
         </div>
     );
 };

From 02177e9812110ea0ee4f4adc51bf6a11e25a4f14 Mon Sep 17 00:00:00 2001
From: Michael Telatynski <7t3chguy@gmail.com>
Date: Tue, 2 Jan 2024 10:47:36 +0000
Subject: [PATCH 04/12] Fix border-radius on widget pip

Signed-off-by: Michael Telatynski <7t3chguy@gmail.com>
---
 res/css/components/views/pips/_WidgetPip.pcss | 2 ++
 res/css/views/rooms/_AppsDrawer.pcss          | 3 +++
 2 files changed, 5 insertions(+)

diff --git a/res/css/components/views/pips/_WidgetPip.pcss b/res/css/components/views/pips/_WidgetPip.pcss
index db619bd4727..eee9a1bd049 100644
--- a/res/css/components/views/pips/_WidgetPip.pcss
+++ b/res/css/components/views/pips/_WidgetPip.pcss
@@ -31,6 +31,8 @@ $height: 220px;
     height: $height;
     position: absolute;
     top: 0;
+    border-radius: 8px;
+    overflow: hidden;
 }
 
 .mx_WidgetPip_header,
diff --git a/res/css/views/rooms/_AppsDrawer.pcss b/res/css/views/rooms/_AppsDrawer.pcss
index 99d64739fa5..fc595683226 100644
--- a/res/css/views/rooms/_AppsDrawer.pcss
+++ b/res/css/views/rooms/_AppsDrawer.pcss
@@ -325,6 +325,9 @@ limitations under the License.
     &.mx_AppTileBody--call {
         border-radius: 0px;
     }
+    &.mx_AppTileBody--call.mx_AppTileBody--mini {
+        border-radius: 8px;
+    }
 }
 
 /* appTileBody is embedded to PersistedElement outside of mx_AppTile,

From 76d5d199e05b3ed2b1d30790c56890bed09bf7bb Mon Sep 17 00:00:00 2001
From: Michael Telatynski <7t3chguy@gmail.com>
Date: Tue, 2 Jan 2024 11:08:12 +0000
Subject: [PATCH 05/12] Fix majority of tests

Signed-off-by: Michael Telatynski <7t3chguy@gmail.com>
---
 test/components/structures/PipContainer-test.tsx | 14 ++++++++++++++
 1 file changed, 14 insertions(+)

diff --git a/test/components/structures/PipContainer-test.tsx b/test/components/structures/PipContainer-test.tsx
index 8851c53dd4e..b4f4d7ad803 100644
--- a/test/components/structures/PipContainer-test.tsx
+++ b/test/components/structures/PipContainer-test.tsx
@@ -64,6 +64,16 @@ import { WidgetType } from "../../../src/widgets/WidgetType";
 import { SdkContextClass } from "../../../src/contexts/SDKContext";
 import { ElementWidgetActions } from "../../../src/stores/widgets/ElementWidgetActions";
 
+jest.mock("../../../src/stores/OwnProfileStore", () => ({
+    OwnProfileStore: {
+        instance: {
+            isProfileInfoFetched: true,
+            removeListener: jest.fn(),
+            getHttpAvatarUrl: jest.fn().mockReturnValue("http://avatar_url"),
+        },
+    },
+}));
+
 describe("PipContainer", () => {
     useMockedCalls();
     jest.spyOn(HTMLMediaElement.prototype, "play").mockImplementation(async () => {});
@@ -91,6 +101,8 @@ describe("PipContainer", () => {
 
         stubClient();
         client = mocked(MatrixClientPeg.safeGet());
+        client.getUserId.mockReturnValue("@alice:example.org");
+        client.getSafeUserId.mockReturnValue("@alice:example.org");
         DMRoomMap.makeShared(client);
 
         room = new Room("!1:example.org", client, "@alice:example.org", {
@@ -161,6 +173,7 @@ describe("PipContainer", () => {
         if (!(call instanceof MockedCall)) throw new Error("Failed to create call");
 
         const widget = new Widget(call.widget);
+        WidgetStore.instance.addVirtualWidget(call.widget, room.roomId);
         WidgetMessagingStore.instance.storeMessaging(widget, room.roomId, {
             stop: () => {},
         } as unknown as ClientWidgetApi);
@@ -175,6 +188,7 @@ describe("PipContainer", () => {
         cleanup();
         call.destroy();
         ActiveWidgetStore.instance.destroyPersistentWidget(widget.id, room.roomId);
+        WidgetStore.instance.removeVirtualWidget(widget.id, room.roomId);
     };
 
     const withWidget = async (fn: () => Promise<void>): Promise<void> => {

From 7ca123ad286374c497acfcc4d1f5b430f53867b6 Mon Sep 17 00:00:00 2001
From: Michael Telatynski <7t3chguy@gmail.com>
Date: Tue, 2 Jan 2024 11:08:24 +0000
Subject: [PATCH 06/12] Fix jest retryTimes applying outside of CI

Signed-off-by: Michael Telatynski <7t3chguy@gmail.com>
---
 test/setupTests.ts | 8 +++++---
 1 file changed, 5 insertions(+), 3 deletions(-)

diff --git a/test/setupTests.ts b/test/setupTests.ts
index f5282231dc7..fa99913b564 100644
--- a/test/setupTests.ts
+++ b/test/setupTests.ts
@@ -38,9 +38,11 @@ beforeEach(() => {
 });
 
 // Retry to work around our flaky app & tests
-jest.retryTimes(2, {
-    logErrorsBeforeRetry: true,
-});
+if (process.env.CI) {
+    jest.retryTimes(2, {
+        logErrorsBeforeRetry: true,
+    });
+}
 
 // Very carefully enable the mocks for everything else in
 // a specific order. We use this order to ensure we properly

From fbdcd96be6f69a6e8cadf9600ce3ecdc5b0b36d5 Mon Sep 17 00:00:00 2001
From: Michael Telatynski <7t3chguy@gmail.com>
Date: Tue, 2 Jan 2024 11:14:06 +0000
Subject: [PATCH 07/12] Fix remaining tests

Signed-off-by: Michael Telatynski <7t3chguy@gmail.com>
---
 test/components/structures/PipContainer-test.tsx | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/test/components/structures/PipContainer-test.tsx b/test/components/structures/PipContainer-test.tsx
index b4f4d7ad803..2af7f05000b 100644
--- a/test/components/structures/PipContainer-test.tsx
+++ b/test/components/structures/PipContainer-test.tsx
@@ -279,7 +279,7 @@ describe("PipContainer", () => {
         const widget = WidgetStore.instance.addVirtualWidget(
             {
                 id: "1",
-                creatorUserId: "@alice:exaxmple.org",
+                creatorUserId: "@alice:example.org",
                 type: WidgetType.CUSTOM.preferred,
                 url: "https://example.org",
                 name: "Example widget",
@@ -293,7 +293,7 @@ describe("PipContainer", () => {
 
             // The return button should maximize the widget
             const moveSpy = jest.spyOn(WidgetLayoutStore.instance, "moveToContainer");
-            await user.click(screen.getByRole("button", { name: "Back" }));
+            await user.click(await screen.findByRole("button", { name: "Back" }));
             expect(moveSpy).toHaveBeenCalledWith(room, widget, Container.Center);
 
             expect(screen.queryByRole("button", { name: "Leave" })).toBeNull();
@@ -309,7 +309,7 @@ describe("PipContainer", () => {
         const widget = WidgetStore.instance.addVirtualWidget(
             {
                 id: "1",
-                creatorUserId: "@alice:exaxmple.org",
+                creatorUserId: "@alice:example.org",
                 type: WidgetType.JITSI.preferred,
                 url: "https://meet.example.org",
                 name: "Jitsi example",
@@ -324,7 +324,7 @@ describe("PipContainer", () => {
             // The return button should view the room
             const dispatcherSpy = jest.fn();
             const dispatcherRef = defaultDispatcher.register(dispatcherSpy);
-            await user.click(screen.getByRole("button", { name: "Back" }));
+            await user.click(await screen.findByRole("button", { name: "Back" }));
             expect(dispatcherSpy).toHaveBeenCalledWith({
                 action: Action.ViewRoom,
                 room_id: room.roomId,

From 977216cb90ddcde4098616570047d9f31798267e Mon Sep 17 00:00:00 2001
From: Michael Telatynski <7t3chguy@gmail.com>
Date: Tue, 2 Jan 2024 11:25:03 +0000
Subject: [PATCH 08/12] Fix React unique key warnings

Signed-off-by: Michael Telatynski <7t3chguy@gmail.com>
---
 src/components/structures/PipContainer.tsx | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/src/components/structures/PipContainer.tsx b/src/components/structures/PipContainer.tsx
index 6f249fd6779..a7676f5bbce 100644
--- a/src/components/structures/PipContainer.tsx
+++ b/src/components/structures/PipContainer.tsx
@@ -305,6 +305,7 @@ class PipContainerInner extends React.Component<IProps, IState> {
             const call = this.state.primaryCall;
             pipContent.push(({ onStartMoving, onResize }) => (
                 <LegacyCallView
+                    key="call-view"
                     onMouseDownOnHeader={onStartMoving}
                     call={call}
                     secondaryCall={this.state.secondaryCall}
@@ -317,6 +318,7 @@ class PipContainerInner extends React.Component<IProps, IState> {
         if (this.state.showWidgetInPip && this.state.persistentWidgetId) {
             pipContent.push(({ onStartMoving }) => (
                 <WidgetPip
+                    key="widget-pip"
                     widgetId={this.state.persistentWidgetId!}
                     room={MatrixClientPeg.safeGet().getRoom(this.state.persistentRoomId ?? undefined)!}
                     viewingRoom={this.state.viewedRoomId === this.state.persistentRoomId}

From 1678e4b4ff4d36b0acc5c05d8c68aa46f68029e4 Mon Sep 17 00:00:00 2001
From: Michael Telatynski <7t3chguy@gmail.com>
Date: Tue, 2 Jan 2024 15:23:26 +0000
Subject: [PATCH 09/12] Fix sticker picker

Signed-off-by: Michael Telatynski <7t3chguy@gmail.com>
---
 res/css/_common.pcss | 1 +
 1 file changed, 1 insertion(+)

diff --git a/res/css/_common.pcss b/res/css/_common.pcss
index 6a95408a1cb..8f71bd311ca 100644
--- a/res/css/_common.pcss
+++ b/res/css/_common.pcss
@@ -56,6 +56,7 @@ limitations under the License.
     /* This is required to ensure Compound tooltips correctly draw where they should with z-index: auto */
     contain: strict;
 }
+.mx_ContextualMenu_Container,
 #mx_PersistedElement_container,
 #mx_Dialog_Container,
 #mx_Dialog_StaticContainer {

From 80baf347a5b7913a5083d0b1b2c4245dada4c966 Mon Sep 17 00:00:00 2001
From: Michael Telatynski <7t3chguy@gmail.com>
Date: Tue, 2 Jan 2024 15:47:52 +0000
Subject: [PATCH 10/12] id not class

Signed-off-by: Michael Telatynski <7t3chguy@gmail.com>
---
 res/css/_common.pcss | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/res/css/_common.pcss b/res/css/_common.pcss
index 8f71bd311ca..c960b0f4a19 100644
--- a/res/css/_common.pcss
+++ b/res/css/_common.pcss
@@ -56,7 +56,7 @@ limitations under the License.
     /* This is required to ensure Compound tooltips correctly draw where they should with z-index: auto */
     contain: strict;
 }
-.mx_ContextualMenu_Container,
+#mx_ContextualMenu_Container,
 #mx_PersistedElement_container,
 #mx_Dialog_Container,
 #mx_Dialog_StaticContainer {

From caefcf4e43a46986d5a9fd3c4b8d93a07460c879 Mon Sep 17 00:00:00 2001
From: Michael Telatynski <7t3chguy@gmail.com>
Date: Thu, 4 Jan 2024 13:33:01 +0000
Subject: [PATCH 11/12] Fix widget pip button colour in light theme

Signed-off-by: Michael Telatynski <7t3chguy@gmail.com>
---
 res/css/components/views/pips/_WidgetPip.pcss | 6 ++----
 1 file changed, 2 insertions(+), 4 deletions(-)

diff --git a/res/css/components/views/pips/_WidgetPip.pcss b/res/css/components/views/pips/_WidgetPip.pcss
index eee9a1bd049..bc0419a4937 100644
--- a/res/css/components/views/pips/_WidgetPip.pcss
+++ b/res/css/components/views/pips/_WidgetPip.pcss
@@ -20,10 +20,6 @@ $height: 220px;
 .mx_WidgetPip {
     width: $width;
     height: $height;
-    border-radius: 8px;
-    contain: paint;
-    color: $call-primary-content;
-    cursor: pointer;
 }
 
 .mx_WidgetPip_overlay {
@@ -33,6 +29,8 @@ $height: 220px;
     top: 0;
     border-radius: 8px;
     overflow: hidden;
+    color: $call-primary-content;
+    cursor: pointer;
 }
 
 .mx_WidgetPip_header,

From bae92891f5ca71d296ff8e096523137a6d37d135 Mon Sep 17 00:00:00 2001
From: Michael Telatynski <7t3chguy@gmail.com>
Date: Mon, 8 Jan 2024 08:41:38 +0000
Subject: [PATCH 12/12] Revert unrelated change

Signed-off-by: Michael Telatynski <7t3chguy@gmail.com>
---
 test/setupTests.ts | 8 +++-----
 1 file changed, 3 insertions(+), 5 deletions(-)

diff --git a/test/setupTests.ts b/test/setupTests.ts
index fa99913b564..f5282231dc7 100644
--- a/test/setupTests.ts
+++ b/test/setupTests.ts
@@ -38,11 +38,9 @@ beforeEach(() => {
 });
 
 // Retry to work around our flaky app & tests
-if (process.env.CI) {
-    jest.retryTimes(2, {
-        logErrorsBeforeRetry: true,
-    });
-}
+jest.retryTimes(2, {
+    logErrorsBeforeRetry: true,
+});
 
 // Very carefully enable the mocks for everything else in
 // a specific order. We use this order to ensure we properly