Adding the click_helper util to BasePage POM and using it in pages/fit_test_kits_page

Author: adrianoaru-nhs

pages/base_page.py

@@ -103,3 +103,15 @@ def go_to_screening_practitioner_appointments_page(self)->None:
 
     def go_to_screening_subject_search_page(self)->None:
         click(self.page, self.screening_subject_search_page)
+
+    def click(self, locator) -> None:
+        try:
+            self.page.wait_for_load_state('load')
+            self.page.wait_for_load_state('domcontentloaded')
+            locator.wait_for("attached")
+            locator.wait_for("visible")
+            locator.click()
+
+        except Exception as locatorClickError:
+            print(f"Failed to click element with error: {locatorClickError}, trying again...")
+            locator.click()

pages/fit_test_kits_page.py

@@ -1,9 +1,9 @@
 from playwright.sync_api import Page
-from utils.click_helper import click
+from pages.base_page import BasePage
 
-
-class FITTestKits:
+class FITTestKits(BasePage):
     def __init__(self, page: Page):
+        super().__init__(page)
         self.page = page
         # Downloads Page
         self.fit_rollout_summary_page = self.page.get_by_role("link", name="FIT Rollout Summary")
@@ -19,34 +19,34 @@ def __init__(self, page: Page):
         self.fit_device_id = self.page.get_by_role("textbox", name="FIT Device ID")
 
     def go_to_fit_rollout_summary_page(self)->None:
-        click(self.page, self.fit_rollout_summary_page)
+        self.click(self.fit_rollout_summary_page)
 
     def go_to_log_devices_page(self)->None:
-        click(self.page, self.log_devices_page)
+        self.click(self.log_devices_page)
 
     def go_to_view_fit_kit_result(self)->None:
-        click(self.page, self.view_fit_kit_result_page)
+        self.click(self.view_fit_kit_result_page)
 
     def go_to_kit_service_management(self)->None:
-        click(self.page, self.kit_service_management_page)
+        self.click(self.kit_service_management_page)
 
     def go_to_kit_result_audit(self)->None:
-        click(self.page, self.kit_result_audit_page)
+        self.click(self.kit_result_audit_page)
 
     def go_to_view_algorithm(self)->None:
-        click(self.page, self.view_algorithm_page)
+        self.click(self.view_algorithm_page)
 
     def go_to_view_screening_centre_fit(self)->None:
-        click(self.page, self.view_screening_centre_fit_page)
+        self.click(self.view_screening_centre_fit_page)
 
     def go_to_screening_incidents_list(self)->None:
-        click(self.page, self.screening_incidents_list_page)
+        self.click(self.screening_incidents_list_page)
 
     def go_to_manage_qc_products(self)->None:
-        click(self.page, self.manage_qc_products_page)
+        self.click(self.manage_qc_products_page)
 
     def go_to_maintain_analysers(self)->None:
-        click(self.page, self.maintain_analysers_page)
+        self.click(self.maintain_analysers_page)
 
     def go_to_fit_device_id(self)->None:
-        click(self.page, self.fit_device_id)
+        self.click(self.fit_device_id)