@kenyaemr/esm-patient-registration-app 8.1.1-pre.129 → 8.1.2-pre.152

package/src/patient-registration/patient-registration.test.tsx

@@ -1,8 +1,8 @@
 import React from 'react';
 import dayjs from 'dayjs';
 import userEvent from '@testing-library/user-event';
-import { BrowserRouter as Router, useParams } from 'react-router-dom';
-import { render, screen, within } from '@testing-library/react';
+import { BrowserRouter as Router, useParams, useLocation } from 'react-router-dom';
+import { act, render, screen, within } from '@testing-library/react';
 import {
   type FetchResponse,
   getDefaultsFromConfigSchema,
@@ -10,22 +10,32 @@ import {
   showSnackbar,
   useConfig,
   usePatient,
+  openmrsFetch,
 } from '@openmrs/esm-framework';
+import type { AddressTemplate, Encounter, FormValues } from './patient-registration.types';
 import { mockedAddressTemplate } from '__mocks__';
-import { mockPatient } from 'tools';
+import { mockPatient, mockOpenMRSIdentificationNumberIdType } from 'tools';
 import { saveEncounter, savePatient } from './patient-registration.resource';
 import { esmPatientRegistrationSchema, type RegistrationConfig } from '../config-schema';
-import type { AddressTemplate, Encounter } from './patient-registration.types';
 import { ResourcesContext } from '../offline.resources';
 import { FormManager } from './form-manager';
 import { PatientRegistration } from './patient-registration.component';
+import { useInitialFormValuesLocal } from './patient-registration-hooks';
+import { useMpiPatient } from './mpi/mpi-patient.resource';
 
 const mockSaveEncounter = jest.mocked(saveEncounter);
 const mockSavePatient = savePatient as jest.Mock;
 const mockShowSnackbar = jest.mocked(showSnackbar);
 const mockUseConfig = jest.mocked(useConfig<RegistrationConfig>);
 const mockUsePatient = jest.mocked(usePatient);
+const mockUseParams = useParams as jest.Mock;
+const mockUseInitialFormValues = jest.mocked(useInitialFormValuesLocal);
 const mockOpenmrsDatePicker = jest.mocked(OpenmrsDatePicker);
+const mockUseMpiPatient = useMpiPatient as jest.Mock;
+
+jest.mock('./mpi/mpi-patient.resource', () => ({
+  useMpiPatient: jest.fn(),
+}));
 
 jest.mock('./field/field.resource', () => ({
   useConcept: jest.fn().mockImplementation((uuid: string) => {
@@ -86,7 +96,7 @@ jest.mock('./field/field.resource', () => ({
 
 jest.mock('react-router-dom', () => ({
   ...(jest.requireActual('react-router-dom') as any),
-  useLocation: () => ({
+  useLocation: jest.fn().mockReturnValue({
     pathname: 'openmrs/spa/patient-registration',
   }),
   useHistory: () => [],
@@ -99,6 +109,14 @@ jest.mock('./patient-registration.resource', () => ({
   savePatient: jest.fn(),
 }));
 
+jest.mock('./patient-registration-hooks', () => ({
+  ...jest.requireActual('./patient-registration-hooks'),
+  useInitialFormValuesLocal: jest.fn().mockReturnValue([{}, jest.fn()]),
+  useMpiInitialFormValues: jest.fn().mockReturnValue([{}, jest.fn()]),
+  useInitialAddressFieldValues: jest.fn().mockReturnValue([{}, jest.fn()]),
+  usePatientUuidMap: jest.fn().mockReturnValue([{}, jest.fn()]),
+}));
+
 mockOpenmrsDatePicker.mockImplementation(({ id, labelText, value, onChange }) => {
   return (
     <>
@@ -126,10 +144,10 @@ const mockResourcesContextValue = {
   identifierTypes: [],
 };
 
-let mockOpenmrsConfig: RegistrationConfig = {
+const mockOpenmrsConfig: RegistrationConfig = {
   sections: ['demographics', 'contact'],
   sectionDefinitions: [
-    { id: 'demographics', name: 'Demographics', fields: ['name', 'gender', 'dob'] },
+    { id: 'demographics', name: 'Demographics', fields: ['name', 'gender', 'dob', 'id'] },
     { id: 'contact', name: 'Contact Info', fields: ['address'] },
     { id: 'relationships', name: 'Relationships', fields: ['relationship'] },
   ],
@@ -159,6 +177,13 @@ let mockOpenmrsConfig: RegistrationConfig = {
         value: 'male',
         label: 'Male',
       },
+      {
+        value: 'female',
+        label: 'Female',
+      },
+    ],
+    identifierMappings: [
+      { fhirIdentifierSystem: 'MPI OpenMRS ID', openmrsIdentifierTypeUuid: '8d793bee-c2cc-11de-8d13-0010c6dffd0f' },
     ],
     address: {
       useAddressHierarchy: {
@@ -174,17 +199,17 @@ let mockOpenmrsConfig: RegistrationConfig = {
   links: {
     submitButton: '#',
   },
-  defaultPatientIdentifierTypes: [],
+  defaultPatientIdentifierTypes: ['8d793bee-c2cc-11de-8d13-0010c6dffd0f'],
   registrationObs: {
     encounterTypeUuid: null,
     encounterProviderRoleUuid: 'asdf',
     registrationFormUuid: null,
   },
+  freeTextFieldConceptUuid: '',
 };
-
 const path = `/patient/:patientUuid/edit`;
-
 const configWithObs = JSON.parse(JSON.stringify(mockOpenmrsConfig));
+
 configWithObs.fieldDefinitions = [
   {
     id: 'weight',
@@ -217,7 +242,6 @@ configWithObs.fieldDefinitions = [
     customConceptAnswers: [],
   },
 ];
-
 configWithObs.sectionDefinitions?.push({
   id: 'custom',
   name: 'Custom',
@@ -238,16 +262,14 @@ const fillRequiredFields = async () => {
   await user.type(familyNameInput, 'Gaihre');
   await user.clear(dateOfBirthInput);
   await user.type(dateOfBirthInput, '02/08/1993');
-  user.click(genderInput);
+  await user.click(genderInput);
 };
 
-function Wrapper({ children }) {
-  return (
-    <ResourcesContext.Provider value={mockResourcesContextValue}>
-      <Router>{children}</Router>
-    </ResourcesContext.Provider>
-  );
-}
+const Wrapper = ({ children }) => (
+  <ResourcesContext.Provider value={mockResourcesContextValue}>
+    <Router>{children}</Router>
+  </ResourcesContext.Provider>
+);
 
 describe('Registering a new patient', () => {
   beforeEach(() => {
@@ -256,17 +278,130 @@ describe('Registering a new patient', () => {
       ...mockOpenmrsConfig,
     });
     mockSavePatient.mockReturnValue({ data: { uuid: 'new-pt-uuid' }, ok: true });
+
+    mockUseMpiPatient.mockReturnValue({
+      isLoading: false,
+      patient: { data: null },
+      error: undefined,
+    });
   });
 
-  it('renders without crashing', () => {
+  // TODO O3-3482: Fix this test case when OpenmrsDatePicker gets fixed on core
+  it.skip('saves the patient without extra info', async () => {
+    const user = userEvent.setup();
     render(<PatientRegistration isOffline={false} savePatientForm={jest.fn()} />, { wrapper: Wrapper });
+
+    await screen.findByRole('heading', { name: /create new patientnj/i });
+
+    await fillRequiredFields();
+    await user.click(await screen.findByText(/Register Patientsed/i));
+
+    expect(mockSavePatient).toHaveBeenCalledWith(
+      true,
+      {
+        addNameInLocalLanguage: false,
+        additionalFamilyName: '',
+        additionalGivenName: '',
+        additionalMiddleName: '',
+        address: { country: 'កម្ពុជា (Cambodia)' },
+        attributes: {},
+        birthdate: '1972-04-04',
+        birthdateEstimated: false,
+        deathCause: '',
+        nonCodedCauseOfDeath: '',
+        deathDate: undefined,
+        deathTime: undefined,
+        deathTimeFormat: 'AM',
+        familyName: 'Smith',
+        gender: 'male',
+        givenName: 'Eric',
+        identifiers: {
+          openMrsId: {
+            autoGeneration: false,
+            identifierName: 'OpenMRS ID',
+            identifierTypeUuid: '05a29f94-c0ed-11e2-94be-8c13b969e334',
+            identifierUuid: '1f0ad7a1-430f-4397-b571-59ea654a52db',
+            identifierValue: '100GEJ',
+            initialValue: '100GEJ',
+            preferred: true,
+            required: true,
+            selectedSource: null,
+          },
+          idCard: {
+            autoGeneration: false,
+            identifierName: 'ID Card',
+            identifierTypeUuid: 'b4143563-16cd-4439-b288-f83d61670fc8',
+            identifierUuid: '346d09b1-8509-43c6-9697-3b4d1ce06ad6',
+            identifierValue: '1234567890',
+            initialValue: '1234567890',
+            preferred: false,
+            required: false,
+            selectedSource: null,
+          },
+        },
+        isDead: false,
+        middleName: 'Johnson',
+        monthsEstimated: 0,
+        patientUuid: '8673ee4f-e2ab-4077-ba55-4980f408773ef',
+        relationships: [],
+        telephoneNumber: '',
+        unidentifiedPatient: undefined,
+        yearsEstimated: 0,
+      },
+      expect.anything(),
+      expect.anything(),
+      null,
+      undefined,
+      expect.anything(),
+      expect.anything(),
+      expect.anything(),
+      { patientSaved: false },
+      expect.anything(),
+    );
+
+    expect(mockSavePatient).toHaveBeenCalledWith(
+      expect.objectContaining({
+        identifiers: [], //TODO when the identifer story is finished: { identifier: '', identifierType: '05a29f94-c0ed-11e2-94be-8c13b969e334', location: '' },
+        person: {
+          addresses: expect.arrayContaining([expect.any(Object)]),
+          attributes: [],
+          birthdate: '1993-8-2',
+          birthdateEstimated: false,
+          gender: expect.stringMatching(/^M$/),
+          names: [{ givenName: 'Paul', middleName: '', familyName: 'Gaihre', preferred: true, uuid: undefined }],
+          dead: false,
+          uuid: expect.anything(),
+        },
+        uuid: expect.anything(),
+      }),
+      undefined,
+    );
   });
 
-  it('has the expected sections', async () => {
+  it('should render all the required fields and sections', async () => {
     render(<PatientRegistration isOffline={false} savePatientForm={jest.fn()} />, { wrapper: Wrapper });
 
-    expect(screen.getByRole('region', { name: /demographics section/i })).toBeInTheDocument();
-    expect(screen.getByRole('region', { name: /contact info section/i })).toBeInTheDocument();
+    await screen.findByRole('heading', { name: /create new patient/i });
+
+    const demographicSection = screen.getByRole('region', { name: /demographics section/i });
+    const contactSection = screen.getByRole('region', { name: /contact info section/i });
+
+    expect(demographicSection).toBeInTheDocument();
+    expect(contactSection).toBeInTheDocument();
+    expect(screen.getByText(/jump to/i)).toBeInTheDocument();
+    expect(within(demographicSection).getByLabelText(/first name/i)).toBeInTheDocument();
+    expect(within(demographicSection).getByLabelText(/middle name \(optional\)/i)).toBeInTheDocument();
+    expect(within(demographicSection).getByLabelText(/family name/i)).toBeInTheDocument();
+    expect(within(demographicSection).getByLabelText(/date of birth/i)).toBeInTheDocument();
+    expect(within(demographicSection).getByRole('radio', { name: /^male$/i })).toBeInTheDocument();
+    expect(within(demographicSection).getByRole('radio', { name: /^female$/i })).toBeInTheDocument();
+    expect(within(demographicSection).getByText(/date of birth known\?/i)).toBeInTheDocument();
+    expect(within(demographicSection).getByLabelText(/date of birth/i)).toBeInTheDocument();
+
+    expect(within(contactSection).getByRole('heading', { name: /address/i })).toBeInTheDocument();
+
+    expect(screen.getByRole('button', { name: /register patient/i })).toBeInTheDocument();
+    expect(screen.getByRole('button', { name: /cancel/i })).toBeInTheDocument();
   });
 
   // TODO O3-3482: Fix this test case when OpenmrsDatePicker gets fixed on core
@@ -300,14 +435,12 @@ describe('Registering a new patient', () => {
 
   it('should not save the patient if validation fails', async () => {
     const user = userEvent.setup();
-
     const mockSavePatientForm = jest.fn();
+
     render(<PatientRegistration isOffline={false} savePatientForm={mockSavePatientForm} />, { wrapper: Wrapper });
 
-    const givenNameInput = (await screen.findByLabelText('First Name')) as HTMLInputElement;
-
-    await user.type(givenNameInput, '5');
-    await user.click(screen.getByText(/Register Patient/i));
+    await screen.findByRole('heading', { name: /create new patient/i });
+    await user.click(screen.getByRole('button', { name: /register patient/i }));
 
     expect(mockSavePatientForm).not.toHaveBeenCalled();
   });
@@ -384,28 +517,97 @@ describe('Registering a new patient', () => {
   });
 });
 
-describe('Updating an existing patient record', () => {
+describe('Import an MPI patient record', () => {
   beforeEach(() => {
-    mockUseConfig.mockReturnValue(mockOpenmrsConfig);
+    mockUseConfig.mockReturnValue({
+      ...getDefaultsFromConfigSchema(esmPatientRegistrationSchema),
+      ...mockOpenmrsConfig,
+    });
+
+    mockUsePatient.mockImplementation(() => {
+      return {
+        error: null,
+        isLoading: false,
+        patient: null,
+        patientUuid: null,
+      };
+    });
     mockSavePatient.mockReturnValue({ data: { uuid: 'new-pt-uuid' }, ok: true });
+    mockUseParams.mockReturnValue({ patientUuid: mockPatient.id });
+    (useParams as jest.Mock).mockReturnValue({ patientUuid: undefined }),
+      (useLocation as jest.Mock).mockReturnValue({
+        pathname: 'openmrs/spa/patient-registration',
+        state: undefined,
+        key: '',
+        search: '?sourceRecord=55',
+        hash: '',
+      });
   });
 
-  it('edits patient demographics', async () => {
+  it('fills patient demographics from MPI patient', async () => {
     const user = userEvent.setup();
-    mockSavePatient.mockResolvedValue({} as FetchResponse);
-
-    const mockUseParams = useParams as jest.Mock;
-
-    mockUseParams.mockReturnValue({ patientUuid: mockPatient.id });
-
-    mockUsePatient.mockReturnValue({
+    const mockSavePatientForm = jest.fn();
+    mockUseMpiPatient.mockReturnValue({
       isLoading: false,
-      patient: mockPatient,
-      patientUuid: mockPatient.id,
-      error: null,
+      patient: { data: mockPatient },
+      error: undefined,
     });
 
-    render(<PatientRegistration isOffline={false} savePatientForm={mockSavePatient} />, { wrapper: Wrapper });
+    mockUseInitialFormValues.mockReturnValue([
+      {
+        additionalFamilyName: '',
+        additionalGivenName: '',
+        additionalMiddleName: '',
+        addNameInLocalLanguage: false,
+        address: {},
+        birthdate: mockPatient.birthDate,
+        birthdateEstimated: false,
+        deathCause: '',
+        deathDate: undefined,
+        deathTime: undefined,
+        deathTimeFormat: 'AM',
+        familyName: mockPatient.name[0].family,
+        gender: mockPatient.gender,
+        givenName: mockPatient.name[0].given[0],
+        identifiers: {
+          openMrsId: {
+            autoGeneration: false,
+            identifierName: 'OpenMRS ID',
+            identifierTypeUuid: '05a29f94-c0ed-11e2-94be-8c13b969e334',
+            identifierUuid: '1f0ad7a1-430f-4397-b571-59ea654a52db',
+            identifierValue: '100GEJ',
+            initialValue: '100GEJ',
+            preferred: true,
+            required: true,
+            selectedSource: null,
+          },
+          idCard: {
+            autoGeneration: false,
+            identifierName: 'ID Card',
+            identifierTypeUuid: 'b4143563-16cd-4439-b288-f83d61670fc8',
+            identifierUuid: '346d09b1-8509-43c6-9697-3b4d1ce06ad6',
+            identifierValue: '1234567890',
+            initialValue: '1234567890',
+            preferred: false,
+            required: false,
+            selectedSource: null,
+          },
+        },
+        isDead: false,
+        middleName: '',
+        monthsEstimated: 0,
+        nonCodedCauseOfDeath: '',
+        patientUuid: mockPatient.id,
+        relationships: [],
+        telephoneNumber: '',
+        yearsEstimated: 0,
+      } as FormValues,
+      jest.fn(),
+    ]);
+    // eslint-disable-next-line testing-library/no-unnecessary-act
+    await act(async () => {
+      render(<PatientRegistration isOffline={false} savePatientForm={mockSavePatientForm} />, { wrapper: Wrapper });
+    });
 
     const givenNameInput: HTMLInputElement = screen.getByLabelText(/First Name/);
     const familyNameInput: HTMLInputElement = screen.getByLabelText(/Family Name/);
@@ -427,23 +629,19 @@ describe('Updating an existing patient record', () => {
     await user.type(givenNameInput, 'Eric');
     await user.type(middleNameInput, 'Johnson');
     await user.type(familyNameInput, 'Smith');
-    await user.click(screen.getByText(/Update patient/i));
+    await user.click(screen.getByText(/Register patient/i));
 
-    expect(mockSavePatient).toHaveBeenCalledWith(
-      false,
+    expect(mockSavePatientForm).toHaveBeenCalledWith(
+      true,
       {
-        '0': {
-          oldIdentificationNumber: '100732HE',
-        },
-        '1': {
-          openMrsId: '100GEJ',
-        },
-        addNameInLocalLanguage: undefined,
+        addNameInLocalLanguage: false,
         additionalFamilyName: '',
         additionalGivenName: '',
         additionalMiddleName: '',
-        address: {},
-        birthdate: new Date('1972-04-04T00:00:00.000Z'),
+        address: {
+          country: 'កម្ពុជា (Cambodia)',
+        },
+        birthdate: '1972-04-04',
         birthdateEstimated: false,
         deathCause: '',
         nonCodedCauseOfDeath: '',
@@ -451,9 +649,32 @@ describe('Updating an existing patient record', () => {
         deathTime: undefined,
         deathTimeFormat: 'AM',
         familyName: 'Smith',
-        gender: expect.stringMatching(/male/i),
+        gender: 'male',
         givenName: 'Eric',
-        identifiers: {},
+        identifiers: {
+          idCard: {
+            autoGeneration: false,
+            identifierName: 'ID Card',
+            identifierTypeUuid: 'b4143563-16cd-4439-b288-f83d61670fc8',
+            identifierUuid: '346d09b1-8509-43c6-9697-3b4d1ce06ad6',
+            identifierValue: '1234567890',
+            initialValue: '1234567890',
+            preferred: false,
+            required: false,
+            selectedSource: null,
+          },
+          openMrsId: {
+            autoGeneration: false,
+            identifierName: 'OpenMRS ID',
+            identifierTypeUuid: '05a29f94-c0ed-11e2-94be-8c13b969e334',
+            identifierUuid: '1f0ad7a1-430f-4397-b571-59ea654a52db',
+            identifierValue: '100GEJ',
+            initialValue: '100GEJ',
+            preferred: true,
+            required: true,
+            selectedSource: null,
+          },
+        },
         isDead: false,
         middleName: 'Johnson',
         monthsEstimated: 0,
@@ -475,3 +696,168 @@ describe('Updating an existing patient record', () => {
     );
   });
 });
+
+describe('Updating an existing patient record', () => {
+  beforeEach(() => {
+    mockUseConfig.mockReturnValue({
+      ...getDefaultsFromConfigSchema(esmPatientRegistrationSchema),
+      ...mockOpenmrsConfig,
+    });
+    mockUsePatient.mockImplementation(() => {
+      return {
+        error: null,
+        isLoading: false,
+        patient: mockPatient,
+        patientUuid: mockPatient.id,
+      };
+    });
+    mockSavePatient.mockReturnValue({ data: { uuid: 'new-pt-uuid' }, ok: true });
+    mockUseParams.mockReturnValue({ patientUuid: mockPatient.id });
+  });
+
+  it('edits patient demographics', async () => {
+    const user = userEvent.setup();
+    const mockSavePatientForm = jest.fn();
+
+    mockUseInitialFormValues.mockReturnValue([
+      {
+        additionalFamilyName: '',
+        additionalGivenName: '',
+        additionalMiddleName: '',
+        addNameInLocalLanguage: false,
+        address: {},
+        birthdate: mockPatient.birthDate,
+        birthdateEstimated: false,
+        deathCause: '',
+        deathDate: undefined,
+        deathTime: undefined,
+        deathTimeFormat: 'AM',
+        familyName: mockPatient.name[0].family,
+        gender: mockPatient.gender,
+        givenName: mockPatient.name[0].given[0],
+        identifiers: {
+          openMrsId: {
+            autoGeneration: false,
+            identifierName: 'OpenMRS ID',
+            identifierTypeUuid: '05a29f94-c0ed-11e2-94be-8c13b969e334',
+            identifierUuid: '1f0ad7a1-430f-4397-b571-59ea654a52db',
+            identifierValue: '100GEJ',
+            initialValue: '100GEJ',
+            preferred: true,
+            required: true,
+            selectedSource: null,
+          },
+          idCard: {
+            autoGeneration: false,
+            identifierName: 'ID Card',
+            identifierTypeUuid: 'b4143563-16cd-4439-b288-f83d61670fc8',
+            identifierUuid: '346d09b1-8509-43c6-9697-3b4d1ce06ad6',
+            identifierValue: '1234567890',
+            initialValue: '1234567890',
+            preferred: false,
+            required: false,
+            selectedSource: null,
+          },
+        },
+        isDead: false,
+        middleName: '',
+        monthsEstimated: 0,
+        nonCodedCauseOfDeath: '',
+        patientUuid: mockPatient.id,
+        relationships: [],
+        telephoneNumber: '',
+        yearsEstimated: 0,
+      } as FormValues,
+      jest.fn(),
+    ]);
+
+    render(<PatientRegistration isOffline={false} savePatientForm={mockSavePatientForm} />, { wrapper: Wrapper });
+
+    await screen.findByRole('heading', { name: /edit patient details/i });
+
+    expect(screen.queryByRole('button', { name: /register patient/i })).not.toBeInTheDocument();
+    expect(screen.getByRole('button', { name: /update patient/i })).toBeInTheDocument();
+    expect(screen.getByRole('button', { name: /cancel/i })).toBeInTheDocument();
+
+    expect(screen.getByLabelText(/first name/i)).toHaveValue(mockPatient.name[0].given[0]);
+    expect(screen.getByLabelText(/family name/i)).toHaveValue(mockPatient.name[0].family);
+    expect(screen.getByLabelText(/date of birth/i)).toHaveValue('04/04/1972');
+    expect(
+      screen.getByRole('radio', {
+        name: /^male$/i,
+      }),
+    ).toBeChecked();
+    expect(
+      screen.getByRole('radio', {
+        name: /^female$/i,
+      }),
+    ).not.toBeChecked();
+    expect(screen.getAllByRole('tab', { name: /yes/i })).toHaveLength(2);
+
+    await user.click(screen.getByRole('button', { name: /update patient/i }));
+
+    expect(mockSavePatientForm).toHaveBeenCalledWith(
+      false,
+      {
+        addNameInLocalLanguage: false,
+        additionalFamilyName: '',
+        additionalGivenName: '',
+        additionalMiddleName: '',
+        address: {
+          country: 'កម្ពុជា (Cambodia)',
+        },
+        birthdate: '1972-04-04',
+        birthdateEstimated: false,
+        deathCause: '',
+        nonCodedCauseOfDeath: '',
+        deathDate: undefined,
+        deathTime: undefined,
+        deathTimeFormat: 'AM',
+        familyName: 'Wilson',
+        gender: 'male',
+        givenName: 'John',
+        identifiers: {
+          idCard: {
+            autoGeneration: false,
+            identifierName: 'ID Card',
+            identifierTypeUuid: 'b4143563-16cd-4439-b288-f83d61670fc8',
+            identifierUuid: '346d09b1-8509-43c6-9697-3b4d1ce06ad6',
+            identifierValue: '1234567890',
+            initialValue: '1234567890',
+            preferred: false,
+            required: false,
+            selectedSource: null,
+          },
+          openMrsId: {
+            autoGeneration: false,
+            identifierName: 'OpenMRS ID',
+            identifierTypeUuid: '05a29f94-c0ed-11e2-94be-8c13b969e334',
+            identifierUuid: '1f0ad7a1-430f-4397-b571-59ea654a52db',
+            identifierValue: '100GEJ',
+            initialValue: '100GEJ',
+            preferred: true,
+            required: true,
+            selectedSource: null,
+          },
+        },
+        isDead: false,
+        middleName: '',
+        monthsEstimated: 0,
+        patientUuid: '8673ee4f-e2ab-4077-ba55-4980f408773e',
+        relationships: [],
+        telephoneNumber: '',
+        unidentifiedPatient: undefined,
+        yearsEstimated: 0,
+      },
+      expect.anything(),
+      expect.anything(),
+      null,
+      undefined,
+      expect.anything(),
+      expect.anything(),
+      expect.anything(),
+      { patientSaved: false },
+      expect.anything(),
+    );
+  });
+});

package/src/patient-registration/section/demographics/demographics-section.component.tsx

@@ -1,8 +1,8 @@
 import React, { useContext, useEffect } from 'react';
-import styles from './../section.scss';
 import { useField } from 'formik';
-import { PatientRegistrationContext } from '../../patient-registration-context';
 import { Field } from '../../field/field.component';
+import { PatientRegistrationContext } from '../../patient-registration-context';
+import styles from './../section.scss';
 
 export interface DemographicsSectionProps {
   fields: Array<string>;
@@ -18,7 +18,7 @@ export const DemographicsSection: React.FC<DemographicsSectionProps> = ({ fields
       setFieldValue('additionalMiddleName', '');
       setFieldValue('additionalFamilyName', '');
     }
-  }, [field.value, meta.touched]);
+  }, [field.value, meta.touched, setFieldValue]);
 
   return (
     <section className={styles.formSection} aria-label="Demographics Section">

package/src/patient-registration/section/patient-relationships/relationships-section.component.tsx

@@ -40,7 +40,7 @@ const RelationshipView: React.FC<RelationshipViewProps> = ({
 }) => {
   const { t } = useTranslation();
   const { setFieldValue } = React.useContext(PatientRegistrationContext);
-  const [isInvalid, setIsInvalid] = useState<boolean>(false);
+  const [isInvalid, setIsInvalid] = useState(false);
   const newRelationship = !relationship.uuid;
 
   const handleRelationshipTypeChange = useCallback(