diff options
| author | Patrick Cloke <clokep@users.noreply.github.com> | 2020-12-02 07:45:42 -0500 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2020-12-02 07:45:42 -0500 |
| commit | 8388384a640d3381b5858d3fb1d2ea0a8c9c059c (patch) | |
| tree | 1c1471ce836806d18d71579aa32aae6a9a4a7ce4 /tests/handlers/test_oidc.py | |
| parent | Add basic SAML tests for mapping users. (#8800) (diff) | |
| download | synapse-8388384a640d3381b5858d3fb1d2ea0a8c9c059c.tar.xz | |
Fix a regression when grandfathering SAML users. (#8855)
This was broken in #8801 when abstracting code shared with OIDC. After this change both SAML and OIDC have a concept of grandfathering users, but with different implementations.
Diffstat (limited to 'tests/handlers/test_oidc.py')
| -rw-r--r-- | tests/handlers/test_oidc.py | 8 |
1 files changed, 8 insertions, 0 deletions
diff --git a/tests/handlers/test_oidc.py b/tests/handlers/test_oidc.py index c9807a7b73..d485af52fd 100644 --- a/tests/handlers/test_oidc.py +++ b/tests/handlers/test_oidc.py @@ -731,6 +731,14 @@ class OidcHandlerTestCase(HomeserverTestCase): ) self.assertEqual(mxid, "@test_user:test") + # Subsequent calls should map to the same mxid. + mxid = self.get_success( + self.handler._map_userinfo_to_user( + userinfo, token, "user-agent", "10.10.10.10" + ) + ) + self.assertEqual(mxid, "@test_user:test") + # Note that a second SSO user can be mapped to the same Matrix ID. (This # requires a unique sub, but something that maps to the same matrix ID, # in this case we'll just use the same username. A more realistic example |