fix(ui): validate My Data customized layout

Author: harshach

openmetadata-ui/src/main/resources/ui/src/pages/MyDataPage/MyDataPage.component.tsx

@@ -114,11 +114,11 @@ const MyDataPage = () => {
     if (!docData || !selectedPersona) {
       return getDefaultLandingPageLayout();
     }
-    const pageData = getPersonaPage(docData, PageType.LandingPage) ?? {
-      layout: [],
-      pageType: PageType.LandingPage,
-    };
-    const filteredLayout = (pageData.layout as WidgetConfig[])
+    const pageData = getPersonaPage(docData, PageType.LandingPage);
+    const customizedLayout = Array.isArray(pageData?.layout)
+      ? (pageData.layout as WidgetConfig[])
+      : [];
+    const filteredLayout = customizedLayout
       .filter(
         (widget: WidgetConfig) =>
           !widget.i.startsWith(LandingPageWidgetKeys.CURATED_ASSETS) ||

openmetadata-ui/src/main/resources/ui/src/pages/MyDataPage/MyDataPage.test.tsx

@@ -322,6 +322,33 @@ describe('MyDataPage component', () => {
     ).toBeInTheDocument();
   });
 
+  it.each([
+    ['missing', undefined],
+    ['not an array', { invalid: true }],
+  ])(
+    'MyDataPage should use the default layout when the customized layout is %s',
+    async (_description, invalidLayout) => {
+      const landingPage =
+        invalidLayout === undefined
+          ? { pageType: PageType.LandingPage }
+          : { pageType: PageType.LandingPage, layout: invalidLayout };
+
+      (getDocumentByFQN as jest.Mock).mockResolvedValueOnce({
+        ...mockDocumentData,
+        data: { pages: [landingPage] },
+      });
+
+      renderMyDataPage();
+
+      expect(
+        await screen.findByText('KnowledgePanel.DataAssets')
+      ).toBeInTheDocument();
+      expect(
+        await screen.findByText('KnowledgePanel.KnowledgeCenter')
+      ).toBeInTheDocument();
+    }
+  );
+
   it('MyDataPage should not render announcement widget if there are no announcements', async () => {
     (getActiveAnnouncements as jest.Mock).mockImplementationOnce(() =>
       Promise.resolve({

openmetadata-ui/src/main/resources/ui/src/utils/CustomizePage/PersonaPage.utils.test.ts

@@ -21,7 +21,7 @@ import {
 
 const tablePage = {
   pageType: PageType.Table,
-  layout: [],
+  tabs: [{ id: 'overview', layout: [], name: 'overview' }],
 } as unknown as Page;
 
 const dashboardPage = {
@@ -55,7 +55,7 @@ describe('PersonaPage utilities', () => {
     expect(document.data.pages).toEqual([null, tablePage, undefined]);
   });
 
-  it('preserves the document reference when pages are already valid', () => {
+  it('preserves tab-based pages without a top-level layout', () => {
     const document = createDocument([tablePage]);
 
     expect(normalizePersonaDocument(document)).toBe(document);