diff options
author | Richard van der Hoff <1389908+richvdh@users.noreply.github.com> | 2020-12-14 20:42:03 +0000 |
---|---|---|
committer | GitHub <noreply@github.com> | 2020-12-14 20:42:03 +0000 |
commit | 6d02eb22dfde9551c515acaf73503e2500e00eaf (patch) | |
tree | 5a427640a3ae918a569b106c210751bf8840f588 /tests/handlers/test_password_providers.py | |
parent | Various clean-ups to the logging context code (#8935) (diff) | |
download | synapse-6d02eb22dfde9551c515acaf73503e2500e00eaf.tar.xz |
Fix startup failure with localdb_enabled: False (#8937)
Diffstat (limited to 'tests/handlers/test_password_providers.py')
-rw-r--r-- | tests/handlers/test_password_providers.py | 23 |
1 files changed, 23 insertions, 0 deletions
diff --git a/tests/handlers/test_password_providers.py b/tests/handlers/test_password_providers.py index ceaf0902d2..8d50265145 100644 --- a/tests/handlers/test_password_providers.py +++ b/tests/handlers/test_password_providers.py @@ -432,6 +432,29 @@ class PasswordAuthProviderTests(unittest.HomeserverTestCase): @override_config( { + **providers_config(CustomAuthProvider), + "password_config": {"enabled": False, "localdb_enabled": False}, + } + ) + def test_custom_auth_password_disabled_localdb_enabled(self): + """Check the localdb_enabled == enabled == False + + Regression test for https://github.com/matrix-org/synapse/issues/8914: check + that setting *both* `localdb_enabled` *and* `password: enabled` to False doesn't + cause an exception. + """ + self.register_user("localuser", "localpass") + + flows = self._get_login_flows() + self.assertEqual(flows, [{"type": "test.login_type"}] + ADDITIONAL_LOGIN_FLOWS) + + # login shouldn't work and should be rejected with a 400 ("unknown login type") + channel = self._send_password_login("localuser", "localpass") + self.assertEqual(channel.code, 400, channel.result) + mock_password_provider.check_auth.assert_not_called() + + @override_config( + { **providers_config(PasswordCustomAuthProvider), "password_config": {"enabled": False}, } |