CR suggestions

Expensify · Feb 5, 2024 · 1a5f5aa · 1a5f5aa
1 parent c6302be
commit 1a5f5aa
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/src/components/HeaderPageLayout.tsx b/src/components/HeaderPageLayout.tsx
@@ -20,7 +20,7 @@ type HeaderPageLayoutProps = ChildrenProps &
         backgroundColor?: string;
 
         /** TestID to apply to the whole section container */
-        testID?: string;
+        testID: string;
 
         /** A fixed footer to display at the bottom of the page. */
         footer?: ReactNode;
@@ -76,7 +76,7 @@ function HeaderPageLayout({
             shouldEnablePickerAvoiding={false}
             includeSafeAreaPaddingBottom={false}
             offlineIndicatorStyle={[appBGColor]}
-            testID={testID || HeaderPageLayout.displayName}
+            testID={testID}
             shouldShowOfflineIndicatorInWideScreen={shouldShowOfflineIndicatorInWideScreen}
         >
             {({safeAreaPaddingBottomStyle}) => (

diff --git a/src/components/IllustratedHeaderPageLayout.tsx b/src/components/IllustratedHeaderPageLayout.tsx
@@ -18,7 +18,7 @@ type IllustratedHeaderPageLayoutProps = HeaderPageLayoutProps & {
     overlayContent?: () => ReactNode;
 
     /** TestID to apply to the whole section container */
-    testID?: string;
+    testID: string;
 };
 
 function IllustratedHeaderPageLayout({backgroundColor, children, illustration, testID, overlayContent, ...rest}: IllustratedHeaderPageLayoutProps) {