Feature/bcss 21309 fobt regression scenario 6 (#137)

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

<!-- Describe your changes in detail. -->
Migration of fobt regression tests scenario 6 from selenium to
playwright plus updates to any POMs / Utils as required

## Context

<!-- Why is this change required? What problem does it solve? -->
This is part of the selenium to playwright migration work

## Type of changes

<!-- What types of changes does your code introduce? Put an `x` in all
the boxes that apply. -->

- [x] Refactoring (non-breaking change)
- [x] 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
- [x] I have added tests to cover my changes (where appropriate)
- [x] 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: Andyg79

pages/datasets/colonoscopy_dataset_page.py

@@ -3,6 +3,12 @@
 from enum import Enum
 
 
+class FitForColonoscopySspOptions(Enum):
+    YES = "17058"
+    NO = "17059"
+    UNABLE_TO_ASSESS = "17954"
+
+
 class ColonoscopyDatasetsPage(BasePage):
     """Colonoscopy Datasets Page locators, and methods for interacting with the page"""
 
@@ -44,16 +50,19 @@ def select_asa_grade_option(self, option: str) -> None:
         """
         self.select_asa_grade_dropdown.select_option(option)
 
-    def select_fit_for_colonoscopy_option(self, option: str) -> None:
+    def select_fit_for_colonoscopy_option(
+        self, option: FitForColonoscopySspOptions
+    ) -> None:
         """
-        This method is designed to select a specific option from the colonoscopy dataset page, Fit for Colonoscopy (SSP) dropdown menu.
+        Selects a specific option from the 'Fit for Colonoscopy (SSP)' dropdown.
+
         Args:
-            option (str): The option to be selected. This should be a string that matches one of the available options in the dropdown menu.
-                Valid options are: "YES", "NO", or "UNABLE_TO_ASSESS".
+            option (FitForColonoscopySspOptions): Enum member representing the desired option.
+
         Returns:
             None
         """
-        self.select_fit_for_colonoscopy_dropdown.select_option(option)
+        self.select_fit_for_colonoscopy_dropdown.select_option(option.value)
 
     def click_dataset_complete_radio_button_yes(self) -> None:
         """Clicks the 'Yes' radio button for the dataset complete option."""
@@ -72,9 +81,3 @@ class AsaGradeOptions(Enum):
     MORIBUND = "17013"
     NOT_APPLICABLE = "17014"
     NOT_KNOWN = "17015"
-
-
-class FitForColonoscopySspOptions(Enum):
-    YES = "17058"
-    NO = "17059"
-    UNABLE_TO_ASSESS = "17954"

pages/screening_subject_search/advance_fobt_screening_episode_page.py

@@ -17,7 +17,9 @@ def __init__(self, page: Page):
         self.calendar_button = self.page.get_by_role("button", name="Calendar")
         self.test_type_dropdown = self.page.locator("#UI_EXT_TEST_TYPE_2233")
         self.test_type_dropdown_2 = self.page.locator("#UI_EXT_TEST_TYPE_4325")
-        self.advance_checkbox = self.page.get_by_label("There are some events available which should only be used in exceptional circumstances. If you wish to see them, check this box")
+        self.advance_checkbox = self.page.get_by_label(
+            "There are some events available which should only be used in exceptional circumstances. If you wish to see them, check this box"
+        )
         self.invite_for_diagnostic_test_button = self.page.get_by_role(
             "button", name="Invite for Diagnostic Test >>"
         )
@@ -42,12 +44,23 @@ def __init__(self, page: Page):
         self.record_contact_with_patient_button = self.page.get_by_role(
             "button", name="Record Contact with Patient"
         )
+        self.suitable_for_radiological_test_button = self.page.get_by_role(
+            "button", name="Suitable for Radiological Test"
+        )
+        self.decision_not_to_continue_with_diagnostic_test_button = (
+            self.page.get_by_role(
+                "button", name="Decision not to Continue with Diagnostic Test"
+            )
+        )
+        self.waiting_decision_to_proceed_with_diagnostic_test_button = (
+            self.page.get_by_role(
+                "button", name="Waiting Decision to Proceed with Diagnostic Test"
+            )
+        )
 
     def click_suitable_for_endoscopic_test_button(self) -> None:
         """Click the 'Suitable for Endoscopic Test' button."""
-        AdvanceFOBTScreeningEpisodePage(self.page).safe_accept_dialog(
-            self.suitable_for_endoscopic_test_button
-        )
+        self.safe_accept_dialog(self.suitable_for_endoscopic_test_button)
 
     def click_calendar_button(self) -> None:
         """Click the calendar button to open the calendar picker."""
@@ -63,19 +76,15 @@ def select_test_type_dropdown_option_2(self, text: str) -> None:
 
     def click_invite_for_diagnostic_test_button(self) -> None:
         """Click the 'Invite for Diagnostic Test' button."""
-        AdvanceFOBTScreeningEpisodePage(self.page).safe_accept_dialog(
-            self.invite_for_diagnostic_test_button
-        )
+        self.safe_accept_dialog(self.invite_for_diagnostic_test_button)
 
     def click_attend_diagnostic_test_button(self) -> None:
         """Click the 'Attend Diagnostic Test' button."""
         self.click(self.attend_diagnostic_test_button)
 
     def click_other_post_investigation_button(self) -> None:
         """Click the 'Other Post-investigation' button."""
-        AdvanceFOBTScreeningEpisodePage(self.page).safe_accept_dialog(
-            self.other_post_investigation_button
-        )
+        self.safe_accept_dialog(self.other_post_investigation_button)
 
     def get_latest_event_status_cell(self, latest_event_status: str) -> Locator:
         """Get the cell containing the latest event status."""
@@ -116,3 +125,19 @@ def click_record_contact_with_patient_button(self) -> None:
     def check_advance_checkbox(self) -> None:
         """Selects the 'Advance FOBT' checkbox"""
         self.advance_checkbox.check()
+
+    def click_suitable_for_radiological_test_button(self) -> None:
+        """Click the 'Suitable for Radiological Test' button."""
+        self.safe_accept_dialog(self.suitable_for_radiological_test_button)
+
+    def click_decision_not_to_continue_with_diagnostic_test(self) -> None:
+        """Click the 'Decision not to Continue with Diagnostic Test' button."""
+        self.safe_accept_dialog(
+            self.decision_not_to_continue_with_diagnostic_test_button
+        )
+
+    def click_waiting_decision_to_proceed_with_diagnostic_test(self) -> None:
+        """Click the 'Waiting Decision to Proceed with Diagnostic Test' button."""
+        self.safe_accept_dialog(
+            self.waiting_decision_to_proceed_with_diagnostic_test_button
+        )

pages/screening_subject_search/reopen_fobt_screening_episode_page.py

@@ -12,7 +12,14 @@ def __init__(self, page: Page):
         self.reopen_to_book_an_assessment_button = self.page.get_by_role(
             "button", name="Reopen to book an assessment"
         )
+        self.reopen_following_non_response_button = self.page.get_by_role(
+            "button", name="Reopen following Non-Response"
+        )
 
     def click_reopen_to_book_an_assessment_button(self) -> None:
         """Click the 'Reopen to book an assessment' button."""
         self.safe_accept_dialog(self.reopen_to_book_an_assessment_button)
+
+    def click_reopen_following_non_response_button(self) -> None:
+        """Click the 'Reopen following Non-Response' button."""
+        self.safe_accept_dialog(self.reopen_following_non_response_button)

tests/regression/regression_tests/fobt_regression_tests/test_scenario_3.py

@@ -8,7 +8,6 @@
 from utils.batch_processing import batch_processing
 from utils.fit_kit import FitKitGeneration, FitKitLogged
 from utils import screening_subject_page_searcher
-from utils.calendar_picker import CalendarPicker
 from utils.sspi_change_steps import SSPIChangeSteps
 from utils.appointments import book_appointments
 from pages.base_page import BasePage
@@ -22,9 +21,6 @@
 from pages.screening_subject_search.advance_fobt_screening_episode_page import (
     AdvanceFOBTScreeningEpisodePage,
 )
-from pages.screening_practitioner_appointments.book_appointment_page import (
-    BookAppointmentPage,
-)
 from pages.screening_subject_search.record_diagnosis_date_page import (
     RecordDiagnosisDatePage,
 )

tests/regression/regression_tests/fobt_regression_tests/test_scenario_5.py

@@ -21,7 +21,7 @@
 from pages.screening_subject_search.record_diagnosis_date_page import (
     RecordDiagnosisDatePage,
 )
-from utils.appointments import mark_appointment_as_dna
+from utils.appointments import AppointmentAttendance
 
 
 @pytest.mark.usefixtures("setup_org_and_appointments")
@@ -61,6 +61,7 @@ def test_scenario_5(page: Page) -> None:
     """
 
     summary_page = SubjectScreeningSummaryPage(page)
+    attendance = AppointmentAttendance(page)
     logging.info(
         "[TEST START] Regression - Scenario: 5: DNA colonoscopy assessment twice"
     )
@@ -227,7 +228,7 @@ def test_scenario_5(page: Page) -> None:
     # And I view the event history for the subject's latest episode
     # And I view the latest practitioner appointment in the subject's episode
     # And The subject DNAs the practitioner appointment
-    mark_appointment_as_dna(page, "Patient did not attend")
+    attendance.mark_as_dna("Patient did not attend")
 
     # And there is a "J11" letter batch for my subject with the exact title "Practitioner Clinic 1st Appointment Non Attendance (Patient)"
     # When I process the open "J11" letter batch for my subject
@@ -278,7 +279,7 @@ def test_scenario_5(page: Page) -> None:
     # And I view the event history for the subject's latest episode
     # And I view the latest practitioner appointment in the subject's episode
     # And The subject DNAs the practitioner appointment
-    mark_appointment_as_dna(page, "Patient did not attend")
+    attendance.mark_as_dna("Patient did not attend")
 
     # Then my subject has been updated as follows:
     subject_assertion(