@openmrs/esm-billing-app 1.1.2-pre.8 → 1.2.0

package/src/invoice/payments/payment-form/payment-form.test.tsx

@@ -1,4 +1,5 @@
 import React from 'react';
+import { describe, expect, it, vi } from 'vitest';
 import userEvent from '@testing-library/user-event';
 import { render, screen } from '@testing-library/react';
 import { FormProvider, useForm } from 'react-hook-form';
@@ -6,11 +7,11 @@ import type { PaymentFormValue } from '../payments.component';
 import { usePaymentModes } from '../payment.resource';
 import PaymentForm from './payment-form.component';
 
-jest.mock('../payment.resource', () => ({
-  usePaymentModes: jest.fn(),
+vi.mock('../payment.resource', () => ({
+  usePaymentModes: vi.fn(),
 }));
 
-const mockUsePaymentModes = jest.mocked(usePaymentModes);
+const mockUsePaymentModes = vi.mocked(usePaymentModes);
 
 type WrapperProps = {
   children: React.ReactNode;
@@ -26,12 +27,12 @@ const Wrapper: React.FC<WrapperProps> = ({ children, defaultValues }) => {
 };
 
 describe('PaymentForm Component', () => {
-  test('should render skeleton while loading payment modes', () => {
+  it('should render skeleton while loading payment modes', () => {
     mockUsePaymentModes.mockReturnValue({
       paymentModes: [],
       isLoading: true,
       error: null,
-      mutate: jest.fn(),
+      mutate: vi.fn(),
     });
 
     render(
@@ -45,12 +46,12 @@ describe('PaymentForm Component', () => {
     expect(screen.queryByPlaceholderText(/enter amount/i)).not.toBeInTheDocument();
   });
 
-  test('should render error message when payment modes fail to load', () => {
+  it('should render error message when payment modes fail to load', () => {
     mockUsePaymentModes.mockReturnValue({
       paymentModes: [],
       isLoading: false,
       error: new Error('Failed to load payment modes'),
-      mutate: jest.fn(),
+      mutate: vi.fn(),
     });
 
     render(
@@ -62,12 +63,12 @@ describe('PaymentForm Component', () => {
     expect(screen.getByText(/error state/i)).toBeInTheDocument();
   });
 
-  test('should render payment form with method, amount and reference fields when not disabled', () => {
+  it('should render payment form with method, amount and reference fields when not disabled', () => {
     mockUsePaymentModes.mockReturnValue({
       paymentModes: [{ uuid: '1', name: 'Credit Card', description: 'Credit Card', retired: false }],
       isLoading: false,
       error: null,
-      mutate: jest.fn(),
+      mutate: vi.fn(),
     });
 
     render(
@@ -81,12 +82,12 @@ describe('PaymentForm Component', () => {
     expect(screen.getByText(/select payment method/i)).toBeInTheDocument();
   });
 
-  test('should not render form when disablePayment is true', () => {
+  it('should not render form when disablePayment is true', () => {
     mockUsePaymentModes.mockReturnValue({
       paymentModes: [{ uuid: '1', name: 'Credit Card', description: 'Credit Card', retired: false }],
       isLoading: false,
       error: null,
-      mutate: jest.fn(),
+      mutate: vi.fn(),
     });
 
     render(
@@ -99,12 +100,12 @@ describe('PaymentForm Component', () => {
     expect(screen.queryByText(/add payment method/i)).not.toBeInTheDocument();
   });
 
-  test('should render amount input without leading zero', () => {
+  it('should render amount input without leading zero', () => {
     mockUsePaymentModes.mockReturnValue({
       paymentModes: [{ uuid: '1', name: 'Credit Card', description: 'Credit Card', retired: false }],
       isLoading: false,
       error: null,
-      mutate: jest.fn(),
+      mutate: vi.fn(),
     });
 
     render(
@@ -117,13 +118,13 @@ describe('PaymentForm Component', () => {
     expect(amountInput.value).toBe('');
   });
 
-  test('should allow user to clear amount input without reverting to zero', async () => {
+  it('should allow user to clear amount input without reverting to zero', async () => {
     const user = userEvent.setup();
     mockUsePaymentModes.mockReturnValue({
       paymentModes: [{ uuid: '1', name: 'Credit Card', description: 'Credit Card', retired: false }],
       isLoading: false,
       error: null,
-      mutate: jest.fn(),
+      mutate: vi.fn(),
     });
 
     render(
@@ -141,13 +142,13 @@ describe('PaymentForm Component', () => {
     expect(amountInput.value).toBe('');
   });
 
-  test('should handle amount input with decimal values', async () => {
+  it('should handle amount input with decimal values', async () => {
     const user = userEvent.setup();
     mockUsePaymentModes.mockReturnValue({
       paymentModes: [{ uuid: '1', name: 'Credit Card', description: 'Credit Card', retired: false }],
       isLoading: false,
       error: null,
-      mutate: jest.fn(),
+      mutate: vi.fn(),
     });
 
     render(
@@ -162,12 +163,12 @@ describe('PaymentForm Component', () => {
     expect(amountInput.value).toBe('10.5');
   });
 
-  test('should not auto-focus reference number input on mount', () => {
+  it('should not auto-focus reference number input on mount', () => {
     mockUsePaymentModes.mockReturnValue({
       paymentModes: [{ uuid: '1', name: 'Credit Card', description: 'Credit Card', retired: false }],
       isLoading: false,
       error: null,
-      mutate: jest.fn(),
+      mutate: vi.fn(),
     });
 
     render(

package/src/invoice/payments/payment-history/payment-history.test.tsx

@@ -1,15 +1,16 @@
 import React from 'react';
+import { beforeEach, describe, expect, it, vi } from 'vitest';
 import { render, screen } from '@testing-library/react';
 import { getDefaultsFromConfigSchema, useConfig } from '@openmrs/esm-framework';
 import { type BillingConfig, configSchema } from '../../../config-schema';
 import { type MappedBill } from '../../../types';
 import PaymentHistory from './payment-history.component';
 
-jest.mock('../../../helpers', () => ({
-  convertToCurrency: jest.fn((amount, currency) => `${currency} ${amount.toFixed(2)}`),
+vi.mock('../../../helpers', () => ({
+  convertToCurrency: vi.fn((amount, currency) => `${currency} ${amount.toFixed(2)}`),
 }));
 
-const mockUseConfig = jest.mocked(useConfig<BillingConfig>);
+const mockUseConfig = vi.mocked(useConfig<BillingConfig>);
 
 describe('PaymentHistory Component', () => {
   beforeEach(() => {
@@ -73,6 +74,7 @@ describe('PaymentHistory Component', () => {
     dateCreated: '2023-09-01T12:00:00Z',
     lineItems: [],
     billingService: 'Billing Service',
+    netAmount: 0,
   };
 
   const emptyBill: MappedBill = {
@@ -101,13 +103,14 @@ describe('PaymentHistory Component', () => {
     dateCreated: '2023-09-02T10:00:00Z',
     lineItems: [],
     billingService: 'Billing Service',
+    netAmount: 0,
   };
 
-  test('renders without crashing', () => {
+  it('renders without crashing', () => {
     render(<PaymentHistory bill={mockBill} />);
   });
 
-  test('renders correct table headers', () => {
+  it('renders correct table headers', () => {
     render(<PaymentHistory bill={mockBill} />);
     expect(screen.getByText('Date of payment')).toBeInTheDocument();
     expect(screen.getByText('Bill amount')).toBeInTheDocument();
@@ -115,13 +118,13 @@ describe('PaymentHistory Component', () => {
     expect(screen.getByText('Payment method')).toBeInTheDocument();
   });
 
-  test('renders the correct number of rows', () => {
+  it('renders the correct number of rows', () => {
     render(<PaymentHistory bill={mockBill} />);
     const rows = screen.getAllByRole('row');
     expect(rows).toHaveLength(3);
   });
 
-  test('renders correct data in the rows', () => {
+  it('renders correct data in the rows', () => {
     render(<PaymentHistory bill={mockBill} />);
 
     expect(screen.getByText('01-Sept-2023')).toBeInTheDocument();
@@ -135,17 +138,17 @@ describe('PaymentHistory Component', () => {
     expect(screen.getByText('Cash')).toBeInTheDocument();
   });
 
-  test('handles empty payments gracefully', () => {
+  it('handles empty payments gracefully', () => {
     render(<PaymentHistory bill={emptyBill} />);
     expect(screen.queryByRole('table')).not.toBeInTheDocument();
   });
 
-  test('does not render when bill is undefined', () => {
+  it('does not render when bill is undefined', () => {
     render(<PaymentHistory bill={undefined} />);
     expect(screen.queryByRole('table')).not.toBeInTheDocument();
   });
 
-  test('formats dates and converts amounts correctly', () => {
+  it('formats dates and converts amounts correctly', () => {
     render(<PaymentHistory bill={mockBill} />);
 
     expect(screen.getByText('01-Sept-2023')).toBeInTheDocument();

package/src/invoice/payments/payments.component.tsx

@@ -8,10 +8,11 @@ import { CardHeader, navigate, showSnackbar, useConfig, useVisit } from '@openmr
 import { InvoiceBreakDown } from './invoice-breakdown/invoice-breakdown.component';
 import PaymentForm from './payment-form/payment-form.component';
 import PaymentHistory from './payment-history/payment-history.component';
-import { processBillPayment } from '../../billing.resource';
+import { useSWRConfig } from 'swr';
+import { processBillPayment, patientPaymentStatusCacheKey } from '../../billing.resource';
 import { updateBillVisitAttribute } from './payment.resource';
 import { convertToCurrency } from '../../helpers';
-import { BillStatus, type MappedBill } from '../../types';
+import { BillStatus, RefundStatus, type MappedBill } from '../../types';
 import styles from './payments.scss';
 
 type PaymentProps = {
@@ -27,6 +28,7 @@ export type PaymentFormValue = {
 
 const Payments: React.FC<PaymentProps> = ({ bill, mutate }) => {
   const { t } = useTranslation();
+  const { mutate: swrMutate } = useSWRConfig();
   const paymentSchema = z.object({
     method: z.string().refine((value) => !!value, 'Payment method is required'),
     amount: z
@@ -35,7 +37,7 @@ const Payments: React.FC<PaymentProps> = ({ bill, mutate }) => {
         invalid_type_error: t('amountRequired', 'Amount is required'),
       })
       .positive({ message: t('amountMustBePositive', 'Amount must be greater than 0') })
-      .max(bill?.totalAmount - bill?.tenderedAmount, {
+      .max((bill?.netAmount ?? 0) - (bill?.tenderedAmount ?? 0), {
         message: t('paymentAmountCannotExceedAmountDue', 'Payment amount cannot exceed amount due'),
       }),
     referenceCode: z.union([z.number(), z.string()]).optional(),
@@ -69,7 +71,11 @@ const Payments: React.FC<PaymentProps> = ({ bill, mutate }) => {
     return null;
   }
 
-  const amountDue = (bill.totalAmount ?? 0) - (bill.tenderedAmount ?? 0);
+  const refundTotal = (bill.refunds ?? [])
+    .filter((r) => r.status === RefundStatus.COMPLETED)
+    .reduce((sum, r) => sum + r.refundAmount, 0);
+
+  const amountDue = (bill.netAmount ?? 0) - (bill.tenderedAmount ?? 0);
 
   const handleProcessPayment = async () => {
     if (!formValues?.method || formValues.amount == null) {
@@ -95,6 +101,7 @@ const Payments: React.FC<PaymentProps> = ({ bill, mutate }) => {
       }
       methods.reset(defaultPaymentValues);
       mutate();
+      swrMutate(patientPaymentStatusCacheKey(bill.patientUuid));
     } catch (error) {
       showSnackbar({
         title: t('errorProcessingPayment', 'Error processing payment'),
@@ -126,11 +133,18 @@ const Payments: React.FC<PaymentProps> = ({ bill, mutate }) => {
             label={t('totalTendered', 'Total tendered')}
             value={convertToCurrency(bill.tenderedAmount ?? 0, defaultCurrency)}
           />
-          <InvoiceBreakDown label={t('discount', 'Discount')} value={'--'} />
+          <InvoiceBreakDown
+            label={t('discount', 'Discount')}
+            value={`- ${convertToCurrency((bill.totalAmount ?? 0) - (bill.netAmount ?? 0), defaultCurrency)}`}
+          />
+          <InvoiceBreakDown
+            label={t('refunds', 'Refunds')}
+            value={`- ${convertToCurrency(refundTotal, defaultCurrency)}`}
+          />
           <InvoiceBreakDown
             hasBalance={amountDue < 0}
             label={t('amountDue', 'Amount due')}
-            value={convertToCurrency(amountDue < 0 ? -amountDue : amountDue, defaultCurrency)}
+            value={convertToCurrency(amountDue, defaultCurrency)}
           />
           <div className={styles.processPayments}>
             <Button onClick={handleNavigateToBillingDashboard} kind="secondary">

package/src/invoice/payments/payments.test.tsx

@@ -1,4 +1,5 @@
 import React from 'react';
+import { beforeEach, describe, expect, it, vi } from 'vitest';
 import userEvent from '@testing-library/user-event';
 import { render, screen } from '@testing-library/react';
 import {
@@ -9,18 +10,19 @@ import {
   type VisitReturnType,
 } from '@openmrs/esm-framework';
 import { useBillableServices } from '../../billable-services/billable-service.resource';
+import { processBillPayment } from '../../billing.resource';
 import { BillStatus, type MappedBill } from '../../types';
 import { configSchema, type BillingConfig } from '../../config-schema';
 import { usePaymentModes } from './payment.resource';
 import Payments from './payments.component';
 
-const mockUseVisit = jest.mocked(useVisit);
-const mockUseConfig = jest.mocked(useConfig<BillingConfig>);
-const mockUseBillableServices = jest.mocked(useBillableServices);
-const mockUsePaymentModes = jest.mocked(usePaymentModes);
-const mockFormatToParts = jest.fn().mockReturnValue([{ type: 'integer', value: '1000' }]);
-const mockFormat = jest.fn().mockReturnValue('$1000.00');
-const mockResolvedOptions = jest.fn().mockReturnValue({
+const mockUseVisit = vi.mocked(useVisit);
+const mockUseConfig = vi.mocked(useConfig<BillingConfig>);
+const mockUseBillableServices = vi.mocked(useBillableServices);
+const mockUsePaymentModes = vi.mocked(usePaymentModes);
+const mockFormatToParts = vi.fn().mockReturnValue([{ type: 'integer', value: '1000' }]);
+const mockFormat = vi.fn().mockReturnValue('$1000.00');
+const mockResolvedOptions = vi.fn().mockReturnValue({
   locale: 'en-US',
   numberingSystem: 'latn',
   style: 'currency',
@@ -29,24 +31,24 @@ const mockResolvedOptions = jest.fn().mockReturnValue({
   maximumFractionDigits: 2,
 });
 
-global.Intl.NumberFormat.supportedLocalesOf = jest.fn().mockReturnValue(['en-US']);
-global.Intl.NumberFormat = jest.fn().mockImplementation(() => ({
-  formatToParts: mockFormatToParts,
-  format: mockFormat,
-  resolvedOptions: mockResolvedOptions,
-})) as any;
+global.Intl.NumberFormat = vi.fn().mockImplementation(function () {
+  this.formatToParts = mockFormatToParts;
+  this.format = mockFormat;
+  this.resolvedOptions = mockResolvedOptions;
+}) as any;
+global.Intl.NumberFormat.supportedLocalesOf = vi.fn().mockReturnValue(['en-US']);
 
-jest.mock('../../billing.resource', () => ({
-  processBillPayment: jest.fn(),
+vi.mock('../../billing.resource', () => ({
+  processBillPayment: vi.fn(),
 }));
 
-jest.mock('./payment.resource', () => ({
-  updateBillVisitAttribute: jest.fn(),
-  usePaymentModes: jest.fn(),
+vi.mock('./payment.resource', () => ({
+  updateBillVisitAttribute: vi.fn(),
+  usePaymentModes: vi.fn(),
 }));
 
-jest.mock('../../billable-services/billable-service.resource', () => ({
-  useBillableServices: jest.fn(),
+vi.mock('../../billable-services/billable-service.resource', () => ({
+  useBillableServices: vi.fn(),
 }));
 
 describe('Payments', () => {
@@ -107,9 +109,10 @@ describe('Payments', () => {
     dateCreated: '2023-09-01T12:00:00Z',
     lineItems: [],
     billingService: 'Billing Service',
+    netAmount: 0,
   };
 
-  const mockMutate = jest.fn();
+  const mockMutate = vi.fn();
 
   beforeEach(() => {
     mockUseVisit.mockReturnValue({ currentVisit: null } as unknown as VisitReturnType);
@@ -119,7 +122,7 @@ describe('Payments', () => {
       isLoading: false,
       isValidating: false,
       error: null,
-      mutate: jest.fn(),
+      mutate: vi.fn(),
     });
     mockUsePaymentModes.mockReturnValue({
       paymentModes: [
@@ -128,7 +131,7 @@ describe('Payments', () => {
       ],
       isLoading: false,
       error: null,
-      mutate: jest.fn(),
+      mutate: vi.fn(),
     });
   });
 
@@ -243,6 +246,7 @@ describe('Payments', () => {
       ...mockBill,
       status: BillStatus.POSTED,
       totalAmount: 100,
+      netAmount: 100,
       tenderedAmount: 0,
     };
 
@@ -262,6 +266,7 @@ describe('Payments', () => {
       ...mockBill,
       status: BillStatus.POSTED,
       totalAmount: 100,
+      netAmount: 100,
       tenderedAmount: 0,
       lineItems: [],
     };
@@ -272,6 +277,66 @@ describe('Payments', () => {
     expect(screen.getByText(/select payment method/i)).toBeInTheDocument();
   });
 
+  it('blocks payment greater than (netAmount - tendered) on a discounted bill', async () => {
+    const user = userEvent.setup();
+    const mockProcessBillPayment = vi.mocked(processBillPayment);
+    mockProcessBillPayment.mockResolvedValue({} as any);
+
+    const billWithDiscount: MappedBill = {
+      ...mockBill,
+      status: BillStatus.POSTED,
+      totalAmount: 500,
+      netAmount: 400,
+      tenderedAmount: 0,
+      payments: [],
+      lineItems: [],
+    };
+
+    render(<Payments bill={billWithDiscount} mutate={mockMutate} />);
+
+    const methodDropdown = await screen.findByRole('combobox', { name: /payment method/i });
+    await user.click(methodDropdown);
+    await user.click(screen.getByText('Cash'));
+
+    // 450 > netAmount (400). If the schema bound regressed to totalAmount (500) this would pass.
+    await user.type(screen.getByPlaceholderText(/enter amount/i), '450');
+
+    expect(screen.getByText('Process Payment')).toBeDisabled();
+    expect(mockProcessBillPayment).not.toHaveBeenCalled();
+  });
+
+  it('sends a non-null amount in the payment payload when processing a discounted bill', async () => {
+    const user = userEvent.setup();
+    const mockProcessBillPayment = vi.mocked(processBillPayment);
+    mockProcessBillPayment.mockResolvedValue({} as any);
+
+    const billWithDiscount: MappedBill = {
+      ...mockBill,
+      status: BillStatus.POSTED,
+      totalAmount: 500,
+      netAmount: 400,
+      tenderedAmount: 0,
+      payments: [],
+      lineItems: [],
+    };
+
+    render(<Payments bill={billWithDiscount} mutate={mockMutate} />);
+
+    const methodDropdown = await screen.findByRole('combobox', { name: /payment method/i });
+    await user.click(methodDropdown);
+    await user.click(screen.getByText('Cash'));
+
+    await user.type(screen.getByPlaceholderText(/enter amount/i), '100');
+
+    await user.click(screen.getByText('Process Payment'));
+
+    expect(mockProcessBillPayment).toHaveBeenCalledTimes(1);
+    const [payload] = mockProcessBillPayment.mock.calls[0];
+    expect(payload.amount).toBeTypeOf('number');
+    expect(payload.amount).not.toBeNaN();
+    expect(payload.amountTendered).toBe(100);
+  });
+
   it('should display process payment button', () => {
     const billWithAmountDue: MappedBill = {
       ...mockBill,

package/src/invoice/printable-invoice/print-receipt.test.tsx

@@ -1,4 +1,5 @@
-import React, { act } from 'react';
+import React from 'react';
+import { afterEach, beforeEach, describe, expect, it, vi } from 'vitest';
 import userEvent from '@testing-library/user-event';
 import { render, screen, waitFor } from '@testing-library/react';
 import PrintReceipt from './print-receipt.component';
@@ -20,10 +21,10 @@ describe('PrintReceipt', () => {
     });
 
     mockLink = document.createElement('a');
-    jest.spyOn(mockLink, 'click').mockImplementation(() => {});
+    vi.spyOn(mockLink, 'click').mockImplementation(() => {});
 
     const originalCreateElement = document.createElement.bind(document);
-    jest.spyOn(document, 'createElement').mockImplementation((tagName: string) => {
+    vi.spyOn(document, 'createElement').mockImplementation((tagName: string) => {
       if (tagName === 'a') {
         mockLink.href = '';
         mockLink.download = '';
@@ -31,13 +32,10 @@ describe('PrintReceipt', () => {
       }
       return originalCreateElement(tagName);
     });
-
-    jest.useFakeTimers();
   });
 
   afterEach(() => {
-    jest.restoreAllMocks();
-    jest.useRealTimers();
+    vi.restoreAllMocks();
     Object.defineProperty(window, 'location', {
       value: originalLocation,
       writable: true,
@@ -54,7 +52,7 @@ describe('PrintReceipt', () => {
   });
 
   it('shows loading state and disables button during download', async () => {
-    const user = userEvent.setup({ delay: null });
+    const user = userEvent.setup();
     render(<PrintReceipt billUuid={TEST_BILL_UUID} />);
 
     const button = screen.getByRole('button', { name: /print receipt/i });
@@ -66,16 +64,12 @@ describe('PrintReceipt', () => {
   });
 
   it('initiates download when button is clicked', async () => {
-    const user = userEvent.setup({ delay: null });
+    const user = userEvent.setup();
     render(<PrintReceipt billUuid={TEST_BILL_UUID} />);
 
     const button = screen.getByRole('button', { name: /print receipt/i });
     await user.click(button);
 
-    act(() => {
-      jest.advanceTimersByTime(1000);
-    });
-
     await waitFor(() => {
       expect(mockLink.click).toHaveBeenCalled();
     });
@@ -85,7 +79,7 @@ describe('PrintReceipt', () => {
   });
 
   it('re-enables button after download completes', async () => {
-    const user = userEvent.setup({ delay: null });
+    const user = userEvent.setup();
     render(<PrintReceipt billUuid={TEST_BILL_UUID} />);
 
     const button = screen.getByRole('button', { name: /print receipt/i });
@@ -93,10 +87,6 @@ describe('PrintReceipt', () => {
 
     expect(button).toBeDisabled();
 
-    act(() => {
-      jest.advanceTimersByTime(1000);
-    });
-
     await waitFor(() => {
       expect(button).toBeEnabled();
     });
@@ -106,7 +96,7 @@ describe('PrintReceipt', () => {
   });
 
   it('prevents multiple simultaneous downloads', async () => {
-    const user = userEvent.setup({ delay: null });
+    const user = userEvent.setup();
     render(<PrintReceipt billUuid={TEST_BILL_UUID} />);
 
     const button = screen.getByRole('button', { name: /print receipt/i });
@@ -117,10 +107,6 @@ describe('PrintReceipt', () => {
 
     expect(button).toBeDisabled();
 
-    act(() => {
-      jest.advanceTimersByTime(1000);
-    });
-
     await waitFor(() => {
       expect(button).toBeEnabled();
     });
@@ -134,16 +120,12 @@ describe('PrintReceipt', () => {
   });
 
   it('handles empty bill UUID', async () => {
-    const user = userEvent.setup({ delay: null });
+    const user = userEvent.setup();
     render(<PrintReceipt billUuid="" />);
 
     const button = screen.getByRole('button', { name: /print receipt/i });
     await user.click(button);
 
-    act(() => {
-      jest.advanceTimersByTime(1000);
-    });
-
     await waitFor(() => {
       expect(mockLink.click).toHaveBeenCalled();
     });

package/src/invoice/printable-invoice/printable-footer.test.tsx

@@ -1,16 +1,17 @@
 import React from 'react';
+import { describe, expect, it, vi } from 'vitest';
 import { screen, render } from '@testing-library/react';
 import { useDefaultFacility } from '../../billing.resource';
 import PrintableFooter from './printable-footer.component';
 
-const mockUseDefaultFacility = jest.mocked<typeof useDefaultFacility>(useDefaultFacility);
+const mockUseDefaultFacility = vi.mocked<typeof useDefaultFacility>(useDefaultFacility);
 
-jest.mock('../../billing.resource', () => ({
-  useDefaultFacility: jest.fn(),
+vi.mock('../../billing.resource', () => ({
+  useDefaultFacility: vi.fn(),
 }));
 
 describe('PrintableFooter', () => {
-  test('should render PrintableFooter component', () => {
+  it('should render PrintableFooter component', () => {
     mockUseDefaultFacility.mockReturnValue({
       data: { display: 'MTRH', uuid: 'mtrh-uuid', links: [] },
     });

package/src/invoice/printable-invoice/printable-invoice-header.component.tsx

@@ -17,13 +17,13 @@ const PrintableInvoiceHeader: React.FC<PrintableInvoiceHeaderProps> = ({ patient
   const { logo, country, defaultCurrency } = useConfig<BillingConfig>();
 
   const invoiceDetails = {
-    [t('invoiceNumber', 'Invoice #')]: bill?.receiptNumber,
+    [t('invoiceNo', 'Invoice #')]: bill?.receiptNumber,
     [t('invoiceDate', 'Invoice date')]: bill?.dateCreated
       ? formatDate(parseDate(bill.dateCreated), { mode: 'wide', noToday: true, time: false })
       : '--',
-    [t('totalAmount', 'Total Amount')]: `${defaultCurrency} ${bill?.totalAmount}`,
+    [t('totalAmount', 'Total amount')]: `${defaultCurrency} ${bill?.totalAmount}`,
     [t('totalPaid', 'Total paid')]: `${defaultCurrency} ${bill?.tenderedAmount}`,
-    [t('amountBalance', 'Amount balance')]: `${defaultCurrency} ${bill?.totalAmount - bill?.tenderedAmount}`,
+    [t('amountBalance', 'Amount balance')]: `${defaultCurrency} ${bill?.netAmount - bill?.tenderedAmount}`,
     ...(bill?.status && { [t('invoiceStatus', 'Invoice status')]: bill.status }),
   };