Skip to content

Commit 9422293

Browse files
committed
Addressing sonarqube issues
1 parent b925a16 commit 9422293

File tree

1 file changed

+16
-7
lines changed

1 file changed

+16
-7
lines changed

pages/reports/reports_page.py

Lines changed: 16 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,9 @@ def __init__(self, page):
1111

1212
# Initialize TableUtils for different tables
1313
self.failsafe_reports_sub_links_table = TableUtils(page, "#listReportDataTable")
14-
self.fail_safe_reports_screening_subjects_with_inactive_open_episodes_table = TableUtils(page, "#subjInactiveOpenEpisodes")
14+
self.fail_safe_reports_screening_subjects_with_inactive_open_episodes_table = (
15+
TableUtils(page, "#subjInactiveOpenEpisodes")
16+
)
1517

1618
# Reports page main menu links
1719
self.bureau_reports_link = self.page.get_by_text("Bureau Reports")
@@ -51,17 +53,18 @@ def __init__(self, page):
5153
)
5254

5355
# Set patients screening centre dropdown locators
56+
SCREENING_CENTRE = "Screening Centre"
5457
self.set_patients_screening_centre_dropdown = self.page.locator(
5558
"#cboScreeningCentre"
5659
)
5760
self.six_weeks_availability_not_set_up_set_patients_screening_centre_dropdown = self.page.get_by_label(
58-
"Screening Centre"
61+
SCREENING_CENTRE
5962
)
6063
self.practitioner_appointments_set_patients_screening_centre_dropdown = (
61-
page.get_by_label("Screening Centre")
64+
page.get_by_label(SCREENING_CENTRE)
6265
)
6366
self.attendance_not_updated_set_patients_screening_centre_dropdown = (
64-
page.get_by_label("Screening Centre")
67+
page.get_by_label(SCREENING_CENTRE)
6568
)
6669

6770
# Select screening practitioner dropdown locators
@@ -193,14 +196,20 @@ def click_failsafe_reports_sub_links(self):
193196
"""
194197
self.failsafe_reports_sub_links_table.click_first_link_in_column("NHS Number")
195198

196-
def click_fail_safe_reports_screening_subjects_with_inactive_open_episodes_link(self):
199+
def click_fail_safe_reports_screening_subjects_with_inactive_open_episodes_link(
200+
self,
201+
):
197202
"""
198203
Clicks the first NHS number link from the primary report table.
199204
"""
200-
self.fail_safe_reports_screening_subjects_with_inactive_open_episodes_table.click_first_link_in_column("NHS Number")
205+
self.fail_safe_reports_screening_subjects_with_inactive_open_episodes_table.click_first_link_in_column(
206+
"NHS Number"
207+
)
201208

202209
def click_fail_safe_reports_identify_and_link_new_gp_practices_link(self):
203210
"""
204211
Clicks the first Practice Code link from the primary report table.
205212
"""
206-
self.failsafe_reports_sub_links_table.click_first_link_in_column("Practice Code")
213+
self.failsafe_reports_sub_links_table.click_first_link_in_column(
214+
"Practice Code"
215+
)

0 commit comments

Comments
 (0)