@openmrs/esm-appointments-app 9.2.1-pre.7279 → 9.2.1-pre.7288

package/src/form/appointments-form.test.tsx

@@ -2,13 +2,14 @@ import React from 'react';
 import userEvent from '@testing-library/user-event';
 import { fireEvent, screen } from '@testing-library/react';
 import {
-  type FetchResponse,
   getDefaultsFromConfigSchema,
   openmrsFetch,
   showSnackbar,
   useConfig,
   useLocations,
   useSession,
+  type FetchResponse,
+  type Workspace2DefinitionProps,
 } from '@openmrs/esm-framework';
 import { configSchema, type ConfigObject } from '../config-schema';
 import { mockUseAppointmentServiceData, mockSession, mockLocations, mockProviders } from '__mocks__';
@@ -16,15 +17,33 @@ import { mockPatient, renderWithSwr, waitForLoadingToFinish } from 'tools';
 import { saveAppointment, checkAppointmentConflict } from './appointments-form.resource';
 import { useProviders } from '../hooks/useProviders';
 import type { AppointmentKind, AppointmentStatus } from '../types';
-import AppointmentForm from './appointments-form.workspace';
-
-const defaultProps = {
-  context: 'creating',
-  closeWorkspace: jest.fn(),
-  patientUuid: mockPatient.id,
-  promptBeforeClosing: jest.fn(),
-  closeWorkspaceWithSavedChanges: jest.fn(),
-  setTitle: jest.fn(),
+import AppointmentForm, { type AppointmentsFormProps } from './appointments-form.workspace';
+
+const renderAppointmentsForm = (props: Partial<Workspace2DefinitionProps<AppointmentsFormProps>> = {}) => {
+  const closeWorkspace = props.closeWorkspace || jest.fn();
+  const workspaceProps: AppointmentsFormProps = {
+    context: props.workspaceProps?.context || 'creating',
+    patientUuid: props.workspaceProps?.patientUuid || mockPatient.id,
+    appointment: props.workspaceProps?.appointment,
+    recurringPattern: props.workspaceProps?.recurringPattern,
+
+    ...props.workspaceProps,
+  };
+
+  const defaultProps: Workspace2DefinitionProps<AppointmentsFormProps> = {
+    closeWorkspace,
+    workspaceProps,
+    groupProps: props.groupProps || { collapsed: false, name: 'appointmentsFormWorkspaceGroup', overlay: false },
+    windowProps: props.windowProps || { group: 'appointmentsFormWorkspaceGroup', name: 'appointments-form-window' },
+    workspaceName: props.workspaceName || 'appointments-form-workspace',
+    launchChildWorkspace: jest.fn(),
+    windowName: 'appointments-form-window',
+
+    isRootWorkspace: false,
+    ...props,
+  };
+
+  return renderWithSwr(<AppointmentForm {...defaultProps} />);
 };
 
 const mockOpenmrsFetch = jest.mocked(openmrsFetch);
@@ -57,7 +76,7 @@ jest.mock('../workload/workload.resource', () => ({
 }));
 
 describe('AppointmentForm', () => {
-  const dateTimeRegex = /^\d{4}-\d{2}-\d{2}T\d{2}:\d{2}:\d{2}(?:\.\d{3}Z|\+00:00)$/;
+  const dateTimeRegex = /^\d{4}-\d{2}-\d{2}T\d{2}:\d{2}:\d{2}(?:\.\d{3})?(?:Z|[+-]\d{2}:\d{2})$/;
 
   beforeEach(() => {
     mockUseConfig.mockReturnValue({
@@ -81,7 +100,7 @@ describe('AppointmentForm', () => {
   it('renders the appointments form', async () => {
     mockOpenmrsFetch.mockResolvedValue(mockUseAppointmentServiceData as unknown as FetchResponse);
 
-    renderWithSwr(<AppointmentForm {...defaultProps} />);
+    renderAppointmentsForm();
 
     await waitForLoadingToFinish();
 
@@ -104,17 +123,18 @@ describe('AppointmentForm', () => {
 
   it('closes the workspace when the cancel button is clicked', async () => {
     const user = userEvent.setup();
+    const mockCloseWorkspace = jest.fn();
 
     mockOpenmrsFetch.mockResolvedValueOnce(mockUseAppointmentServiceData as unknown as FetchResponse);
 
-    renderWithSwr(<AppointmentForm {...defaultProps} />);
+    renderAppointmentsForm({ closeWorkspace: mockCloseWorkspace });
 
     await waitForLoadingToFinish();
 
     const cancelButton = screen.getByRole('button', { name: /Discard/i });
     await user.click(cancelButton);
 
-    expect(defaultProps.closeWorkspace).toHaveBeenCalledTimes(1);
+    expect(mockCloseWorkspace).toHaveBeenCalledTimes(1);
   });
 
   it('renders a success snackbar upon successfully scheduling an appointment', async () => {
@@ -124,7 +144,7 @@ describe('AppointmentForm', () => {
     mockCheckAppointmentConflict.mockResolvedValue({ status: 204, data: {} } as FetchResponse);
     mockSaveAppointment.mockResolvedValue({ status: 200, statusText: 'Ok' } as FetchResponse);
 
-    renderWithSwr(<AppointmentForm {...defaultProps} />);
+    renderAppointmentsForm();
 
     await waitForLoadingToFinish();
 
@@ -202,7 +222,7 @@ describe('AppointmentForm', () => {
     mockCheckAppointmentConflict.mockResolvedValue({ status: 204, data: {} } as FetchResponse);
     mockSaveAppointment.mockRejectedValue(error);
 
-    renderWithSwr(<AppointmentForm {...defaultProps} />);
+    renderAppointmentsForm();
 
     await waitForLoadingToFinish();
 
@@ -274,7 +294,7 @@ describe('AppointmentForm', () => {
     });
     mockOpenmrsFetch.mockResolvedValue(mockUseAppointmentServiceData as unknown as FetchResponse);
 
-    renderWithSwr(<AppointmentForm {...defaultProps} />);
+    renderAppointmentsForm();
 
     await waitForLoadingToFinish();
 
@@ -292,7 +312,7 @@ describe('AppointmentForm', () => {
 
     mockOpenmrsFetch.mockResolvedValue(mockUseAppointmentServiceData as unknown as FetchResponse);
 
-    renderWithSwr(<AppointmentForm {...defaultProps} />);
+    renderAppointmentsForm();
 
     await waitForLoadingToFinish();
 
@@ -313,7 +333,7 @@ describe('AppointmentForm', () => {
     const user = userEvent.setup();
     mockOpenmrsFetch.mockResolvedValue(mockUseAppointmentServiceData as unknown as FetchResponse);
 
-    renderWithSwr(<AppointmentForm {...defaultProps} />);
+    renderAppointmentsForm();
 
     await waitForLoadingToFinish();
 
@@ -354,7 +374,7 @@ describe('AppointmentForm', () => {
     mockCheckAppointmentConflict.mockResolvedValue({ status: 204, data: {} } as FetchResponse);
     mockSaveAppointment.mockResolvedValue({ status: 200, statusText: 'Ok' } as FetchResponse);
 
-    renderWithSwr(<AppointmentForm {...defaultProps} />);
+    renderAppointmentsForm();
 
     await waitForLoadingToFinish();
 
@@ -395,7 +415,7 @@ describe('AppointmentForm', () => {
 
     mockOpenmrsFetch.mockResolvedValue({ data: mockUseAppointmentServiceData } as unknown as FetchResponse);
 
-    renderWithSwr(<AppointmentForm {...defaultProps} />);
+    renderAppointmentsForm();
 
     await waitForLoadingToFinish();
 
@@ -425,7 +445,7 @@ describe('AppointmentForm', () => {
 
     mockOpenmrsFetch.mockResolvedValue({ data: mockUseAppointmentServiceData } as unknown as FetchResponse);
 
-    renderWithSwr(<AppointmentForm {...defaultProps} />);
+    renderAppointmentsForm();
 
     await waitForLoadingToFinish();
 
@@ -457,7 +477,7 @@ describe('AppointmentForm', () => {
     mockCheckAppointmentConflict.mockResolvedValue({ status: 204, data: {} } as FetchResponse);
     mockSaveAppointment.mockResolvedValue({ status: 200, statusText: 'Ok' } as FetchResponse);
 
-    renderWithSwr(<AppointmentForm {...defaultProps} />);
+    renderAppointmentsForm();
 
     await waitForLoadingToFinish();
 
@@ -512,7 +532,7 @@ describe('AppointmentForm', () => {
 
     mockOpenmrsFetch.mockResolvedValue({ data: mockUseAppointmentServiceData } as unknown as FetchResponse);
 
-    renderWithSwr(<AppointmentForm {...defaultProps} />);
+    renderAppointmentsForm();
 
     await waitForLoadingToFinish();
 
@@ -548,7 +568,7 @@ describe('AppointmentForm', () => {
       mockCheckAppointmentConflict.mockResolvedValue({ status: 204, data: {} } as FetchResponse);
       mockSaveAppointment.mockResolvedValue({ status: 200, statusText: 'Ok' } as FetchResponse);
 
-      renderWithSwr(<AppointmentForm {...defaultProps} />);
+      renderAppointmentsForm();
 
       await waitForLoadingToFinish();
 
@@ -616,7 +636,7 @@ describe('AppointmentForm', () => {
 
       mockOpenmrsFetch.mockResolvedValue({ data: mockUseAppointmentServiceData } as unknown as FetchResponse);
 
-      renderWithSwr(<AppointmentForm {...defaultProps} />);
+      renderAppointmentsForm();
 
       await waitForLoadingToFinish();
 
@@ -649,7 +669,7 @@ describe('AppointmentForm', () => {
         data: { SERVICE_UNAVAILABLE: true },
       } as FetchResponse);
 
-      renderWithSwr(<AppointmentForm {...defaultProps} />);
+      renderAppointmentsForm();
 
       await waitForLoadingToFinish();
 
@@ -703,7 +723,7 @@ describe('AppointmentForm', () => {
         data: { PATIENT_DOUBLE_BOOKING: true },
       } as FetchResponse);
 
-      renderWithSwr(<AppointmentForm {...defaultProps} />);
+      renderAppointmentsForm();
 
       await waitForLoadingToFinish();
 
@@ -783,13 +803,13 @@ describe('AppointmentForm', () => {
 
       mockOpenmrsFetch.mockResolvedValue({ data: mockUseAppointmentServiceData } as unknown as FetchResponse);
 
-      renderWithSwr(<AppointmentForm {...defaultProps} appointment={existingAppointment} context="editing" />);
+      renderAppointmentsForm({ workspaceProps: { appointment: existingAppointment, context: 'editing' } });
 
       await waitForLoadingToFinish();
 
       // Check that form fields are pre-populated
       expect(screen.getByDisplayValue('Existing appointment note')).toBeInTheDocument();
-      expect(screen.getByDisplayValue('Outpatient')).toBeInTheDocument();
+      expect(screen.getByText('Outpatient')).toBeInTheDocument();
       expect(screen.getByRole('combobox', { name: /select the type of appointment/i })).toHaveValue('Scheduled');
     });
 
@@ -829,7 +849,7 @@ describe('AppointmentForm', () => {
       mockCheckAppointmentConflict.mockResolvedValue({ status: 204, data: {} } as FetchResponse);
       mockSaveAppointment.mockResolvedValue({ status: 200, statusText: 'Ok' } as FetchResponse);
 
-      renderWithSwr(<AppointmentForm {...defaultProps} appointment={existingAppointment} context="editing" />);
+      renderAppointmentsForm({ workspaceProps: { appointment: existingAppointment, context: 'editing' } });
 
       await waitForLoadingToFinish();
 
@@ -866,7 +886,7 @@ describe('AppointmentForm', () => {
 
       mockOpenmrsFetch.mockResolvedValue({ data: mockUseAppointmentServiceData } as unknown as FetchResponse);
 
-      renderWithSwr(<AppointmentForm {...defaultProps} />);
+      renderAppointmentsForm();
 
       await waitForLoadingToFinish();
 
@@ -882,10 +902,11 @@ describe('AppointmentForm', () => {
 
     it('should warn before closing with unsaved changes', async () => {
       const user = userEvent.setup();
+      const mockCloseWorkspace = jest.fn();
 
       mockOpenmrsFetch.mockResolvedValue({ data: mockUseAppointmentServiceData } as unknown as FetchResponse);
 
-      renderWithSwr(<AppointmentForm {...defaultProps} />);
+      renderAppointmentsForm({ closeWorkspace: mockCloseWorkspace });
 
       await waitForLoadingToFinish();
 
@@ -898,8 +919,8 @@ describe('AppointmentForm', () => {
       // Try to cancel
       await user.click(cancelButton);
 
-      // Should call promptBeforeClosing if there are unsaved changes
-      expect(defaultProps.promptBeforeClosing).toHaveBeenCalled();
+      // Should call closeWorkspace with discardUnsavedChanges
+      expect(mockCloseWorkspace).toHaveBeenCalledWith({ discardUnsavedChanges: false });
     });
   });
 });