Editting all of the POMs to inherit the click method from the BasePage POM

Author: adrianoaru-nhs

pages/base_page.py

@@ -1,5 +1,4 @@
 from playwright.sync_api import Page, expect
-from utils.click_helper import click
 import logging
 
 class BasePage:
@@ -35,74 +34,74 @@ def __init__(self, page: Page):
     def click_main_menu_link(self)->None:
         for _ in range(3):  # Try up to 3 times
             if self.main_menu_link.is_visible():
-                click(self.page, self.main_menu_link)
+                self.click(self.main_menu_link)
                 return  # Exit if successful
 
     def click_log_out_link(self)->None:
-        click(self.page, self.log_out_link)
+        self.click(self.log_out_link)
 
     def click_sub_menu_link(self)->None:
-        click(self.page, self.sub_menu_link)
+        self.click(self.sub_menu_link)
 
     def click_hide_sub_menu_link(self)->None:
-        click(self.page, self.hide_sub_menu_link)
+        self.click(self.hide_sub_menu_link)
 
     def click_select_org_link(self)->None:
-        click(self.page, self.select_org_link)
+        self.click(self.select_org_link)
 
     def click_back_button(self)->None:
-        click(self.page, self.back_button)
+        self.click(self.back_button)
 
     def click_release_notes_link(self)->None:
-        click(self.page, self.release_notes_link)
+        self.click(self.release_notes_link)
 
     def click_refresh_alerts_link(self)->None:
-        click(self.page, self.refresh_alerts_link)
+        self.click(self.refresh_alerts_link)
 
     def click_user_guide_link(self)->None:
-        click(self.page, self.user_guide_link)
+        self.click(self.user_guide_link)
 
     def click_help_link(self)->None:
-        click(self.page, self.help_link)
+        self.click(self.help_link)
 
     def bowel_cancer_screening_system_header_is_displayed(self)->None:
         expect(self.bowel_cancer_screening_system_header).to_contain_text("Bowel Cancer Screening System")
 
     def go_to_contacts_list_page(self)->None:
-        click(self.page, self.contacts_list_page)
+        self.click(self.contacts_list_page)
 
     def go_to_bowel_scope_page(self)->None:
-        click(self.page, self.bowel_scope_page)
+        self.click(self.bowel_scope_page)
 
     def go_to_call_and_recall_page(self)->None:
-        click(self.page, self.call_and_recall_page)
+        self.click(self.call_and_recall_page)
 
     def go_to_communications_production_page(self)->None:
-        click(self.page, self.communications_production_page)
+        self.click(self.communications_production_page)
 
     def go_to_download_page(self)->None:
-        click(self.page, self.download_page)
+        self.click(self.download_page)
 
     def go_to_fit_test_kits_page(self)->None:
-        click(self.page, self.fit_test_kits_page)
+        self.click(self.fit_test_kits_page)
 
     def go_to_gfob_test_kits_page(self)->None:
-        click(self.page, self.gfob_test_kits_page)
+        self.click(self.gfob_test_kits_page)
 
     def go_to_lynch_surveillance_page(self)->None:
-        click(self.page, self.lynch_surveillance_page)
+        self.click(self.lynch_surveillance_page)
 
     def go_to_organisations_page(self)->None:
-        click(self.page, self.organisations_page)
+        self.click(self.organisations_page)
 
     def go_to_reports_page(self)->None:
-        click(self.page, self.reports_page)
+        self.click(self.reports_page)
 
     def go_to_screening_practitioner_appointments_page(self)->None:
-        click(self.page, self.screening_practitioner_appointments_page)
+        self.click(self.screening_practitioner_appointments_page)
 
     def go_to_screening_subject_search_page(self)->None:
