Skip to content
Snippets Groups Projects
Commit 27a0c21c authored by Christoph Witzany's avatar Christoph Witzany
Browse files

make tests for ldap more specific to not be fooled by Mocks

parent 3555a659
No related branches found
No related tags found
No related merge requests found
...@@ -58,7 +58,7 @@ class AuthHandler(BaseHandler): ...@@ -58,7 +58,7 @@ class AuthHandler(BaseHandler):
self.ldap_email_property = hs.config.ldap_email_property self.ldap_email_property = hs.config.ldap_email_property
self.ldap_full_name_property = hs.config.ldap_full_name_property self.ldap_full_name_property = hs.config.ldap_full_name_property
if self.ldap_enabled: if self.ldap_enabled is True:
import ldap import ldap
logger.info("Import ldap version: %s", ldap.__version__) logger.info("Import ldap version: %s", ldap.__version__)
...@@ -444,8 +444,8 @@ class AuthHandler(BaseHandler): ...@@ -444,8 +444,8 @@ class AuthHandler(BaseHandler):
@defer.inlineCallbacks @defer.inlineCallbacks
def _check_ldap_password(self, user_id, password): def _check_ldap_password(self, user_id, password):
if not self.ldap_enabled: if self.ldap_enabled is not True:
logger.info("LDAP not configured") logger.debug("LDAP not configured")
defer.returnValue(False) defer.returnValue(False)
import ldap import ldap
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment