Skip to content

Commit 13ca365

Browse files
committed
Fixed a couple more failing tests
1 parent b9927e1 commit 13ca365

File tree

2 files changed

+11
-12
lines changed

2 files changed

+11
-12
lines changed

pages/reports/reports_page.py

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,5 @@
11
from playwright.sync_api import Page
2+
import pytest
23
from pages.base_page import BasePage
34

45

@@ -196,6 +197,8 @@ def click_nhs_number_link(self, page: Page) -> None:
196197
"//*[@id='listReportDataTable']/tbody/tr[3]/td[2]",
197198
"#listReportDataTable > tbody > tr:nth-child(3) > td:nth-child(1) > a",
198199
"#subjInactiveOpenEpisodes > tbody > tr:nth-child(1) > td.NHS_NUMBER.dt-type-numeric > a",
200+
"# subjInactiveOpenEpisodes > tbody > tr:nth-child(1) > td.NHS_NUMBER.dt-type-numeric > a",
201+
"//*[@id=\"subjInactiveOpenEpisodes\"]/tbody/tr[1]/td[1]/a",
199202
]
200203

201204
for locator_string in locators:

tests/test_reports_page.py

Lines changed: 8 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -107,7 +107,7 @@ def test_failsafe_reports_date_report_last_requested(page: Page) -> None:
107107
report_timestamp
108108
)
109109

110-
110+
@pytest.mark.only
111111
def test_failsafe_reports_screening_subjects_with_inactive_open_episode(
112112
page: Page,
113113
) -> None:
@@ -449,22 +449,18 @@ def test_operational_reports_screening_practitioner_6_weeks_availability_not_set
449449

450450
# Verify timestamp has updated to current date and time
451451
report_timestamp = DateTimeUtils.report_timestamp_date_format()
452-
ReportsPage(
453-
page
454-
).six_weeks_availability_not_set_up_report_timestamp_element.to_contain_text(
455-
report_timestamp
456-
)
452+
expect(
453+
ReportsPage(page).six_weeks_availability_not_set_up_report_timestamp_element
454+
).to_contain_text(report_timestamp)
457455

458456
# Click "Refresh" button
459457
ReportsPage(page).click_refresh_button()
460458

461459
# Verify timestamp has updated to current date and time
462460
report_timestamp = DateTimeUtils.report_timestamp_date_format()
463-
ReportsPage(
464-
page
465-
).six_weeks_availability_not_set_up_report_timestamp_element.to_contain_text(
466-
report_timestamp
467-
)
461+
expect(
462+
ReportsPage(page).six_weeks_availability_not_set_up_report_timestamp_element
463+
).to_contain_text(report_timestamp)
468464

469465

470466
def test_operational_reports_screening_practitioner_appointments(
@@ -507,6 +503,6 @@ def test_operational_reports_screening_practitioner_appointments(
507503
report_timestamp = (
508504
DateTimeUtils.screening_practitioner_appointments_report_timestamp_date_format()
509505
)
510-
expect(ReportsPage(Page).common_report_timestamp_element).to_contain_text(
506+
expect(ReportsPage(page).common_report_timestamp_element).to_contain_text(
511507
report_timestamp
512508
)

0 commit comments

Comments
 (0)