Skip to content

Commit

Permalink
test: Don't explicitly go to x0.cockpit.lan in TestIPA
Browse files Browse the repository at this point in the history
x0.cockpit.lan is the name of the local machine, but specifying it
with login_and_go will navigate to it as a remote host, using a URL
like

    https://127.0.0.2:9091/@x0.cockpit.lan

This is not needed since the test only wants the session on localhost,
and doesn't care what happens inside the session. It would be harmless
except that Cockpit might spontaneously open the "Add host" dialog for
"x0.cockpit.lan", which will make the session menu inaccessible for
the subsequent logout. (Only on Firefox, Chrome manages to open it
anyway.)
  • Loading branch information
mvollmer committed Oct 2, 2024
1 parent c44f048 commit 7afb886
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 @@ -592,7 +592,7 @@ class TestIPA(TestRealms, CommonTests):

# check respecting FreeIPA's/sssd's ssh known host keys; login_and_go() does not expect/answer
# unknown host prompts
b.login_and_go(user=f'{self.admin_user}@cockpit.lan', password=self.admin_password, host="x0.cockpit.lan")
b.login_and_go(user=f'{self.admin_user}@cockpit.lan', password=self.admin_password)
# starts proper session
m.execute("until loginctl --no-legend list-sessions | grep -qi 'admin@COCKPIT.LAN.*web console'; do sleep 1; done", timeout=10)
b.logout()
Expand Down

0 comments on commit 7afb886

Please sign in to comment.