@openmrs/esm-form-engine-lib 2.1.0-pre.1462 → 2.1.0-pre.1466

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@openmrs/esm-form-engine-lib",
-  "version": "2.1.0-pre.1462",
+  "version": "2.1.0-pre.1466",
   "description": "React Form Engine for O3",
   "browser": "dist/openmrs-esm-form-engine-lib.js",
   "main": "src/index.ts",

package/src/api/index.ts

@@ -106,9 +106,13 @@ export async function fetchOpenMRSForm(nameOrUUID: string): Promise<OpenmrsForm
     return openmrsFormResponse;
   }
 
-  return openmrsFormResponse.results?.length
-    ? openmrsFormResponse.results.find((form) => form.retired === false)
-    : new Error(`Form with ${nameOrUUID} was not found`);
+  if (openmrsFormResponse.results?.length) {
+    const form = openmrsFormResponse.results.find((form) => form.retired === false);
+    if (form) {
+      return form;
+    }
+  }
+  throw new Error(`Form with ID "${nameOrUUID}" was not found`);
 }
 
 /**
@@ -121,7 +125,7 @@ export async function fetchClobData(form: OpenmrsForm): Promise<any | null> {
     return null;
   }
 
-  const jsonSchemaResource = form.resources.find(({ name }) => name === 'JSON schema');
+  const jsonSchemaResource = form.resources?.find(({ name }) => name === 'JSON schema');
   if (!jsonSchemaResource) {
     return null;
   }

package/src/components/inputs/multi-select/multi-select.test.tsx

@@ -1,6 +1,6 @@
 import React from 'react';
 import { act, render, screen } from '@testing-library/react';
-import { userEvent } from '@testing-library/user-event';
+import userEvent from '@testing-library/user-event'; // Correct import for userEvent
 import { type FetchResponse, openmrsFetch, usePatient, useSession } from '@openmrs/esm-framework';
 import { type FormSchema } from '../../../types';
 import { mockPatient } from '__mocks__/patient.mock';
@@ -31,7 +31,7 @@ jest.mock('../../../api', () => {
 });
 
 const renderForm = async () => {
-  await act(() =>
+  await act(async () => {
     render(
       <FormEngine
         formJson={multiSelectFormSchema as unknown as FormSchema}
@@ -40,11 +40,11 @@ const renderForm = async () => {
         formSessionIntent={undefined}
         visit={visit}
       />,
-    ),
-  );
+    );
+  });
 };
 
-describe.skip('MultiSelect Component', () => {
+describe('MultiSelect Component', () => {
   beforeEach(() => {
     mockOpenmrsFetch.mockResolvedValue({
       data: { results: [{ ...multiSelectFormSchema }] },
@@ -69,22 +69,20 @@ describe.skip('MultiSelect Component', () => {
   it('should disable checkbox option if the field value depends on evaluates the expression to true', async () => {
     const user = userEvent.setup();
     await renderForm();
-
     await user.click(screen.getByRole('combobox', { name: /Patient covered by NHIF/i }));
     await user.click(screen.getByRole('option', { name: /no/i }));
-
     await user.click(screen.getByText('Was this visit scheduled?'));
-    expect(screen.getByRole('option', { name: /Unscheduled visit early/i })).toHaveAttribute('disabled');
+    const unscheduledVisitOption = screen.getByRole('option', { name: /Unscheduled visit early/i });
+    expect(unscheduledVisitOption).toHaveAttribute('disabled');
   });
 
   it('should enable checkbox option if the field value depends on evaluates the expression to false', async () => {
     const user = userEvent.setup();
     await renderForm();
-
     await user.click(screen.getByRole('combobox', { name: /patient covered by nhif/i }));
     await user.click(screen.getByRole('option', { name: /yes/i }));
-
     await user.click(screen.getByText('Was this visit scheduled?'));
-    expect(screen.getByRole('option', { name: /Unscheduled visit early/i })).not.toHaveAttribute('disabled');
+    const unscheduledVisitOption = screen.getByRole('option', { name: /Unscheduled visit early/i });
+    expect(unscheduledVisitOption).not.toBeDisabled();
   });
 });

package/src/hooks/useFormJson.test.tsx

@@ -36,6 +36,7 @@ const COMPONENT_ART_SCHEMA_VALUE_REF = '74d06044-850f-11ee-b9d1-0242ac120003';
 const COMPONENT_PRECLINIC_REVIEW = 'component_preclinic-review';
 const COMPONENT_PRECLINIC_REVIEW_UUID = '2f063f32-7f8a-11ee-b962-0242ac120004';
 const COMPONENT_PRECLINIC_REVIEW_SCHEMA_VALUE_REF = '74d06044-850f-11ee-b9d1-0242ac120004';
+const NON_EXISTENT_FORM_NAME = 'non-existent-form';
 
 // Base setup
 const mockOpenmrsFetch = openmrsFetch as jest.Mock;
@@ -74,6 +75,7 @@ when(mockOpenmrsFetch)
 when(mockOpenmrsFetch)
   .calledWith(buildPath(COMPONENT_ART))
   .mockResolvedValue({ data: { results: [artComponentSkeleton] } });
+
 when(mockOpenmrsFetch).calledWith(buildPath(COMPONENT_ART_UUID)).mockResolvedValue({ data: artComponentSkeleton });
 when(mockOpenmrsFetch)
   .calledWith(buildPath(COMPONENT_ART_SCHEMA_VALUE_REF))
@@ -82,6 +84,7 @@ when(mockOpenmrsFetch)
 when(mockOpenmrsFetch)
   .calledWith(buildPath(COMPONENT_PRECLINIC_REVIEW))
   .mockResolvedValue({ data: { results: [preclinicReviewComponentSkeleton] } });
+
 when(mockOpenmrsFetch)
   .calledWith(buildPath(COMPONENT_PRECLINIC_REVIEW_UUID))
   .mockResolvedValue({ data: preclinicReviewComponentSkeleton });
@@ -89,6 +92,10 @@ when(mockOpenmrsFetch)
   .calledWith(buildPath(COMPONENT_PRECLINIC_REVIEW_SCHEMA_VALUE_REF))
   .mockResolvedValue({ data: preclinicReviewComponentBody });
 
+when(mockOpenmrsFetch)
+  .calledWith(buildPath(NON_EXISTENT_FORM_NAME))
+  .mockResolvedValue({ data: { results: [] } });
+
 describe('useFormJson', () => {
   it('should fetch basic form by name', async () => {
     let hook = null;
@@ -97,7 +104,7 @@ describe('useFormJson', () => {
     });
 
     expect(hook.result.current.isLoading).toBe(false);
-    expect(hook.result.current.error).toBe(undefined);
+    expect(hook.result.current.formError).toBe(undefined);
     expect(hook.result.current.formJson.name).toBe(MINI_FORM_NAME);
   });
 
@@ -108,7 +115,7 @@ describe('useFormJson', () => {
     });
 
     expect(hook.result.current.isLoading).toBe(false);
-    expect(hook.result.current.error).toBe(undefined);
+    expect(hook.result.current.formError).toBe(undefined);
     expect(hook.result.current.formJson.name).toBe(MINI_FORM_NAME);
   });
 
@@ -119,7 +126,7 @@ describe('useFormJson', () => {
     });
 
     expect(hook.result.current.isLoading).toBe(false);
-    expect(hook.result.current.error).toBe(undefined);
+    expect(hook.result.current.formError).toBe(undefined);
     expect(hook.result.current.formJson.name).toBe(PARENT_FORM_NAME);
 
     // verify subforms
@@ -133,7 +140,7 @@ describe('useFormJson', () => {
     });
 
     expect(hook.result.current.isLoading).toBe(false);
-    expect(hook.result.current.error).toBe(undefined);
+    expect(hook.result.current.formError).toBe(undefined);
     expect(hook.result.current.formJson.name).toBe(PARENT_FORM_NAME);
 
     // verify subforms
@@ -146,12 +153,26 @@ describe('useFormJson', () => {
       hook = renderHook(() => useFormJson(null, formComponentBody, null, null));
     });
     expect(hook.result.current.isLoading).toBe(false);
-    expect(hook.result.current.error).toBe(undefined);
+    expect(hook.result.current.formError).toBe(undefined);
     expect(hook.result.current.formJson.name).toBe(COMPONENT_FORM_NAME);
 
     // verify form components have been loaded
     verifyFormComponents(hook.result.current.formJson);
   });
+
+  it('should return an error when the form is not found', async () => {
+    // setup and execute
+    let hook = null;
+    await act(async () => {
+      hook = renderHook(() => useFormJson(NON_EXISTENT_FORM_NAME, null, null, null));
+    });
+    // verify
+    expect(hook.result.current.isLoading).toBe(false);
+    expect(hook.result.current.formError.message).toBe(
+      'Error loading form JSON: Form with ID "non-existent-form" was not found',
+    );
+    expect(hook.result.current.formJson).toBe(null);
+  });
 });
 
 function buildPath(path: string) {

package/src/hooks/useFormJson.tsx

@@ -29,7 +29,7 @@ export function useFormJson(formUuid: string, rawFormJson: any, encounterUuid: s
 
   return {
     formJson,
-    isLoading: !formJson,
+    isLoading: !formJson && !error,
     formError: error,
   };
 }