Skip to content

Commit

Permalink
test: Fix wait timeout adjustment in TestIPA.testClientCertAuthentica…
Browse files Browse the repository at this point in the history
…tion

Since commit 49ee017, the step that takes long is already the
`Browser.open()`, as that loads the packages and frame from the remote
machine, and the timeout now happens in `waitPageLoad()`. Still, loading
the frame also takes a while, so keep the long timeout for enter-page()
as well.
  • Loading branch information
martinpitt committed Nov 8, 2023
1 parent ef2eae1 commit f021c51
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion test/verify/check-system-realms
Original file line number Diff line number Diff line change
Expand Up @@ -820,8 +820,8 @@ ipa-advise enable-admins-sudo | sh -ex
# ssh -K is supposed to forward the credentials cache, but doesn't; klist in the ssh session is empty
# and there is no ccache; so, emulate what cockpit-ssh could eventually do and check that *if* the
# session had the ticket forwarded, it *could* do sudo. See https://issues.redhat.com/browse/COCKPIT-643
b.open("/@x0.cockpit.lan/system/terminal")
with b.wait_timeout(60):
b.open("/@x0.cockpit.lan/system/terminal")
b.enter_page("/system/terminal", host="x0.cockpit.lan")
b.wait_in_text(".terminal .xterm-accessibility-tree", "alice")
b.key_press(f"{ccache_env} sudo whoami\r")
Expand Down

0 comments on commit f021c51

Please sign in to comment.