-        click(self.page, self.screening_subject_search_page)
+        self.click(self.screening_subject_search_page)
 
     def click(self, locator) -> None:
         """

pages/batch_list_page.py

@@ -1,9 +1,9 @@
 from playwright.sync_api import Page, expect
-from utils.click_helper import click
+from pages.base_page import BasePage
 
-
-class BatchList:
+class BatchList(BasePage):
     def __init__(self, page: Page):
+        super().__init__(page)
         self.page = page
         #Batch List - page filters
         self.id_filter = self.page.locator("#batchIdFilter")

pages/call_and_recall_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 CallAndRecall:
+class CallAndRecall(BasePage):
     def __init__(self, page: Page):
+        super().__init__(page)
         self.page = page
         # Call and Recall - page links
         self.planning_and_monitoring_page = self.page.get_by_role("link", name="Planning and Monitoring")
@@ -13,16 +13,16 @@ def __init__(self, page: Page):
         self.age_extension_rollout_plans_page = self.page.get_by_role("link", name="Age Extension Rollout Plans")
 
     def go_to_planning_and_monitoring_page(self)->None:
-        click(self.page, self.planning_and_monitoring_page)
+        self.click(self.planning_and_monitoring_page)
 
     def go_to_generate_invitations_page(self)->None:
-        click(self.page, self.generate_invitations_page)
+        self.click(self.generate_invitations_page)
 
     def go_to_invitation_generation_progress_page(self)->None:
-        click(self.page, self.invitation_generation_progress_page)
+        self.click(self.invitation_generation_progress_page)
 
     def go_to_non_invitation_days_page(self)->None:
-        click(self.page, self.non_invitation_days_page)
+        self.click(self.non_invitation_days_page)
 
     def go_to_age_extension_rollout_plans_page(self)->None:
-        click(self.page, self.age_extension_rollout_plans_page)
+        self.click(self.age_extension_rollout_plans_page)

pages/cognito_login_page.py

@@ -1,9 +1,9 @@
 from playwright.sync_api import Page
+from pages.base_page import BasePage
 
-
-class CognitoLoginPage:
-
+class CognitoLoginPage(BasePage):
     def __init__(self, page: Page):
+        super().__init__(page)
         self.page = page
         self.username = page.get_by_role("textbox", name="Username")
         self.password = page.get_by_role("textbox", name="Password")

pages/communications_production_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 CommunicationsProduction:
+class CommunicationsProduction(BasePage):
     def __init__(self, page: Page):
+        super().__init__(page)
         self.page = page
         # Communication Production - page links
         self.active_batch_list_page = self.page.get_by_role("link", name="Active Batch List")
@@ -13,16 +13,16 @@ def __init__(self, page: Page):
         self.electronic_communication_management_page = self.page.get_by_role("link", name="Electronic Communication Management")
 
     def go_to_active_batch_list_page(self)->None:
-        click(self.page, self.active_batch_list_page)
+        self.click(self.active_batch_list_page)
 
     def go_to_archived_batch_list_page(self)->None:
-        click(self.page, self.archived_batch_list_page)
+        self.click(self.archived_batch_list_page)
 
     def go_to_letter_library_index_page(self)->None:
-        click(self.page, self.letter_library_index_page)
+        self.click(self.letter_library_index_page)
 
     def go_to_letter_signatory_page(self)->None:
-        click(self.page, self.letter_signatory_page)
+        self.click(self.letter_signatory_page)
 
     def go_to_electronic_communication_management_page(self)->None:
-        click(self.page, self.electronic_communication_management_page)
+        self.click(self.electronic_communication_management_page)

pages/contacts_list_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 ContactsListPage:
+class ContactsListPage(BasePage):
     def __init__(self, page: Page):
+        super().__init__(page)
         self.page = page
         # ContactsList Page
         self.view_contacts_page = self.page.get_by_role("link", name="View Contacts")
@@ -12,13 +12,13 @@ def __init__(self, page: Page):
         self.my_preference_settings_page = self.page.get_by_role("link", name="My Preference Settings")
 
     def go_to_view_contacts_page(self)->None:
-        click(self.page, self.view_contacts_page)
+        self.click(self.view_contacts_page)
 
     def go_to_edit_my_contact_details_page(self)->None:
-        click(self.page, self.edit_my_contact_details_page)
+        self.click(self.edit_my_contact_details_page)
 
     def go_to_maintain_contacts_details_page(self)->None:
-        click(self.page, self.maintain_contacts_page)
+        self.click(self.maintain_contacts_page)
 
     def go_to_my_preference_settings_page(self)->None:
-        click(self.page, self.my_preference_settings_page)
+        self.click(self.my_preference_settings_page)

pages/create_a_plan_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 CreateAPlan:
+class CreateAPlan(BasePage):
     def __init__(self, page: Page):
+        super().__init__(page)
         self.page = page
         # Call and Recall - page links
         self.set_all_button = self.page.get_by_role("link", name="Set all")
@@ -16,7 +16,7 @@ def __init__(self, page: Page):
         self.save_note_button = self.page.locator("#saveNote").get_by_role("button", name="Save")
 
     def click_set_all_button(self)->None:
-        click(self.page, self.set_all_button)
+        self.click(self.set_all_button)
 
     def fill_daily_invitation_rate_field(self, value: str)->None:
         self.daily_invitation_rate_field.fill(value)
@@ -25,16 +25,16 @@ def fill_weekly_invitation_rate_field(self, value)->None:
         self.weekly_invitation_rate_field.fill(value)
 
     def click_update_button(self)->None:
-        click(self.page, self.update_button)
+        self.click(self.update_button)
 
     def click_confirm_button(self)->None:
-        click(self.page, self.confirm_button)
+        self.click(self.confirm_button)
 
     def click_save_button(self)->None:
-        click(self.page, self.save_button)
+        self.click(self.save_button)
 
     def fill_note_field(self, value)->None:
         self.note_field.fill(value)
 
     def click_save_note_button(self)->None:
-        click(self.page, self.save_note_button)
+        self.click(self.save_note_button)

pages/downloads_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 DownloadsPage:
+class DownloadsPage(BasePage):
     def __init__(self, page: Page):
+        super().__init__(page)
         self.page = page
         # Downloads Page
         self.individual_download_request_page = self.page.get_by_role("link", name="Individual Download Request")
@@ -12,13 +12,13 @@ def __init__(self, page: Page):
         self.list_of_batch_downloads_page = self.page.get_by_role("cell", name="List of Batch Downloads", exact=True)
 
     def go_to_individual_download_request_page(self)->None:
-        click(self.page, self.individual_download_request_page)
+        self.click(self.individual_download_request_page)
 
     def go_to_list_of_individual_downloads_page(self)->None:
-        click(self.page, self.list_of_individual_downloads_page)
+        self.click(self.list_of_individual_downloads_page)
 
     def go_to_batch_download_request_and_page(self)->None:
-        click(self.page, self.batch_download_request_and_page)
+        self.click(self.batch_download_request_and_page)
 
     def go_to_list_of_batch_downloads_page(self)->None:
-        click(self.page, self.list_of_batch_downloads_page)
+        self.click(self.list_of_batch_downloads_page)

pages/generate_invitations_page.py

@@ -1,11 +1,11 @@
 from playwright.sync_api import Page, expect
+from pages.base_page import BasePage
 import pytest
-from utils.click_helper import click
 import logging
 
-
-class GenerateInvitations:
+class GenerateInvitations(BasePage):
     def __init__(self, page: Page):
+        super().__init__(page)
         self.page = page
         # Generate Invitations - page links
         self.generate_invitations_button = self.page.get_by_role("button", name="Generate Invitations")
@@ -15,10 +15,10 @@ def __init__(self, page: Page):
         self.self_referrals_total = self.page.locator("#col9_total")
 
     def click_generate_invitations_button(self)->None:
-        click(self.page, self.generate_invitations_button)
+        self.click(self.generate_invitations_button)
 
     def click_refresh_button(self)->None:
-        click(self.page, self.refresh_button)
+        self.click(self.refresh_button)
 
     def wait_for_invitation_generation_complete(self) -> bool:
         """

pages/invitations_monitoring_page.py

@@ -1,9 +1,10 @@
 from playwright.sync_api import Page
+from pages.base_page import BasePage
 
-
-class InvitationsMonitoring:
+class InvitationsMonitoring(BasePage):
     def __init__(self, page: Page):
+        super().__init__(page)
         self.page = page
 
     def go_to_invitation_plan_page(self, sc_id)->None:
-        self.page.get_by_role("link", name=sc_id).click()
+        self.click(self.page.get_by_role("link", name=sc_id))