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

package/src/invoice/invoice.test.tsx

@@ -1,4 +1,7 @@
 import React from 'react';
+import { beforeEach, describe, expect, it, vi } from 'vitest';
+import { mockBill, mockPatient } from '@mocks/bills.mock';
+import { waitForLoadingToFinish } from '@tools/test-helpers';
 import userEvent from '@testing-library/user-event';
 import { render, screen, waitFor } from '@testing-library/react';
 import { useReactToPrint } from 'react-to-print';
@@ -10,44 +13,49 @@ import {
   usePatient,
 } from '@openmrs/esm-framework';
 import { configSchema, type BillingConfig } from '../config-schema';
-import { BillStatus } from '../types';
-import { mockBill, mockPatient } from 'mocks/bills.mock';
+import {
+  type BillDiscount,
+  BillDiscountStatus,
+  BillDiscountType,
+  BillStatus,
+  RefundStatus,
+  type MappedBill,
+} from '../types';
 import { useBill } from '../billing.resource';
 import { usePaymentModes } from './payments/payment.resource';
-import { waitForLoadingToFinish } from 'tools/test-helpers';
 import Invoice from './invoice.component';
 
-const mockUseConfig = jest.mocked(useConfig<BillingConfig>);
-const mockUseBill = jest.mocked(useBill);
-const mockUsePatient = jest.mocked(usePatient);
-const mockUsePaymentModes = jest.mocked(usePaymentModes);
-const mockUseReactToPrint = jest.mocked(useReactToPrint);
-const mockShowModal = jest.mocked(showModal);
+const mockUseConfig = vi.mocked(useConfig<BillingConfig>);
+const mockUseBill = vi.mocked(useBill);
+const mockUsePatient = vi.mocked(usePatient);
+const mockUsePaymentModes = vi.mocked(usePaymentModes);
+const mockUseReactToPrint = vi.mocked(useReactToPrint);
+const mockShowModal = vi.mocked(showModal);
 
-jest.mock('../helpers/functions', () => ({
-  convertToCurrency: jest.fn((amount) => `USD ${amount}`),
+vi.mock('../helpers/functions', () => ({
+  convertToCurrency: vi.fn((amount) => `USD ${amount}`),
 }));
 
 window.i18next = {
   language: 'en',
 } as any;
 
-jest.mock('./printable-invoice/print-receipt.component', () =>
-  jest.fn(() => <div data-testid="mock-print-receipt">Print Receipt Mock</div>),
-);
+vi.mock('./printable-invoice/print-receipt.component', () => ({
+  default: vi.fn(() => <div data-testid="mock-print-receipt">Print Receipt Mock</div>),
+}));
 
-jest.mock('./printable-invoice/printable-invoice.component', () =>
-  jest.fn(() => <div data-testid="mock-printable-invoice">Printable Invoice Mock</div>),
-);
+vi.mock('./printable-invoice/printable-invoice.component', () => ({
+  default: vi.fn(() => <div data-testid="mock-printable-invoice">Printable Invoice Mock</div>),
+}));
 
-jest.mock('./payments/payment.resource', () => ({
-  usePaymentModes: jest.fn(),
-  updateBillVisitAttribute: jest.fn(),
+vi.mock('./payments/payment.resource', () => ({
+  usePaymentModes: vi.fn(),
+  updateBillVisitAttribute: vi.fn(),
 }));
 
-jest.mock('../billing.resource', () => ({
-  useBill: jest.fn(),
-  useDefaultFacility: jest.fn().mockReturnValue({
+vi.mock('../billing.resource', () => ({
+  useBill: vi.fn(),
+  useDefaultFacility: vi.fn().mockReturnValue({
     data: {
       uuid: '54065383-b4d4-42d2-af4d-d250a1fd2590',
       display: 'MTRH',
@@ -55,23 +63,32 @@ jest.mock('../billing.resource', () => ({
   }),
 }));
 
-jest.mock('react-router-dom', () => ({
-  useParams: jest.fn().mockReturnValue({
+vi.mock('../discounts/discounts-table.component', () => ({
+  default: vi.fn(() => null),
+}));
+
+vi.mock('../refunds/refunds-table.component', () => ({
+  default: vi.fn(() => null),
+}));
+
+vi.mock('react-router-dom', () => ({
+  useParams: vi.fn().mockReturnValue({
     patientUuid: 'patientUuid',
     billUuid: 'billUuid',
   }),
 }));
 
-jest.mock('react-to-print', () => ({
-  useReactToPrint: jest.fn(),
+vi.mock('react-to-print', () => ({
+  useReactToPrint: vi.fn(),
 }));
 
 describe('Invoice', () => {
-  const defaultBillData = {
+  const defaultBillData: MappedBill = {
     ...mockBill,
     uuid: 'test-uuid',
     status: BillStatus.PENDING,
     totalAmount: 1000,
+    netAmount: 1000,
     tenderedAmount: 0,
     receiptNumber: 'RCPT-001',
     dateCreated: '2024-01-01',
@@ -81,7 +98,7 @@ describe('Invoice', () => {
         item: 'Test Service',
         quantity: 1,
         price: 1000,
-        paymentStatus: 'PENDING',
+        status: 'PENDING',
         billableService: 'Test Service',
         display: '',
         voided: false,
@@ -100,7 +117,7 @@ describe('Invoice', () => {
       isLoading: false,
       error: null,
       isValidating: false,
-      mutate: jest.fn(),
+      mutate: vi.fn(),
     });
 
     mockUsePatient.mockReturnValue({
@@ -117,22 +134,38 @@ describe('Invoice', () => {
       ],
       isLoading: false,
       error: null,
-      mutate: jest.fn(),
+      mutate: vi.fn(),
     });
 
     mockUseConfig.mockReturnValue({ ...getDefaultsFromConfigSchema(configSchema), defaultCurrency: 'USD' });
 
-    const printHandler = jest.fn();
+    const printHandler = vi.fn();
     mockUseReactToPrint.mockReturnValue(printHandler);
   });
 
+  const makeDiscount = (overrides: Partial<BillDiscount> = {}): BillDiscount => ({
+    uuid: 'discount-1',
+    billUuid: 'test-uuid',
+    lineItemUuid: null,
+    discountType: BillDiscountType.PERCENTAGE,
+    discountValue: 10,
+    discountAmount: 100,
+    justification: 'goodwill',
+    initiator: { uuid: 'u1', display: 'cashier' },
+    approver: null,
+    dateCreated: '2024-01-01',
+    status: BillDiscountStatus.APPROVED,
+    voided: false,
+    ...overrides,
+  });
+
   it('should render loading state when bill is loading', () => {
     mockUseBill.mockReturnValue({
       bill: null,
       isLoading: true,
       error: null,
       isValidating: false,
-      mutate: jest.fn(),
+      mutate: vi.fn(),
     });
 
     render(<Invoice />);
@@ -157,7 +190,7 @@ describe('Invoice', () => {
       isLoading: false,
       error: new Error('Failed to load bill'),
       isValidating: false,
-      mutate: jest.fn(),
+      mutate: vi.fn(),
     });
 
     render(<Invoice />);
@@ -171,24 +204,12 @@ describe('Invoice', () => {
     expect(screen.getAllByText(/total amount/i).length).toBeGreaterThan(0);
     expect(screen.getAllByText(/amount tendered/i).length).toBeGreaterThan(0);
     expect(screen.getByRole('heading', { name: /invoice number/i })).toBeInTheDocument();
-    expect(screen.getByText(/date and time/i)).toBeInTheDocument();
+    expect(screen.getByText(/date bill created/i)).toBeInTheDocument();
     expect(screen.getByText(/invoice status/i)).toBeInTheDocument();
     expect(screen.getAllByText('RCPT-001').length).toBeGreaterThan(0);
     expect(screen.getAllByText('PENDING').length).toBeGreaterThan(0);
-  });
-
-  it('should render invoice table with line items', async () => {
-    render(<Invoice />);
-    await waitForLoadingToFinish();
-
     expect(screen.getByText(/line items/i)).toBeInTheDocument();
     expect(screen.getByText('Test Service')).toBeInTheDocument();
-  });
-
-  it('should render payments section', async () => {
-    render(<Invoice />);
-    await waitForLoadingToFinish();
-
     expect(screen.getByText(/payments/i)).toBeInTheDocument();
   });
 
@@ -202,7 +223,7 @@ describe('Invoice', () => {
       isLoading: false,
       error: null,
       isValidating: false,
-      mutate: jest.fn(),
+      mutate: vi.fn(),
     });
 
     render(<Invoice />);
@@ -221,7 +242,7 @@ describe('Invoice', () => {
       isLoading: false,
       error: null,
       isValidating: false,
-      mutate: jest.fn(),
+      mutate: vi.fn(),
     });
 
     render(<Invoice />);
@@ -238,7 +259,7 @@ describe('Invoice', () => {
   });
 
   it('should handle print button click', async () => {
-    const handlePrintMock = jest.fn();
+    const handlePrintMock = vi.fn();
     const user = userEvent.setup();
     mockUseReactToPrint.mockReturnValue(handlePrintMock);
 
@@ -269,7 +290,7 @@ describe('Invoice', () => {
   });
 
   it('should search and filter line items in the table', async () => {
-    const billWithMultipleItems = {
+    const billWithMultipleItems: MappedBill = {
       ...defaultBillData,
       lineItems: [
         {
@@ -277,7 +298,7 @@ describe('Invoice', () => {
           item: 'Lab Test',
           quantity: 1,
           price: 500,
-          paymentStatus: 'PENDING',
+          status: 'PENDING',
           billableService: 'Lab Test',
           display: '',
           voided: false,
@@ -292,7 +313,7 @@ describe('Invoice', () => {
           item: 'X-Ray',
           quantity: 1,
           price: 500,
-          paymentStatus: 'PENDING',
+          status: 'PENDING',
           billableService: 'X-Ray',
           display: '',
           voided: false,
@@ -310,7 +331,7 @@ describe('Invoice', () => {
       isLoading: false,
       error: null,
       isValidating: false,
-      mutate: jest.fn(),
+      mutate: vi.fn(),
     });
 
     const user = userEvent.setup();
@@ -330,7 +351,7 @@ describe('Invoice', () => {
   });
 
   it('should handle bill data updates via mutate', async () => {
-    const mockMutate = jest.fn();
+    const mockMutate = vi.fn();
     mockUseBill.mockReturnValue({
       bill: defaultBillData,
       isLoading: false,
@@ -378,7 +399,7 @@ describe('Invoice', () => {
       isLoading: true,
       error: null,
       isValidating: false,
-      mutate: jest.fn(),
+      mutate: vi.fn(),
     });
 
     render(<Invoice />);
@@ -392,7 +413,7 @@ describe('Invoice', () => {
       isLoading: false,
       error: null,
       isValidating: false,
-      mutate: jest.fn(),
+      mutate: vi.fn(),
     });
 
     mockUsePatient.mockReturnValue({
@@ -444,7 +465,7 @@ describe('Invoice', () => {
   });
 
   it('should pass mutate function to Payments component', async () => {
-    const mockMutate = jest.fn();
+    const mockMutate = vi.fn();
     mockUseBill.mockReturnValue({
       bill: defaultBillData,
       isLoading: false,
@@ -471,7 +492,7 @@ describe('Invoice', () => {
       isLoading: false,
       error: null,
       isValidating: false,
-      mutate: jest.fn(),
+      mutate: vi.fn(),
     });
 
     render(<Invoice />);
@@ -499,7 +520,7 @@ describe('Invoice', () => {
       isLoading: false,
       error: null,
       isValidating: false,
-      mutate: jest.fn(),
+      mutate: vi.fn(),
     });
 
     render(<Invoice />);
@@ -525,7 +546,7 @@ describe('Invoice', () => {
       isLoading: false,
       error: null,
       isValidating: false,
-      mutate: jest.fn(),
+      mutate: vi.fn(),
     });
 
     render(<Invoice />);
@@ -547,7 +568,7 @@ describe('Invoice', () => {
       isLoading: false,
       error: null,
       isValidating: false,
-      mutate: jest.fn(),
+      mutate: vi.fn(),
     });
 
     render(<Invoice />);
@@ -562,7 +583,7 @@ describe('Invoice', () => {
       isLoading: false,
       error: null,
       isValidating: false,
-      mutate: jest.fn(),
+      mutate: vi.fn(),
     });
 
     render(<Invoice />);
@@ -572,7 +593,7 @@ describe('Invoice', () => {
   });
 
   it('should open finalize confirmation modal when "Finalize bill" button is clicked', async () => {
-    const mockMutate = jest.fn();
+    const mockMutate = vi.fn();
     const user = userEvent.setup();
 
     mockUseBill.mockReturnValue({
@@ -595,9 +616,288 @@ describe('Invoice', () => {
     });
   });
 
+  describe('discount entry points', () => {
+    it('renders the "Request discount" button when bill is eligible', async () => {
+      render(<Invoice />);
+      expect(await screen.findByRole('button', { name: /request discount/i })).toBeInTheDocument();
+    });
+
+    it('opens the request-discount modal on click', async () => {
+      const user = userEvent.setup();
+      render(<Invoice />);
+      await user.click(await screen.findByRole('button', { name: /request discount/i }));
+      expect(mockShowModal).toHaveBeenCalledWith('request-discount-modal', expect.any(Object));
+    });
+
+    it('shows Total amount / Discount / Net amount trio when an approved discount exists', async () => {
+      mockUseBill.mockReturnValue({
+        bill: {
+          ...defaultBillData,
+          totalAmount: 5000,
+          netAmount: 4500,
+          tenderedAmount: 500,
+          discounts: [makeDiscount({ status: BillDiscountStatus.APPROVED, discountAmount: 500 })],
+        },
+        isLoading: false,
+        error: null,
+        isValidating: false,
+        mutate: vi.fn(),
+      });
+      render(<Invoice />);
+      expect(await screen.findByRole('heading', { name: /^net amount$/i })).toBeInTheDocument();
+      expect(screen.getByRole('heading', { name: /^total amount$/i })).toBeInTheDocument();
+      expect(screen.getByRole('heading', { name: /^discount$/i })).toBeInTheDocument();
+      expect(screen.getAllByText(/^- USD 500$/).length).toBeGreaterThan(0);
+      // Amount due must be netAmount - tendered (4000), not totalAmount - tendered (4500).
+      expect(screen.getAllByText('USD 4000').length).toBeGreaterThan(0);
+    });
+
+    it('hides the Discount / Net amount trio when no approved discount exists, even if totalAmount differs from netAmount', async () => {
+      mockUseBill.mockReturnValue({
+        bill: {
+          ...defaultBillData,
+          totalAmount: 5000,
+          netAmount: 4500,
+          discounts: [makeDiscount({ status: BillDiscountStatus.PENDING })],
+        },
+        isLoading: false,
+        error: null,
+        isValidating: false,
+        mutate: vi.fn(),
+      });
+      render(<Invoice />);
+      await screen.findByRole('heading', { name: /^total amount$/i });
+      expect(screen.queryByRole('heading', { name: /^net amount$/i })).not.toBeInTheDocument();
+      expect(screen.queryByRole('heading', { name: /^discount$/i })).not.toBeInTheDocument();
+    });
+
+    it('passes an onMutate to the modal that revalidates the bill SWR cache', async () => {
+      const mockMutate = vi.fn();
+      mockUseBill.mockReturnValue({
+        bill: defaultBillData,
+        isLoading: false,
+        error: null,
+        isValidating: false,
+        mutate: mockMutate,
+      });
+      const user = userEvent.setup();
+      render(<Invoice />);
+      await user.click(await screen.findByRole('button', { name: /request discount/i }));
+      const [, props] = mockShowModal.mock.calls.at(-1)!;
+      (props as { onMutate: () => void }).onMutate();
+      expect(mockMutate).toHaveBeenCalled();
+    });
+  });
+
+  describe('discount eligibility — bill status', () => {
+    it.each([
+      [BillStatus.PAID, 1000],
+      [BillStatus.ADJUSTED, 0],
+    ])('hides the bill-level "Request discount" button for %s bills', (status, tendered) => {
+      mockUseBill.mockReturnValue({
+        bill: { ...defaultBillData, status, tenderedAmount: tendered },
+        isLoading: false,
+        error: null,
+        isValidating: false,
+        mutate: vi.fn(),
+      });
+      render(<Invoice />);
+      expect(screen.queryByRole('button', { name: /request discount/i })).not.toBeInTheDocument();
+    });
+
+    it.each([
+      [BillStatus.PAID, 1000],
+      [BillStatus.ADJUSTED, 0],
+    ])('hides the per-line-item "Request discount" action for %s bills', async (status, tendered) => {
+      mockUseBill.mockReturnValue({
+        bill: { ...defaultBillData, status, tenderedAmount: tendered },
+        isLoading: false,
+        error: null,
+        isValidating: false,
+        mutate: vi.fn(),
+      });
+      const user = userEvent.setup();
+      render(<Invoice />);
+      await user.click(screen.getByTestId('action-menu-item-1'));
+      expect(screen.queryByTestId('request-discount-button-item-1')).not.toBeInTheDocument();
+    });
+  });
+
+  describe('discount eligibility — line already discounted', () => {
+    it.each([BillDiscountStatus.PENDING, BillDiscountStatus.APPROVED, BillDiscountStatus.REJECTED])(
+      'hides the per-line-item "Request discount" action when an existing %s discount targets the line',
+      async (discountStatus) => {
+        mockUseBill.mockReturnValue({
+          bill: { ...defaultBillData, discounts: [makeDiscount({ lineItemUuid: 'item-1', status: discountStatus })] },
+          isLoading: false,
+          error: null,
+          isValidating: false,
+          mutate: vi.fn(),
+        });
+        const user = userEvent.setup();
+        render(<Invoice />);
+        await user.click(screen.getByTestId('action-menu-item-1'));
+        expect(screen.queryByTestId('request-discount-button-item-1')).not.toBeInTheDocument();
+      },
+    );
+  });
+
+  describe('discount eligibility — cross-blocking between bill-level and line-level', () => {
+    it.each([BillDiscountStatus.PENDING, BillDiscountStatus.APPROVED, BillDiscountStatus.REJECTED])(
+      'disables the bill-level "Request discount" button when a %s line-item discount exists',
+      (discountStatus) => {
+        mockUseBill.mockReturnValue({
+          bill: { ...defaultBillData, discounts: [makeDiscount({ lineItemUuid: 'item-1', status: discountStatus })] },
+          isLoading: false,
+          error: null,
+          isValidating: false,
+          mutate: vi.fn(),
+        });
+        render(<Invoice />);
+        expect(screen.getByRole('button', { name: /request discount/i })).toBeDisabled();
+      },
+    );
+
+    it.each([BillDiscountStatus.PENDING, BillDiscountStatus.APPROVED, BillDiscountStatus.REJECTED])(
+      'hides the per-line-item "Request discount" action when a %s bill-level discount exists',
+      async (discountStatus) => {
+        mockUseBill.mockReturnValue({
+          bill: { ...defaultBillData, discounts: [makeDiscount({ lineItemUuid: null, status: discountStatus })] },
+          isLoading: false,
+          error: null,
+          isValidating: false,
+          mutate: vi.fn(),
+        });
+        const user = userEvent.setup();
+        render(<Invoice />);
+        await user.click(screen.getByTestId('action-menu-item-1'));
+        expect(screen.queryByTestId('request-discount-button-item-1')).not.toBeInTheDocument();
+      },
+    );
+  });
+
+  describe('refund entry points', () => {
+    const paidBill: MappedBill = {
+      ...defaultBillData,
+      status: BillStatus.PAID,
+      totalAmount: 1000,
+      netAmount: 1000,
+      tenderedAmount: 1000,
+    };
+
+    beforeEach(() => {
+      mockUseBill.mockReturnValue({
+        bill: paidBill,
+        isLoading: false,
+        error: null,
+        isValidating: false,
+        mutate: vi.fn(),
+      });
+    });
+
+    it('shows "Request refund" button for PAID bills with no active refund', async () => {
+      render(<Invoice />);
+      expect(await screen.findByRole('button', { name: /request refund/i })).toBeInTheDocument();
+    });
+
+    it('hides "Request refund" button when bill is in REFUND_REQUESTED status', async () => {
+      mockUseBill.mockReturnValue({
+        bill: { ...paidBill, status: BillStatus.REFUND_REQUESTED },
+        isLoading: false,
+        error: null,
+        isValidating: false,
+        mutate: vi.fn(),
+      });
+      render(<Invoice />);
+      await waitForLoadingToFinish();
+      expect(screen.queryByRole('button', { name: /request refund/i })).not.toBeInTheDocument();
+    });
+
+    const billWithActiveLineRefund: MappedBill = {
+      ...paidBill,
+      refunds: [
+        {
+          uuid: 'r-active',
+          billUuid: 'test-uuid',
+          lineItemUuid: 'item-1',
+          refundAmount: 200,
+          reason: 'duplicate',
+          initiator: { uuid: 'u1', display: 'cashier' },
+          approver: null,
+          completer: null,
+          dateApproved: null,
+          dateCompleted: null,
+          dateCreated: '2026-06-01T00:00:00.000+0000',
+          status: RefundStatus.REQUESTED,
+          voided: false,
+        },
+      ],
+    };
+
+    it('disables "Request refund" button when an active line-item refund is in progress', async () => {
+      mockUseBill.mockReturnValue({
+        bill: billWithActiveLineRefund,
+        isLoading: false,
+        error: null,
+        isValidating: false,
+        mutate: vi.fn(),
+      });
+      render(<Invoice />);
+      await waitForLoadingToFinish();
+      expect(
+        screen.getByRole('button', { name: /a refund is already in progress for one or more line items/i }),
+      ).toBeDisabled();
+    });
+
+    it('shows a tooltip explaining why "Request refund" is disabled when a line-item refund is in progress', async () => {
+      mockUseBill.mockReturnValue({
+        bill: billWithActiveLineRefund,
+        isLoading: false,
+        error: null,
+        isValidating: false,
+        mutate: vi.fn(),
+      });
+      render(<Invoice />);
+      await waitForLoadingToFinish();
+      expect(screen.getByText(/a refund is already in progress for one or more line items/i)).toBeInTheDocument();
+    });
+
+    it('passes remainingRefundable that deducts COMPLETED refunds when opening the request-refund modal', async () => {
+      const completedRefund = {
+        uuid: 'r-done',
+        billUuid: 'test-uuid',
+        lineItemUuid: null,
+        refundAmount: 300,
+        reason: 'overcharged',
+        initiator: { uuid: 'u1', display: 'cashier' },
+        approver: { uuid: 'u2', display: 'admin' },
+        completer: { uuid: 'u3', display: 'cashier2' },
+        dateApproved: '2026-05-21T00:00:00.000+0000',
+        dateCompleted: '2026-05-21T00:00:00.000+0000',
+        dateCreated: '2026-05-21T00:00:00.000+0000',
+        status: RefundStatus.COMPLETED,
+        voided: false,
+      };
+      mockUseBill.mockReturnValue({
+        bill: { ...paidBill, refunds: [completedRefund] },
+        isLoading: false,
+        error: null,
+        isValidating: false,
+        mutate: vi.fn(),
+      });
+      const user = userEvent.setup();
+      render(<Invoice />);
+      await user.click(await screen.findByRole('button', { name: /request refund/i }));
+      expect(mockShowModal).toHaveBeenCalledWith(
+        'request-refund-modal',
+        expect.objectContaining({ remainingRefundable: 700 }),
+      );
+    });
+  });
+
   it('should launch workspace with billUuid when "Add items to bill" is clicked', async () => {
-    const mockMutate = jest.fn();
-    const mockLaunchWorkspace2 = jest.mocked(launchWorkspace2);
+    const mockMutate = vi.fn();
+    const mockLaunchWorkspace2 = vi.mocked(launchWorkspace2);
 
     mockUseBill.mockReturnValue({
       bill: defaultBillData,

package/src/invoice/line-item-action-menu.component.tsx

@@ -1,4 +1,5 @@
 import React, { useCallback } from 'react';
+import { useTranslation } from 'react-i18next';
 import { Layer, OverflowMenu, OverflowMenuItem } from '@carbon/react';
 import { getCoreTranslation, isDesktop, showModal, useLayoutType } from '@openmrs/esm-framework';
 import { BillStatus, type LineItem, type MappedBill } from '../types';
@@ -8,9 +9,22 @@ type LineItemActionMenuProps = {
   bill: MappedBill;
   item: LineItem;
   onMutate?: () => void;
+  showDiscountRequest?: boolean;
+  onDiscountRequest?: () => void;
+  showRefundRequest?: boolean;
+  onRefundRequest?: () => void;
 };
 
-const LineItemActionMenu: React.FC<LineItemActionMenuProps> = ({ bill, item, onMutate }) => {
+const LineItemActionMenu: React.FC<LineItemActionMenuProps> = ({
+  bill,
+  item,
+  onMutate,
+  showDiscountRequest,
+  onDiscountRequest,
+  showRefundRequest,
+  onRefundRequest,
+}) => {
+  const { t } = useTranslation();
   const layout = useLayoutType();
 
   const handleEditLineItem = useCallback(() => {
@@ -54,6 +68,22 @@ const LineItemActionMenu: React.FC<LineItemActionMenuProps> = ({ bill, item, onM
           itemText={getCoreTranslation('delete')}
           onClick={handleDeleteLineItem}
         />
+        {showDiscountRequest && (
+          <OverflowMenuItem
+            className={styles.menuitem}
+            data-testid={`request-discount-button-${item.uuid}`}
+            itemText={t('requestDiscount', 'Request discount')}
+            onClick={onDiscountRequest}
+          />
+        )}
+        {showRefundRequest && (
+          <OverflowMenuItem
+            className={styles.menuitem}
+            data-testid={`request-refund-button-${item.uuid}`}
+            itemText={t('requestRefund', 'Request refund')}
+            onClick={onRefundRequest}
+          />
+        )}
       </OverflowMenu>
     </Layer>
   );