@kenyaemr/esm-admin-app 5.4.4-pre.45 → 5.4.4-pre.452

package/src/components/logs-table/operation-log-table.component.tsx

@@ -12,7 +12,6 @@ import {
   TableRow,
   Tag,
   DataTableSkeleton,
-  InlineLoading,
 } from '@carbon/react';
 import { CardHeader, PatientChartPagination } from '@openmrs/esm-patient-common-lib';
 import { useLayoutType, usePagination, formatDate } from '@openmrs/esm-framework';
@@ -21,13 +20,14 @@ import { ETLResponse } from '../../types';
 import EmptyState from '../empty-state/empty-state-log.components';
 
 interface LogTableProps {
-  logData: ETLResponse[];
+  logData: Array<ETLResponse>;
   isLoading: boolean;
 }
 
+const PAGE_SIZE = 10;
+
 const LogTable: React.FC<LogTableProps> = ({ logData, isLoading }) => {
   const { t } = useTranslation();
-  const pageSize = 10;
   const isTablet = useLayoutType() === 'tablet';
 
   const headers = [
@@ -37,82 +37,94 @@ const LogTable: React.FC<LogTableProps> = ({ logData, isLoading }) => {
     { header: t('completionStatus', 'Completion status'), key: 'status' },
   ];
 
-  const rows = logData?.map((item, index) => ({
-    id: index.toString(),
-    script_name: item.script_name,
-    start_time: item?.start_time ? formatDate(new Date(item?.start_time)) : '--',
-    stop_time: item?.stop_time ? formatDate(new Date(item?.stop_time)) : '--',
-    status: item.status,
-  }));
+  const rows =
+    logData?.map((item, index) => ({
+      id: index.toString(),
+      script_name: item.script_name,
+      start_time: item?.start_time ? formatDate(new Date(item.start_time)) : '--',
+      stop_time: item?.stop_time ? formatDate(new Date(item.stop_time)) : '--',
+      status: item.status,
+    })) ?? [];
+
+  const { results: paginatedData, currentPage, goTo } = usePagination(rows, PAGE_SIZE);
+
+  const renderTable = ({ rows, headers, getHeaderProps, getTableProps }: any) => (
+    <TableContainer className={styles.tableContainer}>
+      <Table {...getTableProps()}>
+        <TableHead>
+          <TableRow>
+            {headers.map((header: any) => (
+              <TableHeader
+                className={classNames(styles.productiveHeading01, styles.text02)}
+                {...getHeaderProps({ header })}>
+                {header.header}
+              </TableHeader>
+            ))}
+          </TableRow>
+        </TableHead>
+        <TableBody>
+          {rows.map((row: any) => (
+            <TableRow key={row.id}>
+              {row.cells.map((cell: any) => (
+                <TableCell key={cell.id}>
+                  {cell.info.header === 'status' ? (
+                    <Tag size="md" type={cell.value === 'Success' ? 'green' : 'red'}>
+                      {cell.value}
+                    </Tag>
+                  ) : (
+                    cell.value
+                  )}
+                </TableCell>
+              ))}
+            </TableRow>
+          ))}
+        </TableBody>
+      </Table>
+    </TableContainer>
+  );
+
+  const renderContent = () => {
+    if (isLoading && logData.length === 0) {
+      return (
+        <DataTableSkeleton
+          headers={headers}
+          aria-label="etl table"
+          showToolbar={false}
+          showHeader={false}
+          rowCount={4}
+          zebra
+          columnCount={3}
+          className={styles.dataTableSkeleton}
+        />
+      );
+    }
+
+    if (logData.length === 0) {
+      return <EmptyState subTitle={t('noRecordsFound', 'No ETL Operation logs found')} />;
+    }
 
-  const { results: paginatedData, currentPage, goTo } = usePagination(rows, pageSize);
+    return (
+      <>
+        <DataTable rows={paginatedData} headers={headers} isSortable size={isTablet ? 'lg' : 'sm'} useZebraStyles>
+          {renderTable}
+        </DataTable>
+        <PatientChartPagination
+          currentItems={paginatedData.length}
+          totalItems={rows.length}
+          onPageNumberChange={({ page }: { page: number }) => goTo(page)}
+          pageNumber={currentPage}
+          pageSize={PAGE_SIZE}
+        />
+      </>
+    );
+  };
 
   return (
     <div className={styles.table}>
-      <CardHeader title={t('facilityInfo', 'Facility Info')} children={''} />
-      <div className={styles.logTable}>
-        {isLoading && logData.length === 0 ? (
-          <DataTableSkeleton
-            headers={headers}
-            aria-label="etl table"
-            showToolbar={false}
-            showHeader={false}
-            rowCount={4}
-            zebra
-            columnCount={3}
-            className={styles.dataTableSkeleton}
-          />
-        ) : logData.length === 0 ? (
-          <EmptyState subTitle={t('noRecordsFound', 'No ETL Operation logs found')} />
-        ) : (
-          <>
-            <DataTable rows={paginatedData} headers={headers} isSortable size={isTablet ? 'lg' : 'sm'} useZebraStyles>
-              {({ rows, headers, getHeaderProps, getTableProps }) => (
-                <TableContainer className={styles.tableContainer}>
-                  <Table {...getTableProps()}>
-                    <TableHead>
-                      <TableRow>
-                        {headers.map((header) => (
-                          <TableHeader
-                            key={header.key}
-                            className={classNames(styles.productiveHeading01, styles.text02)}
-                            {...getHeaderProps({ header })}>
-                            {header.header}
-                          </TableHeader>
-                        ))}
-                      </TableRow>
-                    </TableHead>
-                    <TableBody>
-                      {rows.map((row) => (
-                        <TableRow key={row.id}>
-                          {row.cells.map((cell) => (
-                            <TableCell key={cell.id}>
-                              {cell.info.header === 'status' ? (
-                                <Tag size="md" type={cell.value === 'Success' ? 'green' : 'red'}>
-                                  {cell.value}
-                                </Tag>
-                              ) : (
-                                cell.value
-                              )}
-                            </TableCell>
-                          ))}
-                        </TableRow>
-                      ))}
-                    </TableBody>
-                  </Table>
-                </TableContainer>
-              )}
-            </DataTable>
-            <PatientChartPagination
-              currentItems={paginatedData.length}
-              totalItems={rows.length}
-              onPageNumberChange={({ page }) => goTo(page)}
-              pageNumber={currentPage}
-              pageSize={pageSize}
-            />
-          </>
-        )}
-      </div>
+      <CardHeader title={t('facilityInfo', 'Facility Info')}>
+        <></>
+      </CardHeader>
+      <div className={styles.logTable}>{renderContent()}</div>
     </div>
   );
 };

package/src/components/logs-table/operation-log.test.tsx

@@ -1,47 +1,153 @@
 import React from 'react';
 import { render, screen } from '@testing-library/react';
 import '@testing-library/jest-dom';
+import { vi, describe, it, expect, beforeEach } from 'vitest';
 import LogTable from './operation-log-table.component';
 import { ETLResponse } from '../../types';
 
-describe('LogTable Component', () => {
-  const mockLogData: ETLResponse[] = [
-    {
-      script_name: 'Initial Population of Tables',
-      start_time: '2024-11-28T09:30:12',
-      stop_time: '2024-11-28T09:33:15',
-      status: 'Success',
-    },
-    {
-      script_name: 'Initial Creation of Tables',
-      start_time: '2024-11-28T09:30:12',
-      stop_time: '2024-11-28T09:33:15',
-      status: 'Failed',
+vi.mock('react-i18next', () => ({
+  useTranslation: () => ({ t: (key: string, fallback: string) => fallback }),
+}));
+
+vi.mock('@openmrs/esm-framework', () => ({
+  useLayoutType: vi.fn(() => 'desktop'),
+  usePagination: vi.fn((data) => ({
+    results: data,
+    currentPage: 1,
+    goTo: vi.fn(),
+  })),
+  formatDate: vi.fn((date: Date) => date.toISOString()),
+}));
+
+vi.mock('@openmrs/esm-patient-common-lib', () => ({
+  CardHeader: ({ title }: { title: string }) => <div>{title}</div>,
+  PatientChartPagination: ({ currentItems, totalItems }: { currentItems: number; totalItems: number }) => (
+    <div data-testid="pagination">
+      {currentItems}/{totalItems}
+    </div>
+  ),
+}));
+
+vi.mock('../empty-state/empty-state-log.components', () => ({
+  default: ({ subTitle }: { subTitle: string }) => <div data-testid="empty-state">{subTitle}</div>,
+}));
+
+vi.mock('@carbon/react', async (importOriginal) => {
+  const original = await importOriginal<typeof import('@carbon/react')>();
+  return {
+    ...original,
+    DataTable: ({
+      children,
+      rows,
+      headers,
+    }: {
+      children: (props: any) => React.ReactNode;
+      rows: any[];
+      headers: any[];
+    }) => {
+      if (typeof children === 'function') {
+        const renderRows = rows.map((row) => ({
+          ...row,
+          cells: headers.map((h) => ({
+            id: `${row.id}:${h.key}`,
+            value: row[h.key],
+            info: { header: h.key },
+          })),
+        }));
+        return children({
+          rows: renderRows,
+          headers,
+          getTableProps: () => ({}),
+          getHeaderProps: ({ header }: any) => ({ key: header.key }),
+          getRowProps: () => ({}),
+        });
+      }
+      return children;
     },
-  ];
+  };
+});
 
-  it('renders the table with correct headers and rows', () => {
-    render(<LogTable logData={mockLogData} isLoading={false} />);
+vi.mock('classnames', () => ({ default: (...args: string[]) => args.filter(Boolean).join(' ') }));
 
-    const headers = ['Procedure', 'Start time', 'End time', 'Completion status'];
-    headers.forEach((header) => {
-      expect(screen.getByText(header)).toBeInTheDocument();
-    });
+vi.mock('./operation-log.scss', () => ({ default: {} }));
 
-    const rows = screen.getAllByRole('row');
-    expect(rows.length).toBe(3);
+const mockLogData: ETLResponse[] = [
+  {
+    script_name: 'Script A',
+    start_time: '2024-01-01T08:00:00Z',
+    stop_time: '2024-01-01T09:00:00Z',
+    status: 'Success',
+  },
+  {
+    script_name: 'Script B',
+    start_time: '2024-01-02T10:00:00Z',
+    stop_time: '2024-01-02T11:00:00Z',
+    status: 'Failed',
+  },
+];
+
+describe('LogTable', () => {
+  beforeEach(() => {
+    vi.clearAllMocks();
   });
 
-  it('displays "No ETL Operation logs found" when logData is empty', () => {
-    render(<LogTable logData={[]} isLoading={false} />);
+  it('renders the card header', () => {
+    render(<LogTable logData={mockLogData} isLoading={false} />);
+    expect(screen.getByText('Facility Info')).toBeInTheDocument();
+  });
+
+  it('shows skeleton while loading with no data', () => {
+    render(<LogTable logData={[]} isLoading={true} />);
+    expect(screen.getByRole('table')).toBeInTheDocument();
+  });
 
+  it('shows empty state when not loading and no data', () => {
+    render(<LogTable logData={[]} isLoading={false} />);
+    expect(screen.getByTestId('empty-state')).toBeInTheDocument();
     expect(screen.getByText('No ETL Operation logs found')).toBeInTheDocument();
   });
 
-  it('shows skeleton loading state when isLoading is true', () => {
-    render(<LogTable logData={[]} isLoading={true} />);
+  it('renders table headers when data is present', () => {
+    render(<LogTable logData={mockLogData} isLoading={false} />);
+    expect(screen.getByText('Procedure')).toBeInTheDocument();
+    expect(screen.getByText('Start time')).toBeInTheDocument();
+    expect(screen.getByText('End time')).toBeInTheDocument();
+    expect(screen.getByText('Completion status')).toBeInTheDocument();
+  });
+
+  it('renders script names in the table', () => {
+    render(<LogTable logData={mockLogData} isLoading={false} />);
+    expect(screen.getByText('Script A')).toBeInTheDocument();
+    expect(screen.getByText('Script B')).toBeInTheDocument();
+  });
+
+  it('renders Success status as a green tag', () => {
+    render(<LogTable logData={mockLogData} isLoading={false} />);
+    const successTag = screen.getByText('Success');
+    expect(successTag).toBeInTheDocument();
+  });
+
+  it('renders Failed status as a red tag', () => {
+    render(<LogTable logData={mockLogData} isLoading={false} />);
+    const failedTag = screen.getByText('Failed');
+    expect(failedTag).toBeInTheDocument();
+  });
+
+  it('renders pagination when data is present', () => {
+    render(<LogTable logData={mockLogData} isLoading={false} />);
+    expect(screen.getByTestId('pagination')).toBeInTheDocument();
+  });
+
+  it('renders -- for missing start_time', () => {
+    const dataWithMissingTime: ETLResponse[] = [
+      { script_name: 'Script C', start_time: '', stop_time: '', status: 'Success' },
+    ];
+    render(<LogTable logData={dataWithMissingTime} isLoading={false} />);
+    expect(screen.getAllByText('--').length).toBeGreaterThanOrEqual(2);
+  });
 
-    const skeleton = screen.getByLabelText('etl table');
-    expect(skeleton).toBeInTheDocument();
+  it('does not show empty state when data is present', () => {
+    render(<LogTable logData={mockLogData} isLoading={false} />);
+    expect(screen.queryByTestId('empty-state')).not.toBeInTheDocument();
   });
 });

package/src/components/modal/hwr-confirmation.modal.scss

@@ -2,20 +2,96 @@
 @use '@carbon/layout';
 @use '@carbon/type';
 
+.modalIntro {
+  margin-bottom: layout.$spacing-05;
+  color: colors.$gray-70;
+}
+
 .healthWorkerInfoContainer {
   display: grid;
-  grid-template-columns: 0.25fr 0.75fr;
-  margin: layout.$spacing-02;
+  grid-template-columns: 0.4fr 0.6fr;
+  margin: layout.$spacing-02 0;
+  font-size: 0.875rem;
 }
+
 .healthWorkerInfoLabel {
   min-width: layout.$spacing-11;
-  font-weight: bold;
+  font-weight: 600;
+  color: colors.$gray-70;
 }
+
 .healthWorkerOverview {
   display: flex;
-  margin: layout.$spacing-05;
+  align-items: center;
+  margin: layout.$spacing-05 0;
+  padding: layout.$spacing-05;
+  background-color: colors.$gray-10;
+  border-radius: 4px;
 }
+
 .healthWorkerPhoto {
   display: flex;
   align-items: center;
+  flex-shrink: 0;
+}
+
+.healthWorkerSummary {
+  flex: 1;
+  margin-left: layout.$spacing-05;
+}
+
+.healthWorkerName {
+  margin: 0 0 layout.$spacing-02 0;
+  font-size: 1.25rem;
+  font-weight: 600;
+}
+
+.healthWorkerSubtitle {
+  color: colors.$gray-70;
+  font-size: 0.875rem;
+  margin-bottom: layout.$spacing-03;
+}
+
+.tagRow {
+  display: flex;
+  gap: layout.$spacing-02;
+  flex-wrap: wrap;
+}
+
+.sectionHeading {
+  margin: layout.$spacing-05 0 layout.$spacing-03 0;
+  padding-bottom: layout.$spacing-02;
+  border-bottom: 1px solid colors.$gray-20;
+  font-size: 0.875rem;
+  font-weight: 600;
+  text-transform: uppercase;
+  letter-spacing: 0.04em;
+  color: colors.$gray-70;
+}
+
+.licenseList {
+  display: flex;
+  flex-direction: column;
+  gap: layout.$spacing-03;
+}
+
+.licenseCard {
+  padding: layout.$spacing-04;
+  border: 1px solid colors.$gray-20;
+  border-radius: 4px;
+  background-color: colors.$white;
+}
+
+.licenseHeader {
+  display: flex;
+  justify-content: space-between;
+  align-items: center;
+  margin-bottom: layout.$spacing-03;
+}
+
+.licenseType {
+  font-weight: 600;
+  text-transform: uppercase;
+  font-size: 0.75rem;
+  letter-spacing: 0.04em;
 }