Feature/bcss 21318 fobtregressiontests scenario 15 (#149)

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

<!-- Describe your changes in detail. -->
Adding scenario 15 from FOBTRegressionTeests feature file

## Context

<!-- Why is this change required? What problem does it solve? -->
Adding scenario 15 from FOBTRegressionTeests feature file

## 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: adrianoaru-nhs

investigation_dataset_ui.py

@@ -42,6 +42,7 @@
     PolypInterventionRetrievedOptions,
     OpticalDiagnosisOptions,
     OpticalDiagnosisConfidenceOptions,
+    PolypTypeLeftInSituOptions,
 )
 
 
@@ -90,6 +91,7 @@
     "ExtracolonicSummaryCodeOptions": ExtracolonicSummaryCodeOptions,
     "OpticalDiagnosisOptions": OpticalDiagnosisOptions,
     "OpticalDiagnosisConfidenceOptions": OpticalDiagnosisConfidenceOptions,
+    "PolypTypeLeftInSituOptions": PolypTypeLeftInSituOptions,
     "PolypInterventionRetrievedOptions": PolypInterventionRetrievedOptions,
 }
 

investigation_dataset_ui_app/dataset_fields.json

@@ -290,6 +290,16 @@
           "field": "left in situ",
           "value": "YesNoOptions.YES"
         }
+      },
+      {
+        "key": "polyp type left in situ",
+        "type": "PolypTypeLeftInSituOptions",
+        "description": "Polyp type that was left in situ",
+        "optional": true,
+        "conditional_on": {
+          "field": "reason left in situ",
+          "value": "PolypReasonLeftInSituOptions.POLYP_TYPE"
+        }
       }
     ],
     "list": true

pages/datasets/investigation_dataset_page.py

@@ -1678,6 +1678,13 @@ class OpticalDiagnosisConfidenceOptions(StrEnum):
     LOW = "305746"
 
 
+class PolypTypeLeftInSituOptions(StrEnum):
+    """Enum for Polyp Type Left in Situ options"""
+
+    INFLAMMATORY_POLYP = "17300~Sub Type Not Applicable"
+    LYMPHOID_FOLLICLE = "200599~Sub Type Not Applicable"
+
+
 # Registry of all known Enums to search when matching string values
 ALL_ENUMS: List[type[Enum]] = [
     obj

pages/screening_subject_search/advance_fobt_screening_episode_page.py

@@ -125,6 +125,12 @@ def __init__(self, page: Page):
                 name="Redirect to Re-record the Outcome of Symptomatic Referral",
             )
         )
+        self.redirect_to_establish_suitability_for_diagnostic_tests_button = (
+            self.page.get_by_role(
+                "button",
+                name="Redirect to Establish Suitability for Diagnostic Tests",
+            )
+        )
         # Contact recording locators
         self.contact_direction_dropdown = self.page.get_by_label("Contact Direction")
         self.contact_made_between_dropdown = self.page.get_by_label(
@@ -464,6 +470,18 @@ def click_refer_to_survelliance_after_symptomatic_referral_button(
             self.refer_to_survelliance_after_symptomatic_referral_button
         )
 
-    def click_redirect_to_rerecord_the_outcome_of_symptomatic_referral_button(self) -> None:
+    def click_redirect_to_rerecord_the_outcome_of_symptomatic_referral_button(
+        self,
+    ) -> None:
         """Click the 'Redirect to Re-record the Outcome of Symptomatic Referral' button."""
-        self.safe_accept_dialog(self.redirect_to_rerecord_the_outcome_of_symptomatic_referral_button)
+        self.safe_accept_dialog(
+            self.redirect_to_rerecord_the_outcome_of_symptomatic_referral_button
+        )
+
+    def click_redirect_to_establish_suitability_for_diagnostic_tests_button(
+        self,
+    ) -> None:
+        """Click the 'Redirect to Establish Suitability for Diagnostic Tests' button"""
+        self.safe_accept_dialog(
+            self.redirect_to_establish_suitability_for_diagnostic_tests_button
+        )

pages/screening_subject_search/diagnostic_test_outcome_page.py

@@ -6,17 +6,27 @@
 
 
 class ReferralProcedureType(StrEnum):
+    """Enum class referral procedure type options"""
+
     ENDOSCOPIC = "20356"
     RADIOLOGICAL = "20357"
 
 
 class ReasonForOnwardReferral(StrEnum):
+    """Enum class containing reason for onward referral options"""
+
     CURRENTLY_UNSUITABLE_FOR_ENDOSCOPIC_REFERRAL = "20358"
     FURTHER_CLINICAL_ASSESSMENT = "20359"
     INCOMPLETE_COLONIC_VISUALISATION = "20481"
     POLYP_EXCISION = "203011"
     CORRECTIVE_SURGERY = "203012"
     SUSPECTED_CANCER_SURGERY = "203013"
+    POLYP_NOT_FULLY_EXCISED = "20321"
+    CHECK_POLYP_SITE = "20322"
+    MULTIPLE_POLYPS_NOT_ALL_REMOVED = "20323"
+    HISTOLOGY_REQUIRED = "20324"
+    UNEXPLAINED_SYMPTOMS = "20325"
+    INTERVENTIONS_REQUIRED = "20326"
 
 
 class DiagnosticTestOutcomePage(BasePage):

tests/regression/regression_tests/fobt_regression_tests/test_scenario_13.py

@@ -134,6 +134,7 @@ def test_scenario_13(page: Page) -> None:
         "subject age": "Between 60 and 72",
         "subject has unprocessed sspi updates": "No",
         "subject has user dob updates": "No",
+        "subject hub code": "User's hub",
     }
 
     query, bind_vars = SubjectSelectionQueryBuilder().build_subject_selection_query(

tests/regression/regression_tests/fobt_regression_tests/test_scenario_14.py

@@ -2,10 +2,8 @@
 import logging
 from datetime import datetime, timedelta
 from playwright.sync_api import Page
-from classes.subject import subject
 from classes.subject.subject import Subject
 from classes.user.user import User
-from pages.screening_subject_search import refer_to_mdt_page
 from pages.screening_subject_search.non_neoplastic_result_from_symptomatic_procedure_page import (
     NonNeoplasticResultFromSymptomaticProcedurePage,
 )
@@ -74,14 +72,10 @@
     PolypExcisionCompleteOptions,
     PolypDysplasiaOptions,
     YesNoUncertainOptions,
-    ReasonPathologyLostOptions,
 )
 from pages.screening_subject_search.reopen_fobt_screening_episode_page import (
     ReopenFOBTScreeningEpisodePage,
 )
-from pages.screening_subject_search.contact_with_patient_page import (
-    ContactWithPatientPage,
-)
 from pages.screening_subject_search.refer_to_mdt_page import ReferToMDTPage
 from pages.screening_subject_search.lnpcp_result_from_symptomatic_procedure_page import (
     LnpcpResultFromSymptomaticProcedure,
@@ -90,11 +84,8 @@
     HandoverIntoSymptomaticCarePage,
 )
 from classes.repositories.person_repository import PersonRepository
-from classes.repositories.episode_repository import EpisodeRepository
 from utils.sspi_change_steps import SSPIChangeSteps
 from pages.organisations.organisations_page import OrganisationSwitchPage
-from utils.call_and_recall_utils import CallAndRecallUtils
-from utils.oracle.subject_creation_util import CreateSubjectSteps
 from classes.repositories.subject_repository import SubjectRepository