Skip to content

Commit f963e0f

Browse files
committed
remove logouts
1 parent 1cff1e9 commit f963e0f

File tree

3 files changed

+2
-25
lines changed

3 files changed

+2
-25
lines changed

mavis/test/fixtures/helpers.py

Lines changed: 0 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -67,7 +67,6 @@ def wrapper(school: School, *programmes: Programme):
6767
page, school, list(programmes), [year_group], date_offset=7
6868
)
6969
url = SessionsOverviewPage(page).get_online_consent_url(*programmes)
70-
LogInPage(page).log_out()
7170
yield url
7271

7372
return wrapper
@@ -86,7 +85,6 @@ def wrapper(school: School, *programmes: Programme):
8685
page, school, list(programmes), [year_group], date_offset=7
8786
)
8887
url = SessionsOverviewPage(page).get_online_consent_url(*programmes)
89-
LogInPage(page).log_out()
9088
yield url
9189

9290
return wrapper
@@ -101,24 +99,18 @@ def log_in_as_medical_secretary(
10199
):
102100
LogInPage(page).navigate()
103101
LogInPage(page).log_in_and_choose_team_if_necessary(medical_secretary, team)
104-
yield
105-
LogInPage(page).log_out()
106102

107103

108104
@pytest.fixture
109105
def log_in_as_nurse(set_feature_flags, nurse, team, page):
110106
LogInPage(page).navigate()
111107
LogInPage(page).log_in_and_choose_team_if_necessary(nurse, team)
112-
yield
113-
LogInPage(page).log_out()
114108

115109

116110
@pytest.fixture
117111
def log_in_as_prescriber(set_feature_flags, prescriber, team, page):
118112
LogInPage(page).navigate()
119113
LogInPage(page).log_in_and_choose_team_if_necessary(prescriber, team)
120-
yield
121-
LogInPage(page).log_out()
122114

123115

124116
@pytest.fixture

mavis/test/pages/log_in_page.py

Lines changed: 2 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -83,24 +83,11 @@ def log_in(self, user: User) -> None:
8383

8484
@step("Log out")
8585
def log_out(self) -> None:
86-
if self.log_out_button.is_visible():
87-
# Write logout to audit log
88-
if self.current_user:
89-
self._write_audit_log(
90-
"LOGOUT", self.current_user, self.current_org_code
91-
)
92-
self.current_user = None # Clear current user
93-
94-
self.log_out_button.click()
95-
expect(self.start_page_link).to_be_visible()
86+
self.log_out_button.click()
87+
expect(self.start_page_link).to_be_visible()
9688

9789
@step("Log out")
9890
def log_out_via_reporting_component(self) -> None:
99-
# Write logout to audit log
100-
if self.current_user:
101-
self._write_audit_log("LOGOUT", self.current_user, self.current_org_code)
102-
self.current_user = None # Clear current user
103-
10491
self.log_out_link.click()
10592

10693
@step("Log in as {1} and choose team {2}")

tests/test_online_consent_school_moves.py

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -190,8 +190,6 @@ def test_online_consent_school_moves_with_new_patient(
190190
child, ConsentOption.NASAL_SPRAY_OR_INJECTION
191191
)
192192

193-
LogInPage(page).log_out()
194-
195193

196194
@pytest.mark.accessibility
197195
def test_accessibility(

0 commit comments

Comments
 (0)