fix(patient): make note not required in care plan (#2158)

Co-authored-by: HospitalRun Bot <[email protected]> Co-authored-by: Matteo Vivona <[email protected]> Co-authored-by: Jack Meyer <[email protected]>
HospitalRun · Jun 23, 2020 · 34e6041 · 34e6041 · vercel · Jun 23, 2020
1 parent 46327fd
commit 34e6041
Show file tree

Hide file tree

Showing 5 changed files with 3 additions and 7 deletions.
diff --git a/src/__tests__/patients/care-plans/AddCarePlanModal.test.tsx b/src/__tests__/patients/care-plans/AddCarePlanModal.test.tsx
@@ -9,6 +9,7 @@ import { Router } from 'react-router-dom'
 import createMockStore from 'redux-mock-store'
 import thunk from 'redux-thunk'
 
+import PatientRepository from '../../../clients/db/PatientRepository'
 import { CarePlanIntent, CarePlanStatus } from '../../../model/CarePlan'
 import Patient from '../../../model/Patient'
 import AddCarePlanModal from '../../../patients/care-plans/AddCarePlanModal'
@@ -42,6 +43,8 @@ describe('Add Care Plan Modal', () => {
 
   const onCloseSpy = jest.fn()
   const setup = () => {
+    jest.spyOn(PatientRepository, 'find').mockResolvedValue(patient)
+    jest.spyOn(PatientRepository, 'saveOrUpdate')
     const store = mockStore({ patient: { patient, carePlanError } } as any)
     const history = createMemoryHistory()
     const wrapper = mount(

diff --git a/src/__tests__/patients/care-plans/CarePlanForm.test.tsx b/src/__tests__/patients/care-plans/CarePlanForm.test.tsx
@@ -226,7 +226,6 @@ describe('Care Plan Form', () => {
     const noteInput = wrapper.findWhere((w) => w.prop('name') === 'note')
     expect(noteInput).toHaveLength(1)
     expect(noteInput.prop('patient.carePlan.note'))
-    expect(noteInput.prop('isRequired')).toBeTruthy()
     expect(noteInput.prop('value')).toEqual(carePlan.note)
   })
 

diff --git a/src/__tests__/patients/patient-slice.test.ts b/src/__tests__/patients/patient-slice.test.ts
@@ -699,7 +699,6 @@ describe('patients slice', () => {
         startDate: 'patient.carePlan.error.startDateRequired',
         endDate: 'patient.carePlan.error.endDateRequired',
         condition: 'patient.carePlan.error.conditionRequired',
-        note: 'patient.carePlan.error.noteRequired',
       }
       const store = mockStore()
       const expectedCarePlan = {} as CarePlan

diff --git a/src/patients/care-plans/CarePlanForm.tsx b/src/patients/care-plans/CarePlanForm.tsx
@@ -145,7 +145,6 @@ const CarePlanForm = (props: Props) => {
       <Row>
         <Column sm={12}>
           <TextFieldWithLabelFormGroup
-            isRequired
             value={carePlan.note}
             label={t('patient.carePlan.note')}
             name="note"

diff --git a/src/patients/patient-slice.ts b/src/patients/patient-slice.ts
@@ -457,10 +457,6 @@ function validateCarePlan(carePlan: CarePlan): AddCarePlanError {
     error.condition = 'patient.carePlan.error.conditionRequired'
   }
 
-  if (!carePlan.note) {
-    error.note = 'patient.carePlan.error.noteRequired'
-  }
-
   return error
 }