review: Make widget prop mandatory

Author: BillCarsonFr

src/RichError.tsx

@@ -10,6 +10,7 @@ import { PopOutIcon } from "@vector-im/compound-design-tokens/assets/web/icons";
 
 import type { FC, ReactNode } from "react";
 import { ErrorView } from "./ErrorView";
+import { widget } from "./widget.ts";
 
 /**
  * An error consisting of a terse message to be logged to the console and a
@@ -32,7 +33,7 @@ const OpenElsewhere: FC = () => {
 
   return (
     <ErrorView
-      widget={null}
+      widget={widget}
       Icon={PopOutIcon}
       title={t("error.open_elsewhere")}
     >

src/livekit/useECConnectionState.test.tsx

@@ -60,7 +60,7 @@ test.each<[string, ConnectionError]>([
     const user = userEvent.setup();
     render(
       <MemoryRouter>
-        <GroupCallErrorBoundary recoveryActionHandler={vi.fn()}>
+        <GroupCallErrorBoundary recoveryActionHandler={vi.fn()} widget={null}>
           <TestComponent />
         </GroupCallErrorBoundary>
       </MemoryRouter>,

src/room/GroupCallErrorBoundary.test.tsx

@@ -67,6 +67,7 @@ test.each([
         <GroupCallErrorBoundary
           onError={onErrorMock}
           recoveryActionHandler={vi.fn()}
+          widget={null}
         >
           <TestComponent />
         </GroupCallErrorBoundary>
@@ -95,6 +96,7 @@ test("should render the error page with link back to home", async () => {
       <GroupCallErrorBoundary
         onError={onErrorMock}
         recoveryActionHandler={vi.fn()}
+        widget={null}
       >
         <TestComponent />
       </GroupCallErrorBoundary>
@@ -139,7 +141,10 @@ test("ConnectionLostError: Action handling should reset error state", async () =
 
     return (
       <BrowserRouter>
-        <GroupCallErrorBoundary recoveryActionHandler={reconnectCallback}>
+        <GroupCallErrorBoundary
+          recoveryActionHandler={reconnectCallback}
+          widget={null}
+        >
           <TestComponent fail={failState} />
         </GroupCallErrorBoundary>
       </BrowserRouter>
@@ -181,6 +186,7 @@ describe("Rageshake button", () => {
         <GroupCallErrorBoundary
           onError={vi.fn()}
           recoveryActionHandler={vi.fn()}
+          widget={null}
         >
           <TestComponent />
         </GroupCallErrorBoundary>
@@ -221,7 +227,11 @@ test("should have a close button in widget mode", async () => {
   const onErrorMock = vi.fn();
   const { asFragment } = render(
     <BrowserRouter>
-      <GroupCallErrorBoundary widget={mockWidget} onError={onErrorMock}>
+      <GroupCallErrorBoundary
+        widget={mockWidget}
+        onError={onErrorMock}
+        recoveryActionHandler={vi.fn()}
+      >
         <TestComponent />
       </GroupCallErrorBoundary>
     </BrowserRouter>,

src/room/GroupCallErrorBoundary.tsx

@@ -106,7 +106,7 @@ interface BoundaryProps {
   children: ReactNode | (() => ReactNode);
   recoveryActionHandler: RecoveryActionHandler;
   onError?: (error: unknown) => void;
-  widget?: WidgetHelpers | null;
+  widget: WidgetHelpers | null;
 }
 
 export const GroupCallErrorBoundary = ({