feat(viewpatients): add Tests, Fix bug

feat #1969

src/__tests__/components/PageComponent.test.tsx

@@ -0,0 +1,31 @@
+import '../../__mocks__/matchMediaMock'
+import { Button, Select } from '@hospitalrun/components'
+import { mount } from 'enzyme'
+import React from 'react'
+
+import PageComponent, { defaultPageSize } from '../../components/PageComponent'
+
+describe('PageComponenet test', () => {
+  it('should render PageComponent Component', () => {
+    const wrapper = mount(
+      <PageComponent
+        hasNext={false}
+        hasPrevious={false}
+        pageNumber={1}
+        setPreviousPageRequest={jest.fn()}
+        setNextPageRequest={jest.fn()}
+        onPageSizeChange={jest.fn()}
+      />,
+    )
+    const buttons = wrapper.find(Button)
+    expect(buttons).toHaveLength(2)
+    expect(buttons.at(0).prop('disabled')).toBeTruthy()
+    expect(buttons.at(1).prop('disabled')).toBeTruthy()
+
+    const select = wrapper.find(Select)
+    expect(select.prop('defaultValue')).toEqual(defaultPageSize.value?.toString())
+
+    const options = select.find('option')
+    expect(options).toHaveLength(5)
+  })
+})

src/__tests__/hooks/useUpdateEffect.test.ts

@@ -0,0 +1,15 @@
+import { renderHook } from '@testing-library/react-hooks'
+
+import useUpdateEffect from '../../hooks/useUpdateEffect'
+
+describe('useUpdateEffect', () => {
+  it('should call the function after udpate', () => {
+    const mockFn = jest.fn()
+    let someVal = 'someVal'
+    const { rerender } = renderHook(() => useUpdateEffect(mockFn, [someVal]))
+    expect(mockFn).not.toHaveBeenCalled()
+    someVal = 'newVal'
+    rerender()
+    expect(mockFn).toHaveBeenCalledTimes(1)
+  })
+})

src/__tests__/patients/list/ViewPatients.test.tsx

@@ -1,6 +1,6 @@
 import '../../../__mocks__/matchMediaMock'
 
-import { TextInput, Spinner } from '@hospitalrun/components'
+import { TextInput, Spinner, Select } from '@hospitalrun/components'
 import format from 'date-fns/format'
 import { mount } from 'enzyme'
 import React from 'react'
@@ -13,6 +13,7 @@ import { mocked } from 'ts-jest/utils'
 
 import { UnpagedRequest } from '../../../clients/db/PageRequest'
 import PatientRepository from '../../../clients/db/PatientRepository'
+import SortRequest from '../../../clients/db/SortRequest'
 import Page from '../../../clients/Page'
 import { defaultPageSize } from '../../../components/PageComponent'
 import Patient from '../../../model/Patient'
@@ -155,6 +156,47 @@ describe('Patients', () => {
     })
   })
 
+  describe('change page size', () => {
+    afterEach(() => {
+      jest.restoreAllMocks()
+    })
+    it('should call the change handler on change', () => {
+      const searchPagedSpy = jest.spyOn(patientSlice, 'searchPatients')
+      const wrapper = setup()
+      const sortRequest: SortRequest = {
+        sorts: [{ field: 'index', direction: 'asc' }],
+      }
+
+      expect(searchPagedSpy).toBeCalledWith('', sortRequest, {
+        direction: 'next',
+        nextPageInfo: { index: null },
+        number: 1,
+        previousPageInfo: { index: null },
+        size: defaultPageSize.value,
+      })
+
+      act(() => {
+        ;(wrapper.find(Select).prop('onChange') as any)({
+          target: {
+            value: '50',
+          },
+        } as React.ChangeEvent<HTMLInputElement>)
+      })
+
+      wrapper.update()
+
+      expect(searchPagedSpy).toHaveBeenCalledTimes(2)
+
+      expect(searchPagedSpy).toBeCalledWith('', sortRequest, {
+        direction: 'next',
+        nextPageInfo: { index: null },
+        number: 1,
+        previousPageInfo: { index: null },
+        size: 50,
+      })
+    })
+  })
+
   describe('search functionality', () => {
     beforeEach(() => jest.useFakeTimers())
 

src/clients/db/Repository.ts

@@ -98,17 +98,17 @@ export default class Repository<T extends AbstractDBModel> {
     }
 
     const nextPageInfo: { [key: string]: string } = {}
+    const previousPageInfo: { [key: string]: string } = {}
+
     if (mappedResult.length > 0) {
       sort.sorts.forEach((s) => {
         nextPageInfo[s.field] = mappedResult[mappedResult.length - 1][s.field]
       })
+      sort.sorts.forEach((s) => {
+        previousPageInfo[s.field] = mappedResult[0][s.field]
+      })
     }
 
-    const previousPageInfo: { [key: string]: string } = {}
-    sort.sorts.forEach((s) => {
-      previousPageInfo[s.field] = mappedResult[0][s.field]
-    })
-
     const hasNext: boolean =
       pageRequest.size !== undefined && mappedResult.length === pageRequest.size + 1
     const hasPrevious: boolean = pageRequest.number !== undefined && pageRequest.number > 1