From 6edc0ad3ba352c92c8d85ec58148fedb9fc57a98 Mon Sep 17 00:00:00 2001
From: Jack Meyer <jackcmeyer@gmail.com>
Date: Thu, 2 Apr 2020 20:58:23 -0500
Subject: [PATCH] feat(patient): add notes tab tests

---
 .../patients/notes/NotesTab.test.tsx          | 129 ++++++++++++++++++
 src/patients/notes/NoteTab.tsx                |   8 +-
 2 files changed, 133 insertions(+), 4 deletions(-)
 create mode 100644 src/__tests__/patients/notes/NotesTab.test.tsx

diff --git a/src/__tests__/patients/notes/NotesTab.test.tsx b/src/__tests__/patients/notes/NotesTab.test.tsx
new file mode 100644
index 0000000000..efff6211eb
--- /dev/null
+++ b/src/__tests__/patients/notes/NotesTab.test.tsx
@@ -0,0 +1,129 @@
+import '../../../__mocks__/matchMediaMock'
+import React from 'react'
+import PatientRepository from 'clients/db/PatientRepository'
+import Note from 'model/Note'
+import { createMemoryHistory } from 'history'
+import configureMockStore from 'redux-mock-store'
+import Patient from 'model/Patient'
+import thunk from 'redux-thunk'
+import { mount } from 'enzyme'
+import { Router } from 'react-router'
+import { Provider } from 'react-redux'
+import NoteTab from 'patients/notes/NoteTab'
+import * as components from '@hospitalrun/components'
+import { act } from 'react-dom/test-utils'
+import { mocked } from 'ts-jest/utils'
+import NewNoteModal from 'patients/notes/NewNoteModal'
+import Permissions from '../../../model/Permissions'
+import * as patientSlice from '../../../patients/patient-slice'
+
+const expectedPatient = {
+  id: '123',
+  notes: [{ date: new Date().toISOString(), text: 'notes1' } as Note],
+} as Patient
+
+const mockStore = configureMockStore([thunk])
+const history = createMemoryHistory()
+
+let user: any
+let store: any
+
+const setup = (patient = expectedPatient, permissions = [Permissions.WritePatients]) => {
+  user = { permissions }
+  store = mockStore({ patient, user })
+  const wrapper = mount(
+    <Router history={history}>
+      <Provider store={store}>
+        <NoteTab patient={patient} />
+      </Provider>
+    </Router>,
+  )
+
+  return wrapper
+}
+
+describe('Notes Tab', () => {
+  describe('Add New Note', () => {
+    beforeEach(() => {
+      jest.resetAllMocks()
+      jest.spyOn(PatientRepository, 'saveOrUpdate')
+    })
+
+    it('should render a add notes button', () => {
+      const wrapper = setup()
+
+      const addNoteButton = wrapper.find(components.Button)
+      expect(addNoteButton).toHaveLength(1)
+      expect(addNoteButton.text().trim()).toEqual('patient.notes.new')
+    })
+
+    it('should not render a add notes button if the user does not have permissions', () => {
+      const wrapper = setup(expectedPatient, [])
+
+      const addNotesButton = wrapper.find(components.Button)
+      expect(addNotesButton).toHaveLength(0)
+    })
+
+    it('should open the Add Notes Modal', () => {
+      const wrapper = setup()
+
+      act(() => {
+        const onClick = wrapper.find(components.Button).prop('onClick') as any
+        onClick()
+      })
+      wrapper.update()
+
+      expect(wrapper.find(components.Modal).prop('show')).toBeTruthy()
+    })
+
+    it('should update the patient with the new diagnosis when the save button is clicked', async () => {
+      const expectedNote = {
+        text: 'note text',
+        date: new Date().toISOString(),
+      } as Note
+      const expectedUpdatedPatient = {
+        ...expectedPatient,
+        notes: [...(expectedPatient.notes as any), expectedNote],
+      } as Patient
+
+      const mockedPatientRepository = mocked(PatientRepository, true)
+      mockedPatientRepository.saveOrUpdate.mockResolvedValue(expectedUpdatedPatient)
+
+      const wrapper = setup()
+
+      await act(async () => {
+        const modal = wrapper.find(NewNoteModal)
+        await modal.prop('onSave')(expectedNote)
+      })
+
+      expect(mockedPatientRepository.saveOrUpdate).toHaveBeenCalledWith(expectedUpdatedPatient)
+      expect(store.getActions()).toContainEqual(patientSlice.updatePatientStart())
+      expect(store.getActions()).toContainEqual(
+        patientSlice.updatePatientSuccess(expectedUpdatedPatient),
+      )
+    })
+  })
+
+  describe('notes list', () => {
+    it('should list the patients diagnoses', () => {
+      const notes = expectedPatient.notes as Note[]
+      const wrapper = setup()
+
+      const list = wrapper.find(components.List)
+      const listItems = wrapper.find(components.ListItem)
+
+      expect(list).toHaveLength(1)
+      expect(listItems).toHaveLength(notes.length)
+    })
+
+    it('should render a warning message if the patient does not have any diagnoses', () => {
+      const wrapper = setup({ ...expectedPatient, notes: [] })
+
+      const alert = wrapper.find(components.Alert)
+
+      expect(alert).toHaveLength(1)
+      expect(alert.prop('title')).toEqual('patient.notes.warning.noNotes')
+      expect(alert.prop('message')).toEqual('patient.notes.addNoteAbove')
+    })
+  })
+})
diff --git a/src/patients/notes/NoteTab.tsx b/src/patients/notes/NoteTab.tsx
index ee8570312d..a798a60635 100644
--- a/src/patients/notes/NoteTab.tsx
+++ b/src/patients/notes/NoteTab.tsx
@@ -76,10 +76,10 @@ const NoteTab = (props: Props) => {
         />
       )}
       <List>
-        {patient.notes?.map((a: Note) => (
-          <ListItem>
-            {new Date(a.date).toLocaleString()}
-            <div dangerouslySetInnerHTML={{ __html: a.text }} />
+        {patient.notes?.map((note: Note) => (
+          <ListItem key={note.date}>
+            {new Date(note.date).toLocaleString()}
+            <p>{note.text}</p>
           </ListItem>
         ))}
       </List>