Skip to content

Commit 4b0753e

Browse files
pondrejkrmynar
authored andcommitted
set the logout redirect setting within the test body (SatelliteQE#20824)
1 parent a4d510f commit 4b0753e

1 file changed

Lines changed: 18 additions & 12 deletions

File tree

tests/foreman/destructive/test_ldap_authentication.py

Lines changed: 18 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -262,14 +262,8 @@ def test_single_sign_on_ldap_ad_server(
262262
assert f'{target_sat.url}/new/hosts' in result.stdout
263263

264264

265-
@pytest.mark.parametrize(
266-
'setting_update',
267-
[f'login_delegation_logout_url={LOGIN_DELEGATION_LOGOUT_URL}'],
268-
ids=["external_redirect"],
269-
indirect=True,
270-
)
271265
def test_single_sign_on_using_rhsso(
272-
enable_external_auth_rhsso, rhsso_setting_setup, module_target_sat, setting_update
266+
enable_external_auth_rhsso, rhsso_setting_setup, module_target_sat
273267
):
274268
"""Verify the single sign-on functionality with external authentication RH-SSO
275269
@@ -295,11 +289,23 @@ def test_single_sign_on_using_rhsso(
295289
actual_user = session.task.read_all(widget_names="current_user")['current_user']
296290
assert settings.rhsso.rhsso_user in actual_user
297291
# logout verification for SAT-40322
298-
session.rhsso_login.logout()
299-
assert session.browser.url == LOGIN_DELEGATION_LOGOUT_URL, "Unsuccessful logout redirect"
300-
with pytest.raises(NavigationTriesExceeded) as error:
301-
session.task.read_all(widget_names='current_user')['current_user']
302-
assert error.typename == 'NavigationTriesExceeded'
292+
setting_object = module_target_sat.api.Setting().search(
293+
query={'search': 'name=login_delegation_logout_url'}
294+
)[0]
295+
default_setting_value = '' if setting_object.value is None else setting_object.value
296+
setting_object.value = LOGIN_DELEGATION_LOGOUT_URL
297+
setting_object.update({'value'})
298+
try:
299+
session.rhsso_login.logout()
300+
assert session.browser.url == LOGIN_DELEGATION_LOGOUT_URL, (
301+
"Unsuccessful logout redirect"
302+
)
303+
with pytest.raises(NavigationTriesExceeded) as error:
304+
session.task.read_all(widget_names='current_user')['current_user']
305+
assert error.typename == 'NavigationTriesExceeded'
306+
finally:
307+
setting_object.value = default_setting_value
308+
setting_object.update({'value'})
303309

304310

305311
def test_external_logout_rhsso(rhsso_setting_setup, enable_external_auth_rhsso, module_target_sat):

0 commit comments

Comments
 (0)