feat(patients): fix failing test due to missing mocks

src/__tests__/patients/labs/LabsTab.test.tsx

@@ -17,16 +17,18 @@ import LabRepository from '../../../clients/db/LabRepository'
 
 const expectedPatient = {
   id: '123',
-  labs: [
-    {
-      patientId: '123',
-      type: 'type',
-      status: 'requested',
-      requestedOn: new Date().toISOString(),
-    } as Lab,
-  ],
 } as Patient
 
+const labs = [
+  {
+    id: 'labId',
+    patientId: '123',
+    type: 'type',
+    status: 'requested',
+    requestedOn: new Date().toISOString(),
+  } as Lab,
+]
+
 const mockStore = configureMockStore([thunk])
 const history = createMemoryHistory()
 
@@ -36,7 +38,7 @@ let store: any
 const setup = (patient = expectedPatient, permissions = [Permissions.WritePatients]) => {
   user = { permissions }
   store = mockStore({ patient, user })
-  jest.spyOn(LabRepository, 'findLabsByPatientId').mockResolvedValue(expectedPatient.labs as Lab[])
+  jest.spyOn(LabRepository, 'findAllByPatientId').mockResolvedValue(labs)
   const wrapper = mount(
     <Router history={history}>
       <Provider store={store}>
@@ -50,7 +52,7 @@ const setup = (patient = expectedPatient, permissions = [Permissions.WritePatien
 
 describe('Labs Tab', () => {
   it('should list the patients labs', async () => {
-    const expectedLabs = expectedPatient.labs as Lab[]
+    const expectedLabs = labs
     let wrapper: any
     await act(async () => {
       wrapper = await setup()
@@ -76,8 +78,12 @@ describe('Labs Tab', () => {
     expect(tableData.at(2).text()).toEqual(expectedLabs[0].status)
   })
 
-  it('should render a warning message if the patient does not have any labs', () => {
-    const wrapper = setup({ ...expectedPatient, labs: [] })
+  it('should render a warning message if the patient does not have any labs', async () => {
+    let wrapper: any
+
+    await act(async () => {
+      wrapper = await setup({ ...expectedPatient })
+    })
 
     const alert = wrapper.find(components.Alert)
 

src/__tests__/patients/view/ViewPatient.test.tsx

@@ -22,6 +22,7 @@ import ViewPatient from '../../../patients/view/ViewPatient'
 import * as patientSlice from '../../../patients/patient-slice'
 import Permissions from '../../../model/Permissions'
 import LabsTab from '../../../patients/labs/LabsTab'
+import LabRepository from '../../../clients/db/LabRepository'
 
 const mockStore = configureMockStore([thunk])
 
@@ -48,6 +49,7 @@ describe('ViewPatient', () => {
 
   const setup = (permissions = [Permissions.ReadPatients]) => {
     jest.spyOn(PatientRepository, 'find')
+    jest.spyOn(LabRepository, 'findAllByPatientId').mockResolvedValue([])
     const mockedPatientRepository = mocked(PatientRepository, true)
     mockedPatientRepository.find.mockResolvedValue(patient)
     history = createMemoryHistory()

src/__tests__/scheduling/appointments/new/NewAppointment.test.tsx

@@ -34,7 +34,7 @@ describe('New Appointment', () => {
     mocked(AppointmentRepository, true).save.mockResolvedValue(
       expectedNewAppointment as Appointment,
     )
-    jest.spyOn(LabRepository, 'findLabsByPatientId').mockResolvedValue([] as Lab[])
+    jest.spyOn(LabRepository, 'findAllByPatientId').mockResolvedValue([] as Lab[])
 
     history = createMemoryHistory()
     store = mockStore({

src/clients/db/LabRepository.ts

@@ -10,7 +10,7 @@ export class LabRepository extends Repository<Lab> {
     })
   }
 
-  async findLabsByPatientId(patientId: string): Promise<Lab[]> {
+  async findAllByPatientId(patientId: string): Promise<Lab[]> {
     return super.search({
       selector: {
         $and: [

src/patients/labs/LabsTab.tsx

@@ -19,7 +19,7 @@ const LabsTab = (props: Props) => {
 
   useEffect(() => {
     const fetch = async () => {
-      const fetchedLabs = await LabRepository.findLabsByPatientId(patientId)
+      const fetchedLabs = await LabRepository.findAllByPatientId(patientId)
       setLabs(fetchedLabs)
     }