@openmrs/esm-stock-management-app 3.0.1-pre.853 → 3.0.1-pre.855

package/src/stock-operations/stock-operations-forms/input-components/user-selector.test.tsx

@@ -1,51 +1,42 @@
+import { fireEvent, render, screen, waitFor } from '@testing-library/react';
 import React from 'react';
-import userEvent from '@testing-library/user-event';
-import { render, screen } from '@testing-library/react';
-import { useFormContext, type Control, type FieldValues, type UseFormReturn } from 'react-hook-form';
-import { otherUser } from '../../../core/utils/utils';
+import { useFormContext } from 'react-hook-form';
 import { useUser } from '../../../stock-lookups/stock-lookups.resource';
 import useSearchUser from '../hooks/useSearchUser';
 import UsersSelector from './users-selector.component';
+import { otherUser } from '../../../core/utils/utils';
+import userEvent from '@testing-library/user-event';
 
-jest.mock('../hooks/useSearchUser', () => ({
-  __esModule: true,
-  default: jest.fn(),
-}));
-
-jest.mock('../../../stock-lookups/stock-lookups.resource', () => ({
-  useUser: jest.fn(),
-}));
-
+jest.mock('../hooks/useSearchUser');
+jest.mock('../../../stock-lookups/stock-lookups.resource');
 jest.mock('react-hook-form', () => ({
   useFormContext: jest.fn(),
   Controller: ({ render }) => render({ field: {}, fieldState: {} }),
 }));
+jest.mock('react-i18next', () => ({
+  useTranslation: () => ({ t: (key: string) => key }),
+}));
 
-const mockUseSearchUser = jest.mocked(useSearchUser);
-const mockUseUser = jest.mocked(useUser);
-const mockUseFormContext = jest.mocked(useFormContext);
+const mockUseSearchUser = useSearchUser as jest.Mock;
+const mockUseUser = useUser as jest.Mock;
+const mockUseFormContext = useFormContext as jest.Mock;
 
-describe('User Selector', () => {
+describe('UsersSelector', () => {
   beforeEach(() => {
+    jest.clearAllMocks();
     mockUseFormContext.mockReturnValue({
-      control: {} as Control<FieldValues>,
+      control: {},
       watch: jest.fn().mockImplementation((field) => {
         if (field === 'responsiblePersonUuid') return 'responsibleperson.uuid';
         if (field === 'responsiblePersonOther') return 'responsiblepersonother.uuid';
         return '';
       }),
       resetField: jest.fn(),
-    } as Partial<UseFormReturn<FieldValues>> as UseFormReturn<FieldValues>);
+    });
   });
 
   it('renders loading state', async () => {
-    mockUseSearchUser.mockReturnValue({
-      isLoading: true,
-      userList: [],
-      setSearchString: jest.fn(),
-      setLimit: jest.fn(),
-      setRepresentation: jest.fn(),
-    });
+    mockUseSearchUser.mockReturnValue({ isLoading: true, userList: [], setSearchString: jest.fn() });
     mockUseUser.mockReturnValue({ isLoading: true, data: null, error: null });
 
     render(<UsersSelector />);
@@ -55,108 +46,64 @@ describe('User Selector', () => {
 
   it('renders error state', () => {
     const errorMessage = 'Error message';
-    mockUseSearchUser.mockReturnValue({
-      isLoading: false,
-      userList: [],
-      setSearchString: jest.fn(),
-      setLimit: jest.fn(),
-      setRepresentation: jest.fn(),
-    });
+    mockUseSearchUser.mockReturnValue({ isLoading: false, userList: [], setSearchString: jest.fn() });
     mockUseUser.mockReturnValue({ isLoading: false, data: null, error: new Error(errorMessage) });
-
     render(<UsersSelector />);
-
-    expect(screen.getByText(/error loading responsible person/i)).toBeInTheDocument();
+    expect(screen.getByText('responsiblePersonError')).toBeInTheDocument();
     expect(screen.getByText(errorMessage)).toBeInTheDocument();
   });
 
   it('renders ComboBox with user list', async () => {
-    const user = userEvent.setup();
-
     mockUseSearchUser.mockReturnValue({
       isLoading: false,
       userList: [
-        {
-          uuid: '1',
-          person: { display: 'User 1', uuid: '1' },
-          display: 'User 1',
-          givenName: 'User',
-          familyName: '1',
-          firstName: 'User',
-          lastName: '1',
-          privileges: [],
-        },
-        {
-          uuid: '2',
-          person: { display: 'User 2', uuid: '2' },
-          display: 'User 2',
-          givenName: 'User',
-          familyName: '2',
-          firstName: 'User',
-          lastName: '2',
-          privileges: [],
-        },
+        { uuid: '1', person: { display: 'User 1' } },
+        { uuid: '2', person: { display: 'User 2' } },
       ],
       setSearchString: jest.fn(),
-      setLimit: jest.fn(),
-      setRepresentation: jest.fn(),
     });
 
     mockUseUser.mockReturnValue({ isLoading: false, data: null, error: null });
-
     render(<UsersSelector />);
-
-    expect(screen.getByText(/responsible person/i)).toBeInTheDocument();
+    expect(screen.getByText('responsiblePerson')).toBeInTheDocument();
     const combobox = screen.getByRole('combobox');
-    await user.click(combobox);
+    fireEvent.click(combobox);
     expect(screen.getByText('User 1')).toBeInTheDocument();
     expect(screen.getByText('User 2')).toBeInTheDocument();
   });
 
   it('renders TextInput for other user', async () => {
-    mockUseSearchUser.mockReturnValue({
-      isLoading: false,
-      userList: [],
-      setSearchString: jest.fn(),
-      setLimit: jest.fn(),
-      setRepresentation: jest.fn(),
-    });
+    mockUseSearchUser.mockReturnValue({ isLoading: false, userList: [], setSearchString: jest.fn() });
     mockUseFormContext.mockReturnValue({
-      control: {} as Control<FieldValues>,
+      control: {},
       watch: jest.fn().mockImplementation((field) => {
         if (field === 'responsiblePersonUuid') return otherUser.uuid;
         if (field === 'responsiblePersonOther') return '';
         return '';
       }),
       resetField: jest.fn(),
-    } as Partial<UseFormReturn<FieldValues>> as UseFormReturn<FieldValues>);
+    });
     mockUseUser.mockReturnValue({ isLoading: false, data: null, error: null });
 
     render(<UsersSelector />);
-
     expect(screen.getByRole('textbox')).toBeInTheDocument();
-    expect(screen.getByPlaceholderText(/please specify/i)).toBeInTheDocument();
+    expect(screen.getByPlaceholderText('pleaseSpecify')).toBeInTheDocument();
   });
 
   it('calls setSearchString on input change after delay simulating debounce timout', async () => {
-    const user = userEvent.setup();
-
     const setSearchString = jest.fn();
     mockUseSearchUser.mockReturnValue({
       isLoading: false,
       userList: [],
       setSearchString,
-      setLimit: jest.fn(),
-      setRepresentation: jest.fn(),
     });
 
     mockUseUser.mockReturnValue({ isLoading: false, data: null, error: null });
 
     render(<UsersSelector />);
-
     const combobox = screen.getByRole('combobox');
-    await user.click(combobox);
-    await user.type(combobox, 'test');
+    await userEvent.click(combobox);
+    await userEvent.type(combobox, 'test');
     await new Promise((resolve) => setTimeout(resolve, 2000)); // Simulate debounce
     expect(setSearchString).toHaveBeenCalledWith('test');
   });