fix: linting issues

Signed-off-by: Nicolas <32845761+nicoklaus@users.noreply.github.com>

Author: nicoklaus

dejacode/settings.py

@@ -17,6 +17,7 @@
 import ldap
 from django_auth_ldap.config import GroupOfNamesType
 from django_auth_ldap.config import LDAPSearch
+
 from dejacode_toolkit.ldap import build_user_search
 
 # The home directory of the dejacode user that owns the installation.

dejacode_toolkit/ldap.py

@@ -8,14 +8,16 @@
 
 import json
 
-import ldap
 from django.core.exceptions import ImproperlyConfigured
+
+import ldap
 from django_auth_ldap.config import LDAPSearch
 from django_auth_ldap.config import LDAPSearchUnion
 
 
 def build_user_search(user_searches, user_dn, user_filterstr):
-    """Return the ``AUTH_LDAP_USER_SEARCH`` object.
+    """
+    Return the ``AUTH_LDAP_USER_SEARCH`` object.
 
     When ``user_searches`` (a raw JSON string) is provided, parse and validate
     it and return an ``LDAPSearchUnion``. Otherwise, fall back to a single
@@ -27,9 +29,7 @@ def build_user_search(user_searches, user_dn, user_filterstr):
     try:
         definitions = json.loads(user_searches)
     except json.JSONDecodeError as e:
-        raise ImproperlyConfigured(
-            f"Invalid JSON in AUTH_LDAP_USER_SEARCHES: {e}"
-        ) from e
+        raise ImproperlyConfigured(f"Invalid JSON in AUTH_LDAP_USER_SEARCHES: {e}") from e
 
     if not isinstance(definitions, list):
         raise ImproperlyConfigured("AUTH_LDAP_USER_SEARCHES must be a JSON list")
@@ -40,9 +40,7 @@ def build_user_search(user_searches, user_dn, user_filterstr):
     searches = []
     for index, entry in enumerate(definitions):
         if not isinstance(entry, dict):
-            raise ImproperlyConfigured(
-                f"AUTH_LDAP_USER_SEARCHES[{index}] must be a JSON object"
-            )
+            raise ImproperlyConfigured(f"AUTH_LDAP_USER_SEARCHES[{index}] must be a JSON object")
 
         base_dn = entry.get("base")
         filterstr = entry.get("filter")

dje/tests/test_ldap_config.py

@@ -12,7 +12,7 @@
 from django_auth_ldap.config import LDAPSearch
 from django_auth_ldap.config import LDAPSearchUnion
 
-from dejacode.ldap_config import build_user_search
+from dejacode_toolkit.ldap import build_user_search
 
 
 class BuildUserSearchTestCase(SimpleTestCase):
@@ -51,24 +51,18 @@ def test_build_user_search_entry_not_object_raises(self):
 
     def test_build_user_search_missing_base_raises(self):
         raw = '[{"filter": "(uid=%(user)s)"}]'
-        with self.assertRaisesMessage(
-            ImproperlyConfigured, "[0] must define 'base' and 'filter'"
-        ):
+        with self.assertRaisesMessage(ImproperlyConfigured, "[0] must define 'base' and 'filter'"):
             build_user_search(raw, "", "")
 
     def test_build_user_search_missing_filter_raises(self):
         raw = '[{"base": "ou=a,dc=example,dc=com"}]'
-        with self.assertRaisesMessage(
-            ImproperlyConfigured, "[0] must define 'base' and 'filter'"
-        ):
+        with self.assertRaisesMessage(ImproperlyConfigured, "[0] must define 'base' and 'filter'"):
             build_user_search(raw, "", "")
 
     def test_build_user_search_error_index_points_to_bad_entry(self):
         raw = (
             '[{"base": "ou=a,dc=example,dc=com", "filter": "(uid=%(user)s)"},'
             ' {"base": "ou=b,dc=example,dc=com"}]'
         )
-        with self.assertRaisesMessage(
-            ImproperlyConfigured, "[1] must define 'base' and 'filter'"
-        ):
+        with self.assertRaisesMessage(ImproperlyConfigured, "[1] must define 'base' and 'filter'"):
             build_user_search(raw, "", "")