Debugging test_run_fobt_invitations_and_process_s1_batch

Author: Andyg79

pages/call_and_recall/generate_invitations_page.py

@@ -91,7 +91,7 @@ def wait_for_invitation_generation_complete(
         value = value.strip()  # Get text and remove extra spaces
         if int(value) < number_of_invitations:
             pytest.fail(
-                f"There are less than {number_of_invitations} invitations generated"
+                f"Expected {number_of_invitations} invitations generated but got {value}"
             )
 
         self_referrals_total_text = self.self_referrals_total.text_content()

pages/communication_production/batch_list_page.py

@@ -98,6 +98,11 @@ def open_letter_batch(
     ) -> None:
         """
         Finds and opens the batch row based on type, status, level, and description.
+        Args:
+            batch_type (str): The type of the batch (e.g., "Original").
+            status (str): The status of the batch (e.g., "Open").
+            level (str): The level of the batch (e.g., "S1").
+            description (str): The description of the batch (e.g., "Pre-invitation (FIT)").
         """
         # Step 1: Match the row using nested filters, one per column value
         row = (
@@ -109,7 +114,9 @@ def open_letter_batch(
         )
 
         # Step 2: Click the "View" link in the matched row
-        view_link = row.get_by_role("link", name="View")
+        view_link = row.locator(
+            "a"
+        )  # Click the first link in the row identified in step 1
         expect(view_link).to_be_visible()
         view_link.click()
 

tests/bcss_tests.properties

@@ -25,5 +25,5 @@ episode_closed_date=22/09/2020
 # ----------------------------------
 # CALL AND RECALL TEST DATA
 # ----------------------------------
-daily_invitation_rate=28
+daily_invitation_rate=6
 weekly_invitation_rate=130

tests/regression/call_and_recall/test_generate_fobt_invitation_regression.py

@@ -12,7 +12,6 @@
     ManageActiveBatchPage,
 )
 from utils.user_tools import UserTools
-from utils.batch_processing import prepare_and_print_batch
 
 
 @pytest.fixture(scope="function", autouse=True)
@@ -27,10 +26,11 @@ def before_each(page: Page):
     BasePage(page).go_to_call_and_recall_page()
 
 
-@pytest.mark.wip
 @pytest.mark.regression
 @pytest.mark.call_and_recall
-def test_run_fobt_invitations_and_process_s1_batch(page, general_properties: dict):
+def test_run_fobt_invitations_and_process_s1_batch(
+    page: Page, general_properties: dict
+):
     """
     Run FOBT invitations, open the S1 batch, prepare, retrieve+confirm,
     and assert we have a subject at status S9.
@@ -47,6 +47,8 @@ def test_run_fobt_invitations_and_process_s1_batch(page, general_properties: dic
     logging.info("Invitations generated successfully")
 
     # And I view the active batch list
+    BasePage(page).click_main_menu_link()
+    BasePage(page).go_to_communications_production_page()
     CommunicationsProductionPage(page).go_to_active_batch_list_page()
 
     # And I open the "Original" / "Open" / "S1" / "Pre-invitation (FIT)" batch
@@ -57,15 +59,9 @@ def test_run_fobt_invitations_and_process_s1_batch(page, general_properties: dic
         description="Pre-invitation (FIT)",
     )
 
-    # # And I prepare, retrieve & confirm
-    # ManageActiveBatchPage(page).click_prepare_button()
-    # ManageActiveBatchPage(page).click_retrieve_button()
-    # ManageActiveBatchPage(page).click_confirm_button()
-
-    # # Then there is a subject matching the criteria
-    # assert lb.subject_exists(
-    #     latest_kit_class="FIT",
-    #     latest_event_status="S9",
-    #     latest_episode_type="FOBT",
-    #     subject_hub_code="BCS01",
-    # ), "Expected at least one subject at S9 with FIT/FOBT/BCS01"
+    # Then I retrieve and confirm the letters
+    ManageActiveBatchPage(page).click_prepare_button()
+    ManageActiveBatchPage(page).click_retrieve_button()
+    BasePage(page).safe_accept_dialog(
+        page.get_by_role("button", name="Confirm Printed")
+    )  # Click the confirm button and accept the confirmation dialog