fix(ui): unit tests console error cleanups (#7368)

* fix(ui): unit tests console error cleanups

* fix forward ref error

openmetadata-ui/src/main/resources/ui/src/components/ActivityFeed/ActivityFeedEditor/ActivityFeedEditor.test.tsx

@@ -12,7 +12,7 @@
  */
 
 import { findByTestId, render } from '@testing-library/react';
-import React from 'react';
+import React, { forwardRef } from 'react';
 import { MemoryRouter } from 'react-router-dom';
 import ActivityFeedEditor from './ActivityFeedEditor';
 
@@ -28,16 +28,20 @@ jest.mock('../../../utils/FeedUtils', () => ({
 }));
 
 jest.mock('../../FeedEditor/FeedEditor', () => ({
-  FeedEditor: jest.fn().mockImplementation(({ onChangeHandler, onSave }) => {
+  __esModule: true,
+  FeedEditor: forwardRef(
+    jest.fn().mockImplementation(({ onChangeHandler, onSave }, ref) => {
       return (
         <div
           data-testid="feed-editor"
+          ref={ref}
           onChange={onChangeHandler}
           onClick={onSave}>
           FeedEditor
         </div>
       );
-  }),
+    })
+  ),
 }));
 
 jest.mock('./SendButton', () => ({

openmetadata-ui/src/main/resources/ui/src/components/DashboardDetails/DashboardDetails.test.tsx

@@ -12,6 +12,7 @@
  */
 
 import {
+  act,
   findByTestId,
   findByText,
   fireEvent,
@@ -427,6 +428,7 @@ describe('Test DashboardDetails component', () => {
   });
 
   it('Check that tags and glossary terms are not present', async () => {
+    await act(async () => {
       (getTagCategories as jest.Mock).mockImplementationOnce(() =>
         Promise.reject()
       );
@@ -453,3 +455,4 @@ describe('Test DashboardDetails component', () => {
       expect(glossaryTerm1).not.toBeInTheDocument();
     });
   });
+});

openmetadata-ui/src/main/resources/ui/src/components/common/entityPageInfo/EntityPageInfo.test.tsx

@@ -12,11 +12,13 @@
  */
 
 import {
+  act,
   findByTestId,
   findByText,
   fireEvent,
   queryByTestId,
   render,
+  screen,
 } from '@testing-library/react';
 import { flatten } from 'lodash';
 import { FormattedGlossaryTermData, TagOption } from 'Models';
@@ -613,29 +615,29 @@ describe('Test EntityPageInfo component', () => {
   });
 
   it('Check that tags and glossary terms are not present', async () => {
+    await act(async () => {
       (getTagCategories as jest.Mock).mockImplementationOnce(() =>
         Promise.reject()
       );
       (fetchGlossaryTerms as jest.Mock).mockImplementationOnce(() =>
         Promise.reject()
       );
-    const { getByTestId, queryByText } = render(
-      <EntityPageInfo {...mockEntityInfoProp} isTagEditable />,
-      {
-        wrapper: MemoryRouter,
-      }
-    );
 
-    const tagWrapper = getByTestId('tags-wrapper');
+      render(<EntityPageInfo {...mockEntityInfoProp} isTagEditable />, {
+        wrapper: MemoryRouter,
+      });
+      const tagWrapper = screen.getByTestId('tags-wrapper');
+
       fireEvent.click(
         tagWrapper,
         new MouseEvent('click', { bubbles: true, cancelable: true })
       );
 
-    const tag1 = queryByText('TagCat1.Tag1');
+      const tag1 = screen.queryByText('TagCat1.Tag1');
-    const glossaryTerm1 = queryByText('Glossary.Tag1');
+      const glossaryTerm1 = screen.queryByText('Glossary.Tag1');
 
       expect(tag1).not.toBeInTheDocument();
       expect(glossaryTerm1).not.toBeInTheDocument();
     });
   });
+});

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

@@ -1104,6 +1104,7 @@ describe('Test DatasetDetails page', () => {
     });
 
     it('Show error message on resolve of CTA api without response data', async () => {
+      await act(async () => {
         (patchTableDetails as jest.Mock).mockImplementation(() =>
           Promise.resolve({ data: '' })
         );
@@ -1146,7 +1147,10 @@ describe('Test DatasetDetails page', () => {
           container,
           'removeLineageHandler'
         );
-      const postFeedHandler = await findByTestId(container, 'postFeedHandler');
+        const postFeedHandler = await findByTestId(
+          container,
+          'postFeedHandler'
+        );
         const createThread = await findByTestId(container, 'createThread');
 
         expect(ContainerText).toBeInTheDocument();
@@ -1170,6 +1174,7 @@ describe('Test DatasetDetails page', () => {
         fireEvent.click(postFeedHandler);
         fireEvent.click(createThread);
       });
+    });
 
     it('Show error message on resolve of getUpdatedThread api without response data', async () => {
       (getUpdatedThread as jest.Mock).mockImplementationOnce(() =>

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

@@ -213,6 +213,7 @@ describe('Test GlossaryComponent page', () => {
   });
 
   it('All Function call should work properly - part 2', async () => {
+    await act(async () => {
       render(<GlossaryPageV1 />);
 
       const glossaryComponent = await screen.findByText(/Glossary.component/i);
@@ -222,7 +223,9 @@ describe('Test GlossaryComponent page', () => {
       const handleAddGlossaryTermClick = await screen.findByTestId(
         'handleAddGlossaryTermClick'
       );
-    const handleChildLoading = await screen.findByTestId('handleChildLoading');
+      const handleChildLoading = await screen.findByTestId(
+        'handleChildLoading'
+      );
       const handleExpandedKey = await screen.findByTestId('handleExpandedKey');
       const handleGlossaryDelete = await screen.findByTestId(
         'handleGlossaryDelete'
@@ -248,6 +251,7 @@ describe('Test GlossaryComponent page', () => {
 
       fireEvent.click(handleSearchText);
     });
+  });
 
   describe('Render Sad Paths', () => {
     it('show error if deleteGlossaryTerm API fails', async () => {
@@ -261,8 +265,10 @@ describe('Test GlossaryComponent page', () => {
 
       expect(handleGlossaryTermDelete).toBeInTheDocument();
 
+      await act(async () => {
         fireEvent.click(handleGlossaryTermDelete);
       });
+    });
 
     it('show error if deleteGlossary API fails', async () => {
       (deleteGlossary as jest.Mock).mockImplementationOnce(() =>
@@ -275,8 +281,10 @@ describe('Test GlossaryComponent page', () => {
 
       expect(handleGlossaryDelete).toBeInTheDocument();
 
+      await act(async () => {
         fireEvent.click(handleGlossaryDelete);
       });
+    });
 
     it('show error if patchGlossaryTerm API resolves without data', async () => {
       (patchGlossaryTerm as jest.Mock).mockImplementation(() =>
@@ -289,7 +297,9 @@ describe('Test GlossaryComponent page', () => {
 
       expect(handleGlossaryTermUpdate).toBeInTheDocument();
 
+      await act(async () => {
         fireEvent.click(handleGlossaryTermUpdate);
       });
     });
   });
+});