Remove leftover print

snok · Aug 14, 2021 · 4210666 · 4210666
1 parent f9e7319
commit 4210666
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 5 deletions.
diff --git a/CHANGELOG.rst b/CHANGELOG.rst
@@ -5,6 +5,7 @@ Changelog
 ---------------------
 
 **Features**
+
 * ``GROUP_TO_FLAG_MAPPING`` can now be a list of groups. @joachimBurket #104
 
 

diff --git a/django_auth_adfs/config.py b/django_auth_adfs/config.py
@@ -133,10 +133,8 @@ def __init__(self):
             self.CLAIM_MAPPING = {"first_name": "given_name",
                                   "last_name": "family_name",
                                   "email": "email"}
-        else:
-            if "VERSION" in _settings:
-                print('got here')
-                raise ImproperlyConfigured("The VERSION cannot be set when TENANT_ID is set.")
+        elif "VERSION" in _settings:
+            raise ImproperlyConfigured("The VERSION cannot be set when TENANT_ID is set.")
 
         # Overwrite defaults with user settings
         for setting, value in _settings.items():
@@ -165,7 +163,7 @@ def __init__(self):
 
         # Validate setting conflicts
         usermodel = get_user_model()
-        if usermodel.USERNAME_FIELD in self.CLAIM_MAPPING.keys():
+        if usermodel.USERNAME_FIELD in self.CLAIM_MAPPING:
             raise ImproperlyConfigured("You cannot set the username field of the user model from "
                                        "the CLAIM_MAPPING setting. Instead use the USERNAME_CLAIM setting.")