(test): Add e2e to check for bill line status (#615)

Author: NethmiRodrigo

e2e/specs/billing-dashboard.spec.ts

@@ -156,28 +156,20 @@ test.describe('Billing Dashboard workflow', () => {
       expect(extractNumericValue(updatedAmountDue)).toBe(0);
     });
 
-    /**
-     * TODO: Uncomment this after the ticket below has been implemented:
-     * https://openmrs.atlassian.net/browse/O3-5394
-     *
-     * The ticket is about updating the payment status of the line items when the payment is processed.
-     * Once the ticket is implemented, we can uncomment the test step below
-     */
-
-    // await test.step('And the Service status in the Line items table should change to PAID', async () => {
-    //   const lineItems = await invoicePage.getLineItems();
-    //   expect(lineItems.length).toBeGreaterThan(0);
-    //   lineItems.forEach((lineItem) => {
-    //     expect(lineItem.status).toBe('PAID');
-    //   });
-
-    //   // Also verify backend line items have paymentStatus set to PAID
-    //   const billResponse = await api.get(`billing/bill/${billUuid}`);
-    //   const billData = await billResponse.json();
-    //   billData.lineItems.forEach((lineItem: { paymentStatus: string }) => {
-    //     expect(lineItem.paymentStatus).toBe('PAID');
-    //   });
-    // });
+    await test.step('And the Service status in the Line items table should change to PAID', async () => {
+      const lineItems = await invoicePage.getLineItems();
+      expect(lineItems.length).toBeGreaterThan(0);
+      lineItems.forEach((lineItem) => {
+        expect(lineItem.status).toBe('PAID');
+      });
+
+      // Also verify backend line items have paymentStatus set to PAID
+      const billResponse = await api.get(`billing/bill/${billUuid}`);
+      const billData = await billResponse.json();
+      billData.lineItems.forEach((lineItem: { paymentStatus: string }) => {
+        expect(lineItem.paymentStatus).toBe('PAID');
+      });
+    });
 
     await test.step('And the Payments section should display a payment record', async () => {
       const paymentHistory = await paymentPage.getPaymentHistory();

e2e/specs/billing-patient-chart.spec.ts

@@ -505,30 +505,22 @@ test.describe('Billing: Patient Chart workflow', () => {
       await waitForSuccessNotification(page, 'Payment processed successfully');
     });
 
-    /**
-     * TODO: Uncomment this after the ticket below has been implemented:
-     * https://openmrs.atlassian.net/browse/O3-5394
-     *
-     * The ticket is about updating the payment status of the line items when the payment is processed.
-     * Once the ticket is implemented, we can uncomment the test step below
-     */
-
-    // await test.step('Then the line item should be marked as PAID', async () => {
-    //   await page.reload();
-    //   await invoicePage.waitForInvoiceToLoad();
-
-    //   const lineItems = await invoicePage.getLineItems();
-    //   lineItems.forEach((lineItem) => {
-    //     expect(lineItem.status).toBe('PAID');
-    //   });
-
-    //   // Verify backend state
-    //   const billResponse = await api.get(`billing/bill/${billUuid}?v=full`);
-    //   const billData = await billResponse.json();
-    //   billData.lineItems.forEach((lineItem: { paymentStatus: string }) => {
-    //     expect(lineItem.paymentStatus).toBe('PAID');
-    //   });
-    // });
+    await test.step('Then the line item should be marked as PAID', async () => {
+      await page.reload();
+      await invoicePage.waitForInvoiceToLoad();
+
+      const lineItems = await invoicePage.getLineItems();
+      lineItems.forEach((lineItem) => {
+        expect(lineItem.status).toBe('PAID');
+      });
+
+      // Verify backend state
+      const billResponse = await api.get(`billing/bill/${billUuid}?v=full`);
+      const billData = await billResponse.json();
+      billData.lineItems.forEach((lineItem: { paymentStatus: string }) => {
+        expect(lineItem.paymentStatus).toBe('PAID');
+      });
+    });
   });
 
   test('Process payment with multiple payment methods', async ({ page, api, patient }) => {