Skip to content

Commit 8a8d35e

Browse files
committed
Worked on review comments
1 parent 146abc0 commit 8a8d35e

File tree

2 files changed

+8
-4
lines changed

2 files changed

+8
-4
lines changed

pages/screening_subject_search/handover_into_symptomatic_care_page.py

Lines changed: 7 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -28,9 +28,13 @@ def select_referral_dropdown_option(self, value: str) -> None:
2828
"""
2929
self.referral_dropdown.select_option(value)
3030

31-
def select_first_practitioner(self) -> None:
32-
"""Select the first option from the Practitioner dropdown."""
33-
self.practitioner_dropdown.select_option(index=1)
31+
def select_practitioner_from_index(self, practitioner_index: int) -> None:
32+
"""
33+
Select the first option from the Practitioner dropdown.
34+
Args:
35+
practitioner_index (int): The index of the practitioner to select.
36+
"""
37+
self.practitioner_dropdown.select_option(index=practitioner_index)
3438

3539
def click_calendar_button(self) -> None:
3640
"""Click the calendar button to open the calendar picker."""

tests/regression/regression_tests/fobt_regression_tests/test_scenario_17.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -641,7 +641,7 @@ def test_scenario_17(page: Page) -> None:
641641
HandoverIntoSymptomaticCarePage(page).select_referral_dropdown_option(
642642
"Referral to Patient's GP Practice"
643643
)
644-
HandoverIntoSymptomaticCarePage(page).select_first_practitioner()
644+
HandoverIntoSymptomaticCarePage(page).select_practitioner_from_index(1)
645645
HandoverIntoSymptomaticCarePage(page).fill_notes("Handover notes - unfit (cease)")
646646
HandoverIntoSymptomaticCarePage(page).select_cease_from_program(True)
647647
HandoverIntoSymptomaticCarePage(page).click_save_button()

0 commit comments

Comments
 (0)