correcting formatting errors

Author: mepr1

pages/base_page.py

@@ -249,7 +249,6 @@ def safe_accept_dialog(self, locator: Locator) -> None:
         except Exception as e:
             logging.error(f"Click failed: {e}")
 
-
     def assert_dialog_text(self, expected_text: str) -> None:
         """
         Asserts that a dialog appears and contains the expected text.
@@ -280,5 +279,4 @@ def safe_accept_dialog_select_option(self, locator: Locator, option: str) -> Non
         try:
             locator.select_option(option)
         except Exception as e:
-            logging.error(f"Option selection failed: {e}")   
-            
+            logging.error(f"Option selection failed: {e}")

pages/screening_subject_search/subject_events_notes.py

@@ -106,11 +106,13 @@ def get_title_and_note_from_row(self, row_number: int = 0) -> dict:
         """
         Extracts title and note from a specific row's 'Notes' column using dynamic column index.
         """
-        cell_text= self.table_utils.get_cell_value("Notes", row_number)
+        cell_text = self.table_utils.get_cell_value("Notes", row_number)
         lines = cell_text.split("\n\n")
         title = lines[0].strip() if len(lines) > 0 else ""
         note = lines[1].strip() if len(lines) > 1 else ""
-        logging.info(f"Extracted title: '{title}' and note: '{note}' from row {row_number}")    
+        logging.info(
+            f"Extracted title: '{title}' and note: '{note}' from row {row_number}"
+        )
         return {"title": title, "note": note}
 
 
@@ -134,7 +136,7 @@ class NotesOptions(StrEnum):
     SUBJECT_NOTE = "4111"
     KIT_NOTE = "308015"
     ADDITIONAL_CARE_NOTE = "4112"
-    EPISODE_NOTE= "4110"
+    EPISODE_NOTE = "4110"
 
 
 class NotesStatusOptions(StrEnum):

pages/screening_subject_search/subject_screening_summary_page.py

@@ -72,9 +72,7 @@ def verify_subject_search_results_title_subject_screening_summary(self) -> None:
 
     def verify_subject_search_results_title_subject_search_results(self) -> None:
         """Verify that the subject search results title contains 'Subject Search Results'."""
-        self.bowel_cancer_screening_page_title_contains_text(
-            "Subject Search Results"
-        )
+        self.bowel_cancer_screening_page_title_contains_text("Subject Search Results")
 
     def get_latest_event_status_cell(self, latest_event_status: str) -> Locator:
         """Get the latest event status cell by its name."""
@@ -208,25 +206,34 @@ def verify_note_link_present(self, note_type_name: str) -> None:
             AssertionError: If the link is not visible on the page.
         """
         logging.info(f"Checking if the '{note_type_name}' link is visible.")
-        note_link_locator = self.page.get_by_role("link", name=f"({note_type_name})")  # Dynamic locator for the note type link
-        assert note_link_locator.is_visible(), f"'{note_type_name}' link is not visible, but it should be."
-        logging.info(f"Verified: '{note_type_name}' link is visible.")    
+        note_link_locator = self.page.get_by_role(
+            "link", name=f"({note_type_name})"
+        )  # Dynamic locator for the note type link
+        assert (
+            note_link_locator.is_visible()
+        ), f"'{note_type_name}' link is not visible, but it should be."
+        logging.info(f"Verified: '{note_type_name}' link is visible.")
 
     def verify_note_link_not_present(self, note_type_name: str) -> None:
         """
-    Verifies that the link for the specified note type is not visible on the page.
+        Verifies that the link for the specified note type is not visible on the page.
 
-    Args:
-        note_type_name (str): The name of the note type to check (e.g., 'Additional Care Note', 'Episode Note').
+        Args:
+            note_type_name (str): The name of the note type to check (e.g., 'Additional Care Note', 'Episode Note').
 
-    Raises:
-        AssertionError: If the link is visible on the page.
-    """
+        Raises:
+            AssertionError: If the link is visible on the page.
+        """
         logging.info(f"Checking if the '{note_type_name}' link is not visible.")
-        note_link_locator = self.page.get_by_role("link", name=f"({note_type_name})")  # Dynamic locator for the note type link
-        assert not note_link_locator.is_visible(), f"'{note_type_name}' link is visible, but it should not be."
+        note_link_locator = self.page.get_by_role(
+            "link", name=f"({note_type_name})"
+        )  # Dynamic locator for the note type link
+        assert (
+            not note_link_locator.is_visible()
+        ), f"'{note_type_name}' link is visible, but it should not be."
         logging.info(f"Verified: '{note_type_name}' link is not visible.")
 
+
 class ChangeScreeningStatusOptions(Enum):
     """Enum for Change Screening Status options."""
 

utils/oracle/oracle_specific_functions.py

@@ -517,15 +517,15 @@ def get_subjects_with_multiple_notes(note_type: int) -> pd.DataFrame:
         SELECT COUNT(sn.screening_subject_id)
         FROM supporting_notes_t sn
         WHERE sn.screening_subject_id = ss.screening_subject_id
-          AND sn.type_id = :note_type
-          AND sn.status_id = 4100
+        AND sn.type_id = :note_type
+        AND sn.status_id = 4100
         GROUP BY sn.screening_subject_id
     )
     AND 200 > (SELECT COUNT(sn.screening_subject_id)
         FROM supporting_notes_t sn
         WHERE sn.screening_subject_id = ss.screening_subject_id
-          AND sn.type_id = :note_type
-          AND sn.status_id = 4100
+        AND sn.type_id = :note_type
+        AND sn.status_id = 4100
         GROUP BY sn.screening_subject_id
     )
     AND ROWNUM = 1