From 88a7c0863c0a7752b99683e864d461209fe16b5d Mon Sep 17 00:00:00 2001
From: Jack Meyer <jackcmeyer@gmail.com>
Date: Sun, 28 Jun 2020 23:01:07 -0500
Subject: [PATCH] feat(labs): labs tab should use Table component

---
 src/__tests__/patients/labs/LabsTab.test.tsx | 34 ++++++++---------
 src/patients/labs/LabsTab.tsx                | 40 +++++++++-----------
 2 files changed, 34 insertions(+), 40 deletions(-)

diff --git a/src/__tests__/patients/labs/LabsTab.test.tsx b/src/__tests__/patients/labs/LabsTab.test.tsx
index 6e3e42288c..bd7d7fa884 100644
--- a/src/__tests__/patients/labs/LabsTab.test.tsx
+++ b/src/__tests__/patients/labs/LabsTab.test.tsx
@@ -1,5 +1,5 @@
 import * as components from '@hospitalrun/components'
-import format from 'date-fns/format'
+import { Table } from '@hospitalrun/components'
 import { mount, ReactWrapper } from 'enzyme'
 import { createMemoryHistory } from 'history'
 import React from 'react'
@@ -61,23 +61,23 @@ describe('Labs Tab', () => {
   it('should list the patients labs', async () => {
     const { wrapper } = await setup()
 
-    const table = wrapper.find('table')
-    const tableHeader = wrapper.find('thead')
-    const tableHeaders = wrapper.find('th')
-    const tableBody = wrapper.find('tbody')
-    const tableData = wrapper.find('td')
-
-    expect(table).toHaveLength(1)
-    expect(tableHeader).toHaveLength(1)
-    expect(tableBody).toHaveLength(1)
-    expect(tableHeaders.at(0).text()).toEqual('labs.lab.type')
-    expect(tableHeaders.at(1).text()).toEqual('labs.lab.requestedOn')
-    expect(tableHeaders.at(2).text()).toEqual('labs.lab.status')
-    expect(tableData.at(0).text()).toEqual(expectedLabs[0].type)
-    expect(tableData.at(1).text()).toEqual(
-      format(new Date(expectedLabs[0].requestedOn), 'yyyy-MM-dd hh:mm a'),
+    const table = wrapper.find(Table)
+    const columns = table.prop('columns')
+    const actions = table.prop('actions') as any
+    expect(columns[0]).toEqual(expect.objectContaining({ label: 'labs.lab.type', key: 'type' }))
+    expect(columns[1]).toEqual(
+      expect.objectContaining({ label: 'labs.lab.requestedOn', key: 'requestedOn' }),
+    )
+    expect(columns[2]).toEqual(
+      expect.objectContaining({
+        label: 'labs.lab.status',
+        key: 'status',
+      }),
     )
-    expect(tableData.at(2).text()).toEqual(expectedLabs[0].status)
+
+    expect(actions[0]).toEqual(expect.objectContaining({ label: 'actions.view' }))
+    expect(table.prop('actionsHeaderText')).toEqual('actions.label')
+    expect(table.prop('data')).toEqual(expectedLabs)
   })
 
   it('should render a warning message if the patient does not have any labs', async () => {
diff --git a/src/patients/labs/LabsTab.tsx b/src/patients/labs/LabsTab.tsx
index 03ec2a2a71..5461183ac1 100644
--- a/src/patients/labs/LabsTab.tsx
+++ b/src/patients/labs/LabsTab.tsx
@@ -1,4 +1,4 @@
-import { Alert } from '@hospitalrun/components'
+import { Alert, Table } from '@hospitalrun/components'
 import format from 'date-fns/format'
 import React, { useEffect, useState } from 'react'
 import { useTranslation } from 'react-i18next'
@@ -27,10 +27,6 @@ const LabsTab = (props: Props) => {
     fetch()
   }, [patientId])
 
-  const onTableRowClick = (lab: Lab) => {
-    history.push(`/labs/${lab.id}`)
-  }
-
   return (
     <div>
       {(!labs || labs.length === 0) && (
@@ -41,24 +37,22 @@ const LabsTab = (props: Props) => {
         />
       )}
       {labs && labs.length > 0 && (
-        <table className="table table-hover">
-          <thead className="thead-light">
-            <tr>
-              <th>{t('labs.lab.type')}</th>
-              <th>{t('labs.lab.requestedOn')}</th>
-              <th>{t('labs.lab.status')}</th>
-            </tr>
-          </thead>
-          <tbody>
-            {labs.map((lab) => (
-              <tr onClick={() => onTableRowClick(lab)} key={lab.id}>
-                <td>{lab.type}</td>
-                <td>{format(new Date(lab.requestedOn), 'yyyy-MM-dd hh:mm a')}</td>
-                <td>{lab.status}</td>
-              </tr>
-            ))}
-          </tbody>
-        </table>
+        <Table
+          tableClassName="table table-hover"
+          actionsHeaderText={t('actions.label')}
+          getID={(row) => row.id}
+          data={labs}
+          columns={[
+            { label: t('labs.lab.type'), key: 'type' },
+            {
+              label: t('labs.lab.requestedOn'),
+              key: 'requestedOn',
+              formatter: (row) => format(new Date(row.requestedOn), 'yyyy-MM-dd hh:mm a'),
+            },
+            { label: t('labs.lab.status'), key: 'status' },
+          ]}
+          actions={[{ label: t('actions.view'), action: (row) => history.push(`/labs/${row.id}`) }]}
+        />
       )}
     </div>
   )