Formatting code using the Black Formatter extension in VSCode (#10)

<!-- markdownlint-disable-next-line first-line-heading -->
## Description

<!-- Describe your changes in detail. -->

## Context

<!-- Why is this change required? What problem does it solve? -->
Reformatting all the code using the Black Formatter VSCode extension

## Type of changes

<!-- What types of changes does your code introduce? Put an `x` in all
the boxes that apply. -->
This was done to ensure consistency of formatting across the codebase

- [x] Refactoring (non-breaking change)
- [ ] New feature (non-breaking change which adds functionality)
- [ ] Breaking change (fix or feature that would change existing
functionality)
- [ ] Bug fix (non-breaking change which fixes an issue)

## Checklist

<!-- Go over all the following points, and put an `x` in all the boxes
that apply. -->

- [x] I am familiar with the [contributing
guidelines](https://github.com/nhs-england-tools/playwright-python-blueprint/blob/main/CONTRIBUTING.md)
- [x] I have followed the code style of the project
- [ ] I have added tests to cover my changes (where appropriate)
- [ ] I have updated the documentation accordingly
- [ ] This PR is a result of pair or mob programming

---

## Sensitive Information Declaration

To ensure the utmost confidentiality and protect your and others
privacy, we kindly ask you to NOT including [PII (Personal Identifiable
Information) / PID (Personal Identifiable
Data)](https://digital.nhs.uk/data-and-information/keeping-data-safe-and-benefitting-the-public)
or any other sensitive data in this PR (Pull Request) and the codebase
changes. We will remove any PR that do contain any sensitive
information. We really appreciate your cooperation in this matter.

- [x] I confirm that neither PII/PID nor sensitive data are included in
this PR and the codebase changes.

Author: adrianoaru-nhs

pages/age_extension_rollout_plans_page.py

@@ -1,12 +1,15 @@
 from playwright.sync_api import Page, expect
 from pages.base_page import BasePage
 
+
 class AgeExtensionRolloutPlans(BasePage):
     def __init__(self, page: Page):
         super().__init__(page)
         self.page = page
-        #Age Extension Rollout Plans - page locators
+        # Age Extension Rollout Plans - page locators
         self.age_extension_rollout_plans_title = self.page.locator("#page-title")
 
     def verify_age_extension_rollout_plans_title(self) -> None:
-        expect(self.age_extension_rollout_plans_title).to_contain_text("Age Extension Rollout Plans")
+        expect(self.age_extension_rollout_plans_title).to_contain_text(
+            "Age Extension Rollout Plans"
+        )

pages/batch_download_request_and_retrieval_page.py

@@ -1,16 +1,23 @@
 from playwright.sync_api import Page, expect
 from pages.base_page import BasePage
 
+
 class BatchDownloadRequestAndRetrieval(BasePage):
     def __init__(self, page: Page):
         super().__init__(page)
         self.page = page
-        #Batch Download Request And Retrieval - page locators
-        self.batch_download_request_and_retrieval_title = self.page.locator("#ntshPageTitle")
-        self.page_form = self.page.locator("form[name=\"frm\"]")
+        # Batch Download Request And Retrieval - page locators
+        self.batch_download_request_and_retrieval_title = self.page.locator(
+            "#ntshPageTitle"
+        )
+        self.page_form = self.page.locator('form[name="frm"]')
 
     def expect_form_to_have_warning(self) -> None:
-        expect(self.page_form).to_contain_text("Warning - FS Screening data will not be downloaded")
+        expect(self.page_form).to_contain_text(
+            "Warning - FS Screening data will not be downloaded"
+        )
 
     def verify_batch_download_request_and_retrieval_title(self) -> None:
-        expect(self.batch_download_request_and_retrieval_title).to_contain_text("Batch Download Request and Retrieval")
+        expect(self.batch_download_request_and_retrieval_title).to_contain_text(
+            "Batch Download Request and Retrieval"
+        )

pages/batch_list_page.py

@@ -1,11 +1,12 @@
 from playwright.sync_api import Page, expect
 from pages.base_page import BasePage
 
+
 class BatchList(BasePage):
     def __init__(self, page: Page):
         super().__init__(page)
         self.page = page
-        #Batch List - page filters
+        # Batch List - page filters
         self.id_filter = self.page.locator("#batchIdFilter")
         self.type_filter = self.page.locator("#batchTypeFilter")
         self.original_filter = self.page.locator("#originalBatchIdFilter")
@@ -15,51 +16,55 @@ def __init__(self, page: Page):
         self.screening_centre_filter = self.page.locator("#screeningCentreFilter")
         self.count_filter = self.page.locator("#countFilter")
         self.table_data = self.page.locator("td")
-        self.batch_successfully_archived_msg = self.page.locator('text="Batch Successfully Archived and Printed"')
+        self.batch_successfully_archived_msg = self.page.locator(
+            'text="Batch Successfully Archived and Printed"'
+        )
         self.batch_list_page_title = self.page.locator("#page-title")
 
     def verify_batch_list_page_title(self, text) -> None:
         expect(self.batch_list_page_title).to_contain_text(text)
 
-    def verify_table_data(self, value)->None:
+    def verify_table_data(self, value) -> None:
         expect(self.table_data.filter(has_text=value)).to_be_visible()
 
-    def enter_id_filter(self, search_text: str)->None:
+    def enter_id_filter(self, search_text: str) -> None:
         self.id_filter.fill(search_text)
         self.id_filter.press("Enter")
 
-    def enter_type_filter(self, search_text: str)->None:
+    def enter_type_filter(self, search_text: str) -> None:
         self.type_filter.fill(search_text)
         self.type_filter.press("Enter")
 
-    def enter_original_filter(self, search_text: str)->None:
+    def enter_original_filter(self, search_text: str) -> None:
         self.original_filter.fill(search_text)
         self.original_filter.press("Enter")
 
-    def enter_event_code_filter(self, search_text: str)->None:
+    def enter_event_code_filter(self, search_text: str) -> None:
         self.event_code_filter.fill(search_text)
         self.event_code_filter.press("Enter")
 
-    def enter_description_filter(self, search_text: str)->None:
+    def enter_description_filter(self, search_text: str) -> None:
         self.description_filter.fill(search_text)
         self.description_filter.press("Enter")
 
-    def enter_batch_split_by_filter(self, search_text: str)->None:
+    def enter_batch_split_by_filter(self, search_text: str) -> None:
         self.batch_split_by_filter.fill(search_text)
         self.batch_split_by_filter.press("Enter")
 
-    def enter_screening_centre_filter(self, search_text: str)->None:
+    def enter_screening_centre_filter(self, search_text: str) -> None:
         self.screening_centre_filter.fill(search_text)
         self.screening_centre_filter.press("Enter")
 
-    def enter_count_filter(self, search_text: str)->None:
+    def enter_count_filter(self, search_text: str) -> None:
         self.count_filter.fill(search_text)
         self.count_filter.press("Enter")
 
+
 class ActiveBatchList(BatchList):
     def __init__(self, page):
         super().__init__(page)
 
+
 class ArchivedBatchList(BatchList):
     def __init__(self, page):
         super().__init__(page)

pages/bowel_scope_appointments_page.py

@@ -1,11 +1,12 @@
 from playwright.sync_api import Page, expect
 from pages.base_page import BasePage
 
+
 class BowelScopeAppointments(BasePage):
     def __init__(self, page: Page):
         super().__init__(page)
         self.page = page
-        #Bowel Scope Appointments - page locators
+        # Bowel Scope Appointments - page locators
         self.page_title = self.page.locator("#ntshPageTitle")
 
     def verify_page_title(self) -> None:

pages/bowel_scope_page.py

@@ -1,12 +1,15 @@
 from playwright.sync_api import Page, expect, Locator
 from pages.base_page import BasePage
 
+
 class BowelScope(BasePage):
     def __init__(self, page: Page):
         super().__init__(page)
         self.page = page
-        #Bowel Scope - page locators
-        self.view_bowel_scope_appointments_page = self.page.get_by_role("link", name="View Bowel Scope Appointments")
+        # Bowel Scope - page locators
+        self.view_bowel_scope_appointments_page = self.page.get_by_role(
+            "link", name="View Bowel Scope Appointments"
+        )
 
     def go_to_view_bowel_scope_appointments_page(self) -> None:
         self.click(self.view_bowel_scope_appointments_page)

pages/call_and_recall_page.py

@@ -1,28 +1,39 @@
 from playwright.sync_api import Page
 from pages.base_page import BasePage
 
+
 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")
-        self.generate_invitations_page = self.page.get_by_role("link", name="Generate Invitations")
-        self.invitation_generation_progress_page = self.page.get_by_role("link", name="Invitation Generation Progress")
-        self.non_invitation_days_page = self.page.get_by_role("link", name="Non Invitation Days")
-        self.age_extension_rollout_plans_page = self.page.get_by_role("link", name="Age Extension Rollout Plans")
+        self.planning_and_monitoring_page = self.page.get_by_role(
+            "link", name="Planning and Monitoring"
+        )
+        self.generate_invitations_page = self.page.get_by_role(
+            "link", name="Generate Invitations"
+        )
+        self.invitation_generation_progress_page = self.page.get_by_role(
+            "link", name="Invitation Generation Progress"
+        )
+        self.non_invitation_days_page = self.page.get_by_role(
+            "link", name="Non Invitation Days"
+        )
+        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:
+    def go_to_planning_and_monitoring_page(self) -> None:
         self.click(self.planning_and_monitoring_page)
 
-    def go_to_generate_invitations_page(self)->None:
+    def go_to_generate_invitations_page(self) -> None:
         self.click(self.generate_invitations_page)
 
-    def go_to_invitation_generation_progress_page(self)->None:
+    def go_to_invitation_generation_progress_page(self) -> None:
         self.click(self.invitation_generation_progress_page)
 
-    def go_to_non_invitation_days_page(self)->None:
+    def go_to_non_invitation_days_page(self) -> None:
         self.click(self.non_invitation_days_page)
 
-    def go_to_age_extension_rollout_plans_page(self)->None:
+    def go_to_age_extension_rollout_plans_page(self) -> None:
         self.click(self.age_extension_rollout_plans_page)

pages/cognito_login_page.py

@@ -1,6 +1,7 @@
 from playwright.sync_api import Page
 from pages.base_page import BasePage
 
+
 class CognitoLoginPage(BasePage):
     def __init__(self, page: Page):
         super().__init__(page)

pages/colonoscopy_assessment_appointments_page.py

@@ -1,12 +1,15 @@
 from playwright.sync_api import Page, expect
 from pages.base_page import BasePage
 
+
 class ColonoscopyAssessmentAppointments(BasePage):
     def __init__(self, page: Page):
         super().__init__(page)
         self.page = page
-        #Colonoscopy Assessment Appointments - page locators
+        # Colonoscopy Assessment Appointments - page locators
         self.page_header = self.page.locator("#page-title")
 
     def verify_page_header(self) -> None:
-        expect(self.page_header).to_contain_text("Patients that Require Colonoscopy Assessment Appointments")
+        expect(self.page_header).to_contain_text(
+            "Patients that Require Colonoscopy Assessment Appointments"
+        )

pages/communications_production_page.py

@@ -1,32 +1,45 @@
 from playwright.sync_api import Page, expect
 from pages.base_page import BasePage
 
+
 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")
-        self.archived_batch_list_page = self.page.get_by_role("link", name="Archived Batch List")
-        self.letter_library_index_page = self.page.get_by_role("link", name="Letter Library Index")
-        self.letter_signatory_page = self.page.get_by_role("link", name="Letter Signatory")
-        self.electronic_communication_management_page = self.page.get_by_role("link", name="Electronic Communication Management")
-        self.manage_individual_letter_page = self.page.get_by_text("Manage Individual Letter")
+        self.active_batch_list_page = self.page.get_by_role(
+            "link", name="Active Batch List"
+        )
+        self.archived_batch_list_page = self.page.get_by_role(
+            "link", name="Archived Batch List"
+        )
+        self.letter_library_index_page = self.page.get_by_role(
+            "link", name="Letter Library Index"
+        )
+        self.letter_signatory_page = self.page.get_by_role(
+            "link", name="Letter Signatory"
+        )
+        self.electronic_communication_management_page = self.page.get_by_role(
+            "link", name="Electronic Communication Management"
+        )
+        self.manage_individual_letter_page = self.page.get_by_text(
+            "Manage Individual Letter"
+        )
 
     def verify_manage_individual_letter_page_visible(self) -> None:
         expect(self.manage_individual_letter_page).to_be_visible()
 
-    def go_to_active_batch_list_page(self)->None:
+    def go_to_active_batch_list_page(self) -> None:
         self.click(self.active_batch_list_page)
 
-    def go_to_archived_batch_list_page(self)->None:
+    def go_to_archived_batch_list_page(self) -> None:
         self.click(self.archived_batch_list_page)
 
-    def go_to_letter_library_index_page(self)->None:
+    def go_to_letter_library_index_page(self) -> None:
         self.click(self.letter_library_index_page)
 
-    def go_to_letter_signatory_page(self)->None:
+    def go_to_letter_signatory_page(self) -> None:
         self.click(self.letter_signatory_page)
 
-    def go_to_electronic_communication_management_page(self)->None:
+    def go_to_electronic_communication_management_page(self) -> None:
         self.click(self.electronic_communication_management_page)

pages/contacts_list_page.py

@@ -1,28 +1,39 @@
 from playwright.sync_api import Page, expect
 from pages.base_page import BasePage
 
+
 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")
-        self.edit_my_contact_details_page = self.page.get_by_role("link", name="Edit My Contact Details")
-        self.maintain_contacts_page = self.page.get_by_role("link", name="Maintain Contacts")
-        self.my_preference_settings_page = self.page.get_by_role("link", name="My Preference Settings")
-        self.extract_contact_details_page = self.page.get_by_text("Extract Contact Details")
-        self.resect_and_discard_accredited_page = self.page.get_by_text("Resect and Discard Accredited")
+        self.edit_my_contact_details_page = self.page.get_by_role(
+            "link", name="Edit My Contact Details"
+        )
+        self.maintain_contacts_page = self.page.get_by_role(
+            "link", name="Maintain Contacts"
+        )
+        self.my_preference_settings_page = self.page.get_by_role(
+            "link", name="My Preference Settings"
+        )
+        self.extract_contact_details_page = self.page.get_by_text(
+            "Extract Contact Details"
+        )
+        self.resect_and_discard_accredited_page = self.page.get_by_text(
+            "Resect and Discard Accredited"
+        )
 
-    def go_to_view_contacts_page(self)->None:
+    def go_to_view_contacts_page(self) -> None:
         self.click(self.view_contacts_page)
 
-    def go_to_edit_my_contact_details_page(self)->None:
+    def go_to_edit_my_contact_details_page(self) -> None:
         self.click(self.edit_my_contact_details_page)
 
-    def go_to_maintain_contacts_page(self)->None:
+    def go_to_maintain_contacts_page(self) -> None:
         self.click(self.maintain_contacts_page)
 
-    def go_to_my_preference_settings_page(self)->None:
+    def go_to_my_preference_settings_page(self) -> None:
         self.click(self.my_preference_settings_page)
 
     def verify_extract_contact_details_page_visible(self) -> None: