clinicedc 2.0.7__py3-none-any.whl → 2.0.8__py3-none-any.whl

edc_auth/auth_objects/__init__.py

@@ -1,21 +1,3 @@
-from ..constants import (
-    ACCOUNT_MANAGER,
-    ACCOUNT_MANAGER_ROLE,
-    ADMINISTRATION,
-    AUDITOR,
-    AUDITOR_ROLE,
-    CELERY_MANAGER,
-    CLINIC,
-    CLINIC_SUPER,
-    CLINICIAN_ROLE,
-    CLINICIAN_SUPER_ROLE,
-    CUSTOM_ROLE,
-    EVERYONE,
-    NURSE_ROLE,
-    PII,
-    PII_VIEW,
-    STAFF_ROLE,
-)
 from .codenames import (
     account_manager,
     administration,
@@ -25,6 +7,6 @@ from .codenames import (
     clinic_super,
     everyone,
 )
-from .default_groups import default_groups
+from .default_groups import get_default_groups
 from .default_pii_models import default_pii_models
-from .default_roles import default_roles
+from .default_roles import get_default_roles

edc_auth/auth_objects/default_groups.py

@@ -11,14 +11,16 @@ from ..constants import (
 )
 from .codenames import account_manager, administration, celery_manager, everyone
 
-default_groups = {
-    ACCOUNT_MANAGER: account_manager,
-    ADMINISTRATION: administration,
-    AUDITOR: [],
-    CELERY_MANAGER: celery_manager,
-    CLINIC: [],
-    CLINIC_SUPER: [],
-    EVERYONE: everyone,
-    PII: [],
-    PII_VIEW: [],
-}
+
+def get_default_groups():
+    return {
+        ACCOUNT_MANAGER: account_manager,
+        ADMINISTRATION: administration,
+        AUDITOR: [],
+        CELERY_MANAGER: celery_manager,
+        CLINIC: [],
+        CLINIC_SUPER: [],
+        EVERYONE: everyone,
+        PII: [],
+        PII_VIEW: [],
+    }

edc_auth/auth_objects/default_roles.py

@@ -17,27 +17,29 @@ from ..constants import (
     STATISTICIAN,
 )
 
-# Format {ROLE_NAME: [GROUP_NAME, GROUP_NAME, ...]
-default_roles: dict[str, list[str]] = {
-    ACCOUNT_MANAGER_ROLE: [ACCOUNT_MANAGER],
-    AUDITOR_ROLE: [
-        AUDITOR,
-        PII_VIEW,
-    ],
-    CLINICIAN_ROLE: [
-        CLINIC,
-        PII,
-    ],
-    CLINICIAN_SUPER_ROLE: [
-        CLINIC_SUPER,
-        CLINIC,
-        PII,
-    ],
-    CUSTOM_ROLE: [],
-    NURSE_ROLE: [
-        CLINIC,
-        PII,
-    ],
-    STAFF_ROLE: [ADMINISTRATION, EVERYONE],
-    STATISTICIAN: [AUDITOR],
-}
+
+def get_default_roles() -> dict[str, list[str]]:
+    """Format {ROLE_NAME: [GROUP_NAME, GROUP_NAME, ...]"""
+    return {
+        ACCOUNT_MANAGER_ROLE: [ACCOUNT_MANAGER],
+        AUDITOR_ROLE: [
+            AUDITOR,
+            PII_VIEW,
+        ],
+        CLINICIAN_ROLE: [
+            CLINIC,
+            PII,
+        ],
+        CLINICIAN_SUPER_ROLE: [
+            CLINIC_SUPER,
+            CLINIC,
+            PII,
+        ],
+        CUSTOM_ROLE: [],
+        NURSE_ROLE: [
+            CLINIC,
+            PII,
+        ],
+        STAFF_ROLE: [ADMINISTRATION, EVERYONE],
+        STATISTICIAN: [AUDITOR],
+    }

edc_auth/auth_updater/auth_updater.py

@@ -9,6 +9,7 @@ from django.contrib.auth import get_user_model
 from django.core.exceptions import ObjectDoesNotExist
 from django.core.management.color import color_style
 
+from ..constants import POST_UPDATE_FUNCS_KEY, PRE_UPDATE_FUNCS_KEY
 from ..site_auths import site_auths
 from .group_updater import GroupUpdater
 from .role_updater import RoleUpdater
@@ -17,6 +18,14 @@ style = color_style()
 
 
 class AuthUpdater:
+    """A class to update auth.Group, edc_auth.Role, auth.Permissions
+    models using the site_auth registry.
+
+    Called once on application startup. For example::
+
+        AuthUpdater(verbose=False, warn_only=True)
+    """
+
     group_updater_cls = GroupUpdater
     role_updater_cls = RoleUpdater
 
@@ -38,8 +47,10 @@ class AuthUpdater:
         )
         groups = groups or site_auths.groups
         pii_models = pii_models or site_auths.pii_models
-        post_update_funcs = post_update_funcs or site_auths.post_update_funcs
-        pre_update_funcs = pre_update_funcs or site_auths.pre_update_funcs
+        post_update_funcs = post_update_funcs or [f for f in site_auths.post_update_funcs]
+        site_auths.registry[POST_UPDATE_FUNCS_KEY] = []
+        pre_update_funcs = pre_update_funcs or [f for f in site_auths.pre_update_funcs]
+        site_auths.registry[PRE_UPDATE_FUNCS_KEY] = []
         roles = roles or site_auths.roles
         self.apps = apps
         if not self.edc_auth_skip_auth_updater:

edc_auth/auth_updater/group_updater.py

@@ -11,7 +11,7 @@ from django.core.exceptions import (
 )
 from django.core.management.color import color_style
 
-from ..auth_objects import PII, PII_VIEW
+from ..constants import PII, PII_VIEW
 from ..utils import make_view_only_group_permissions
 
 style = color_style()
@@ -27,7 +27,7 @@ class PermissionsCreatorError(ValidationError):
     pass
 
 
-class CodenameDoesNotExist(Exception):
+class CodenameDoesNotExist(Exception):  # noqa: N818
     pass
 
 
@@ -87,7 +87,7 @@ class GroupUpdater:
             group = self.group_model_cls.objects.get(name=group_name)
         except ObjectDoesNotExist as e:
             if not create_group:
-                raise ObjectDoesNotExist(f"{e} Got {group_name}")
+                raise ObjectDoesNotExist(f"{e} Got {group_name}") from e
             group = self.group_model_cls.objects.create(name=group_name)
         else:
             group.permissions.clear()
@@ -122,7 +122,7 @@ class GroupUpdater:
             try:
                 app_label, codename = self.get_from_dotted_codename(dotted_codename)
             except PermissionsCodenameError as e:
-                warn(str(e))
+                warn(str(e), stacklevel=2)
             else:
                 # if you add extra codenames you must also add custom
                 # codename tuples to the Permissions model before you
@@ -138,7 +138,7 @@ class GroupUpdater:
                 except ObjectDoesNotExist as e:
                     errmsg = f"{e} Got codename={codename},app_label={app_label}"
                     if not self.warn_only:
-                        raise CodenameDoesNotExist(errmsg)
+                        raise CodenameDoesNotExist(errmsg) from e
                     warn(style.ERROR(errmsg))
                 except MultipleObjectsReturned as e:
                     if not allow_multiple_objects:
@@ -166,15 +166,17 @@ class GroupUpdater:
         try:
             app_label, _codename = codename.split(".")
         except ValueError as e:
-            raise PermissionsCodenameError(f"Invalid dotted codename. {e} Got {codename}.")
+            raise PermissionsCodenameError(
+                f"Invalid dotted codename. {e} Got {codename}."
+            ) from e
         else:
             try:
                 self.apps.get_app_config(app_label)
-            except LookupError:
+            except LookupError as e:
                 raise PermissionsCodenameError(
                     "Invalid app_label in codename. Expected format "
                     f"'<app_label>.<some_codename>'. Got {codename}."
-                )
+                ) from e
         prefix = _codename.split("_")[0]
         if prefix not in self.codename_prefixes:
             raise PermissionsCodenameError(
@@ -244,7 +246,7 @@ class GroupUpdater:
         except ObjectDoesNotExist as e:
             raise CodenameDoesNotExist(
                 f"Unable to verify codename. {e} Got '{app_label}.{codename}'"
-            )
+            ) from e
         except MultipleObjectsReturned as e:
             self.delete_and_raise_on_duplicate_codenames(codename, app_label, exception=e)
         return permission
@@ -267,7 +269,7 @@ class GroupUpdater:
         try:
             value, name = codename_tpl
         except ValueError as e:
-            raise ValueError(f"{e} Got {codename_tpl}")
+            raise ValueError(f"{e} Got {codename_tpl}") from e
         _app_label, codename = value.split(".")
         if app_label and _app_label != app_label:
             raise PermissionsCreatorError(

edc_auth/auth_updater/role_updater.py

@@ -56,11 +56,11 @@ class RoleUpdater:
             for group_name in groups:
                 try:
                     group = self.group_model_cls.objects.get(name=group_name)
-                except ObjectDoesNotExist:
+                except ObjectDoesNotExist as e:
                     raise RoleUpdaterError(
                         "Invalid group specified for role. "
                         f"`{group_name}` is not a group. See role `{role}`."
-                    )
+                    ) from e
                     # group = self.group_model_cls.objects.create(name=group_name)
                 role.groups.add(group)
             index += 1

edc_auth/constants.py

@@ -19,3 +19,13 @@ CUSTOM_ROLE = "custom"
 NURSE_ROLE = "research_nurse"
 STAFF_ROLE = "staff"
 STATISTICIAN = "statistician"
+
+# keys for the site_auths registry
+UPDATE_ROLES_KEY = "update_roles"
+UPDATE_GROUPS_KEY = "update_groups"
+PRE_UPDATE_FUNCS_KEY = "pre_update_funcs"
+POST_UPDATE_FUNCS_KEY = "post_update_funcs"
+GROUPS_KEY = "groups"
+ROLES_KEY = "roles"
+PII_MODELS_KEY = "pii_models"
+CUSTOM_PERMISSIONS_TUPLES_KEY = "custom_permissions_tuples"

edc_auth/import_users.py

@@ -183,9 +183,9 @@ class UserImporter:
         self.update_user_sites(site_names or [])
         self.update_user_roles(role_names or [STAFF_ROLE])
         self.user.save()
-        self.user.userprofile.job_title = self.job_title
-        self.user.userprofile.mobile = self.mobile
-        self.user.userprofile.alternate_email = self.alternate_email
+        self.user.userprofile.job_title = self.job_title or ""
+        self.user.userprofile.mobile = self.mobile or ""
+        self.user.userprofile.alternate_email = self.alternate_email or ""
         self.user.userprofile.save()
 
         self.site_names = (

edc_auth/models/user_profile.py

@@ -11,7 +11,7 @@ from edc_export.choices import EXPORT_FORMATS
 from edc_export.constants import CSV
 from edc_notification.model_mixins import NotificationUserProfileModelMixin
 
-from ..auth_objects import CUSTOM_ROLE, STAFF_ROLE
+from ..constants import CUSTOM_ROLE, STAFF_ROLE
 from .role import Role
 
 
@@ -32,21 +32,21 @@ class UserProfile(NotificationUserProfileModelMixin, models.Model):
 
     sites = models.ManyToManyField(Site, blank=True)
 
-    job_title = models.CharField(max_length=100, null=True, blank=True)
+    job_title = models.CharField(max_length=100, default="", blank=True)
 
-    alternate_email = models.EmailField(_("Alternate email address"), blank=True, null=True)
+    alternate_email = models.EmailField(_("Alternate email address"), blank=True, default="")
 
     mobile = models.CharField(
         max_length=25,
         validators=[RegexValidator(regex=r"^\+\d+")],
-        null=True,
+        default="",
         blank=True,
         help_text="e.g. +1234567890",
     )
 
     clinic_label_printer = models.CharField(
         max_length=100,
-        null=True,
+        default="",
         blank=True,
         help_text=format_html(
             'Change in <a href="{href}">{label}</a>',
@@ -57,7 +57,7 @@ class UserProfile(NotificationUserProfileModelMixin, models.Model):
 
     lab_label_printer = models.CharField(
         max_length=100,
-        null=True,
+        default="",
         blank=True,
         help_text=format_html(
             'Change in <a href="{href}">{label}</a>',
@@ -68,7 +68,7 @@ class UserProfile(NotificationUserProfileModelMixin, models.Model):
 
     print_server = models.CharField(
         max_length=100,
-        null=True,
+        default="",
         blank=True,
         help_text=format_html(
             'Change in <a href="{href}">{label}</a>',
@@ -82,7 +82,6 @@ class UserProfile(NotificationUserProfileModelMixin, models.Model):
         max_length=25,
         choices=EXPORT_FORMATS,
         default=CSV,
-        null=True,
         blank=True,
         help_text="Note: requires export permissions",
     )
@@ -101,9 +100,13 @@ class UserProfile(NotificationUserProfileModelMixin, models.Model):
             group_names = [group.name for group in self.user.groups.all()]
             add_group_names = []
             for role in self.roles.all():
-                for group in role.groups.all():
-                    if group.name not in group_names:
-                        add_group_names.append(group.name)
+                add_group_names.extend(
+                    [
+                        group.name
+                        for group in role.groups.all()
+                        if group.name not in group_names
+                    ]
+                )
             add_group_names = list(set(add_group_names))
             for name in add_group_names:
                 self.user.groups.add(Group.objects.get(name=name))

edc_auth/site_auths.py

@@ -9,30 +9,40 @@ from django.apps import apps as django_apps
 from django.conf import settings
 from django.utils.module_loading import import_module, module_has_submodule
 
-from .auth_objects import default_groups, default_pii_models, default_roles
-
-
-class AlreadyRegistered(Exception):
+from .auth_objects import default_pii_models, get_default_groups, get_default_roles
+from .constants import (
+    CUSTOM_PERMISSIONS_TUPLES_KEY,
+    GROUPS_KEY,
+    PII_MODELS_KEY,
+    POST_UPDATE_FUNCS_KEY,
+    PRE_UPDATE_FUNCS_KEY,
+    ROLES_KEY,
+    UPDATE_GROUPS_KEY,
+    UPDATE_ROLES_KEY,
+)
+
+
+class AlreadyRegistered(Exception):  # noqa: N818
     pass
 
 
-class InvalidGroup(Exception):
+class InvalidGroup(Exception):  # noqa: N818
     pass
 
 
-class InvalidRole(Exception):
+class InvalidRole(Exception):  # noqa: N818
     pass
 
 
-class RoleAlreadyExists(Exception):
+class RoleAlreadyExists(Exception):  # noqa: N818
     pass
 
 
-class GroupAlreadyExists(Exception):
+class GroupAlreadyExists(Exception):  # noqa: N818
     pass
 
 
-class PiiModelAlreadyExists(Exception):
+class PiiModelAlreadyExists(Exception):  # noqa: N818
     pass
 
 
@@ -96,39 +106,39 @@ class SiteAuths:
 
     def initialize(self):
         self.registry = {
-            "groups": default_groups,
-            "roles": default_roles,
-            "update_groups": {},
-            "update_roles": {},
-            "custom_permissions_tuples": {},
-            "pre_update_funcs": [],
-            "post_update_funcs": [],
-            "pii_models": default_pii_models,
+            GROUPS_KEY: get_default_groups(),
+            ROLES_KEY: get_default_roles(),
+            UPDATE_GROUPS_KEY: {},
+            UPDATE_ROLES_KEY: {},
+            CUSTOM_PERMISSIONS_TUPLES_KEY: {},
+            PRE_UPDATE_FUNCS_KEY: [],
+            POST_UPDATE_FUNCS_KEY: [],
+            PII_MODELS_KEY: default_pii_models,
         }
 
     def clear(self):
         self.registry = {
-            "groups": {},
-            "roles": {},
-            "update_groups": {},
-            "update_roles": {},
-            "custom_permissions_tuples": {},
-            "pre_update_funcs": [],
-            "post_update_funcs": [],
-            "pii_models": [],
+            GROUPS_KEY: {},
+            ROLES_KEY: {},
+            UPDATE_GROUPS_KEY: {},
+            UPDATE_ROLES_KEY: {},
+            CUSTOM_PERMISSIONS_TUPLES_KEY: {},
+            PRE_UPDATE_FUNCS_KEY: [],
+            POST_UPDATE_FUNCS_KEY: [],
+            PII_MODELS_KEY: [],
         }
 
     def clear_values(self):
         registry = deepcopy(self.registry)
         self.registry = {
-            "groups": {k: [] for k in registry.get("groups")},
-            "roles": {k: [] for k in self.registry.get("roles")},
-            "update_groups": {},
-            "update_roles": {},
-            "custom_permissions_tuples": {},
-            "pre_update_funcs": [],
-            "post_update_funcs": [],
-            "pii_models": [],
+            GROUPS_KEY: {k: [] for k in registry.get(GROUPS_KEY)},
+            ROLES_KEY: {k: [] for k in self.registry.get(ROLES_KEY)},
+            UPDATE_GROUPS_KEY: {},
+            UPDATE_ROLES_KEY: {},
+            CUSTOM_PERMISSIONS_TUPLES_KEY: {},
+            PRE_UPDATE_FUNCS_KEY: [],
+            POST_UPDATE_FUNCS_KEY: [],
+            PII_MODELS_KEY: [],
         }
 
     @property
@@ -136,15 +146,15 @@ class SiteAuths:
         return getattr(settings, "EDC_AUTH_SKIP_SITE_AUTHS", False)
 
     def add_pre_update_func(self, func):
-        self.registry["pre_update_funcs"].append(func)
+        self.registry[PRE_UPDATE_FUNCS_KEY].append(func)
 
     def add_post_update_func(self, app_label: str, func: Callable):
-        self.registry["post_update_funcs"].append((app_label, func))
+        self.registry[POST_UPDATE_FUNCS_KEY].append((app_label, func))
 
     def add_pii_model(self, model_name):
-        if model_name in self.registry["pii_models"]:
+        if model_name in self.registry[PII_MODELS_KEY]:
             raise PiiModelAlreadyExists(f"PII model already exists. Got {model_name}")
-        self.registry["pii_models"].append(model_name)
+        self.registry[PII_MODELS_KEY].append(model_name)
 
     def add_groups(self, data: dict):
         for name, codenames in data.items():
@@ -162,7 +172,7 @@ class SiteAuths:
         convert_to_export=None,
         no_delete=None,
     ):
-        if name in self.registry["groups"]:
+        if name in self.registry[GROUPS_KEY]:
             raise GroupAlreadyExists(f"Group name already exists. Got {name}.")
         if no_delete:
             codenames_or_func = self.remove_delete_codenames(codenames_or_func)
@@ -170,18 +180,18 @@ class SiteAuths:
             codenames_or_func = self.get_view_only_codenames(codenames_or_func)
         if convert_to_export:
             codenames_or_func = self.convert_to_export_codenames(codenames_or_func)
-        self.registry["groups"].update({name: codenames_or_func})
+        self.registry[GROUPS_KEY].update({name: codenames_or_func})
 
     def add_role(self, *group_names, name=None):
-        if name in self.registry["roles"]:
+        if name in self.registry[ROLES_KEY]:
             raise RoleAlreadyExists(f"Role name already exists. Got {name}.")
         group_names = list(set(group_names))
-        self.registry["roles"].update({name: group_names})
+        self.registry[ROLES_KEY].update({name: group_names})
 
     def update_group(
         self, *codenames_or_func, name=None, key=None, view_only=None, no_delete=None
     ) -> None:
-        key = key or "update_groups"
+        key = key or UPDATE_GROUPS_KEY
         if no_delete:
             codenames_or_func = self.remove_delete_codenames(codenames_or_func)
         if view_only:
@@ -191,30 +201,30 @@ class SiteAuths:
         try:
             existing_codenames = list(set(existing_codenames))
         except TypeError as e:
-            raise TypeError(f"{e}. Got {name}")
+            raise TypeError(f"{e}. Got {name}") from e
         existing_codenames.extend(codenames_or_func)
         existing_codenames = list(set(existing_codenames))
         self.registry[key].update({name: existing_codenames})
 
     def update_role(self, *group_names, name=None, key=None) -> None:
-        key = key or "update_roles"
+        key = key or UPDATE_ROLES_KEY
         group_names = list(set(group_names))
-        existing_group_names = deepcopy(self.registry[key].get(name)) or []
-        existing_group_names = list(set(existing_group_names))
+        existing_group_names = [
+            name for name in self.registry[key].get(name) or [] if name not in group_names
+        ]
         existing_group_names.extend(group_names)
-        existing_group_names = list(set(existing_group_names))
         self.registry[key].update({name: existing_group_names})
 
     def add_custom_permissions_tuples(
         self, model: str, codename_tuples: tuple[tuple[str, str], ...]
     ):
         try:
-            self.registry["custom_permissions_tuples"][model]
+            self.registry[CUSTOM_PERMISSIONS_TUPLES_KEY][model]
         except KeyError:
-            self.registry["custom_permissions_tuples"].update({model: []})
+            self.registry[CUSTOM_PERMISSIONS_TUPLES_KEY].update({model: []})
         for codename_tuple in codename_tuples:
-            if codename_tuple not in self.registry["custom_permissions_tuples"][model]:
-                self.registry["custom_permissions_tuples"][model].append(codename_tuple)
+            if codename_tuple not in self.registry[CUSTOM_PERMISSIONS_TUPLES_KEY][model]:
+                self.registry[CUSTOM_PERMISSIONS_TUPLES_KEY][model].append(codename_tuple)
 
     @staticmethod
     def get_view_only_codenames(codenames):
@@ -225,7 +235,8 @@ class SiteAuths:
         Does not remove `edc_navbar`, 'nav_' or `edc_dashboard`
         codenames.
         """
-        callables = [lambda: view_only_wrapper(c) for c in codenames if callable(c)]
+        # callables = [lambda: view_only_wrapper(c) for c in codenames if callable(c)]
+        callables = [lambda c=c: view_only_wrapper(c) for c in codenames if callable(c)]
         view_only_codenames = [
             codename
             for codename in codenames
@@ -263,27 +274,27 @@ class SiteAuths:
 
     @property
     def roles(self):
-        return self.registry["roles"]
+        return self.registry[ROLES_KEY]
 
     @property
     def groups(self):
-        return self.registry["groups"]
+        return self.registry[GROUPS_KEY]
 
     @property
     def pii_models(self):
-        return self.registry["pii_models"]
+        return self.registry[PII_MODELS_KEY]
 
     @property
     def pre_update_funcs(self):
-        return self.registry["pre_update_funcs"]
+        return self.registry[PRE_UPDATE_FUNCS_KEY]
 
     @property
     def post_update_funcs(self) -> tuple[str, Callable]:
-        return self.registry["post_update_funcs"]
+        return self.registry[POST_UPDATE_FUNCS_KEY]
 
     @property
     def custom_permissions_tuples(self):
-        return self.registry["custom_permissions_tuples"]
+        return self.registry[CUSTOM_PERMISSIONS_TUPLES_KEY]
 
     def verify_and_populate(
         self, app_name: str | None = None, warn_only: bool | None = None
@@ -294,28 +305,30 @@ class SiteAuths:
         * Updates data from `update_groups` -> `groups`
         * Updates data from `update_roles` -> `roles`
         """
-        for name, codenames in self.registry["update_groups"].items():
-            if name not in self.registry["groups"]:
+        for name, codenames in self.registry[UPDATE_GROUPS_KEY].items():
+            if name not in self.registry[GROUPS_KEY]:
                 msg = (
                     f"Cannot update group. Group name does not exist. See app={app_name}"
                     f"update_groups['groups']={codenames}. Got {name}"
                 )
                 if warn_only:
-                    warn(msg)
+                    warn(msg, stacklevel=2)
                 else:
                     raise InvalidGroup(msg)
-            self.update_group(*codenames, name=name, key="groups")
-        for name, group_names in self.registry["update_roles"].items():
-            if name not in self.registry["roles"]:
+            self.update_group(*codenames, name=name, key=GROUPS_KEY)
+        self.registry[UPDATE_GROUPS_KEY] = {}
+        for name, group_names in self.registry[UPDATE_ROLES_KEY].items():
+            if name not in self.registry[ROLES_KEY]:
                 msg = (
                     f"Cannot update role. Role name does not exist. See app={app_name}. "
                     f"update_roles['groups']={group_names}. Got {name}"
                 )
                 if warn_only:
-                    warn(msg)
+                    warn(msg, stacklevel=2)
                 else:
                     raise InvalidRole(msg)
-            self.update_role(*group_names, name=name, key="roles")
+            self.update_role(*group_names, name=name, key=ROLES_KEY)
+        self.registry[UPDATE_ROLES_KEY] = {}
 
     def autodiscover(self, module_name=None, verbose=True):
         """Autodiscovers in the auths.py file of any INSTALLED_APP."""
@@ -335,7 +348,7 @@ class SiteAuths:
                     except ImportError as e:
                         site_auths.registry = before_import_registry
                         if module_has_submodule(mod, module_name):
-                            raise SiteAuthError(str(e))
+                            raise SiteAuthError(str(e)) from e
                 except ImportError:
                     pass
             self.verify_and_populate(app_name=app_name)