Skip to content
This repository was archived by the owner on Apr 26, 2024. It is now read-only.

Commit b5dea87

Browse files
committed
Fix test failure due to bad merge
0dd2649 (#9112) changed the signature of `auth_via_oidc`. Meanwhile, 26d1033 (#9091) introduced a new test which relied on the old signature of `auth_via_oidc`. The two branches were never tested together until they landed in develop.
1 parent 350d992 commit b5dea87

File tree

1 file changed

+3
-1
lines changed

1 file changed

+3
-1
lines changed

tests/rest/client/v2_alpha/test_auth.py

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -475,7 +475,9 @@ def test_ui_auth_fails_for_incorrect_sso_user(self):
475475
session_id = channel.json_body["session"]
476476

477477
# do the OIDC auth, but auth as the wrong user
478-
channel = self.helper.auth_via_oidc("wrong_user", ui_auth_session_id=session_id)
478+
channel = self.helper.auth_via_oidc(
479+
{"sub": "wrong_user"}, ui_auth_session_id=session_id
480+
)
479481

480482
# that should return a failure message
481483
self.assertSubstring("We were unable to validate", channel.text_body)

0 commit comments

Comments
 (0)