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

edc_pharmacy/utils/confirm_stock_at_site.py

@@ -26,7 +26,7 @@ def confirm_stock_at_site(
     stock_transfer: StockTransfer,
     stock_codes: list[str],
     location: UUID,
-    request: WSGIRequest = None,
+    request: WSGIRequest | None = None,
 ) -> tuple[list[str], list[str], list[str]]:
     """Confirm stock instances given a list of stock codes
     and a request/receive pk.

edc_pharmacy/views/confirmation_at_site_view.py

@@ -1,5 +1,6 @@
 from __future__ import annotations
 
+import contextlib
 import uuid
 from uuid import uuid4
 
@@ -97,13 +98,12 @@ class ConfirmationAtSiteView(
         return min(unconfirmed_count, 12)
 
     def get_stock_codes(self, stock_transfer):
-        stock_codes = [
+        return [
             code
             for code in stock_transfer.stocktransferitem_set.values_list(
                 "stock__code", flat=True
             ).all()
         ]
-        return stock_codes
 
     def get_unconfirmed_count(self, stock_transfer) -> int:
         return (
@@ -120,10 +120,8 @@ class ConfirmationAtSiteView(
     def site(self) -> Site | None:
         obj = None
         if self.kwargs.get("site_id"):
-            try:
+            with contextlib.suppress(ObjectDoesNotExist):
                 obj = Site.objects.get(id=self.kwargs.get("site_id"))
-            except ObjectDoesNotExist:
-                pass
         return obj
 
     @property
@@ -169,14 +167,13 @@ class ConfirmationAtSiteView(
     def confirmation_at_site_changelist_url(self) -> str:
         if self.confirmation_at_site:
             url = reverse("edc_pharmacy_admin:edc_pharmacy_confirmationatsite_changelist")
-            url = f"{url}?q={self.confirmation_at_site.transfer_confirmation_identifier}"
-            return url
+            return f"{url}?q={self.confirmation_at_site.transfer_confirmation_identifier}"
         return "/"
 
     def get_stock_transfer(
         self,
         stock_transfer_identifier: str,
-        suppress_msg: bool = None,
+        suppress_msg: bool | None = None,
     ) -> StockTransfer | None:
         stock_transfer = None
         try:
@@ -198,7 +195,7 @@ class ConfirmationAtSiteView(
                 )
         return stock_transfer
 
-    def post(self, request, *args, **kwargs) -> HttpResponseRedirect:
+    def post(self, request, *args, **kwargs) -> HttpResponseRedirect:  # noqa: ARG002
         # cancel
         if request.POST.get("cancel") and request.POST.get("cancel") == "cancel":
             url = reverse("edc_pharmacy:home_url")

edc_prn/admin_site.py

@@ -0,0 +1,5 @@
+from edc_model_admin.admin_site import EdcAdminSite
+
+from .apps import AppConfig
+
+edc_prn_admin = EdcAdminSite(name="edc_prn_admin", app_label=AppConfig.name)

edc_prn/prn.py

@@ -58,19 +58,18 @@ class Prn:
         try:
             return django_apps.get_model(self.model)
         except LookupError as e:
-            raise PrnError(f"{e}. See {self!r}")
+            raise PrnError(f"{e}. See {self!r}") from e
 
-    def get_show_on_dashboard(self, subject_identifier=None, **kwargs):
+    def get_show_on_dashboard(self, subject_identifier=None, **kwargs):  # noqa: ARG002
         count = 0
-        if self.show_on_dashboard:
-            if subject_identifier:
-                opts = dict(subject_identifier=subject_identifier)
-                try:
-                    count = self.model_cls.objects.filter(**opts).count()
-                except FieldError:
-                    opts = self.get_query_opts(subject_identifier)
-                    count = self.model_cls.objects.filter(**opts).count()
-        return True if count and self.show_on_dashboard else False
+        if self.show_on_dashboard and subject_identifier:
+            opts = dict(subject_identifier=subject_identifier)
+            try:
+                count = self.model_cls.objects.filter(**opts).count()
+            except FieldError:
+                opts = self.get_query_opts(subject_identifier)
+                count = self.model_cls.objects.filter(**opts).count()
+        return count and self.show_on_dashboard
 
     def get_query_opts(self, subject_identifier):
         """Returns alternative query opts to search on

edc_prn/urls.py

@@ -0,0 +1,11 @@
+from django.urls import path
+from django.views.generic.base import RedirectView
+
+from .admin_site import edc_prn_admin
+
+app_name = "edc_prn"
+
+urlpatterns = [
+    path("admin/", edc_prn_admin.urls),
+    path("", RedirectView.as_view(url="admin/"), name="home_url"),
+]

edc_protocol_incident/action_items.py

@@ -9,8 +9,8 @@ from .constants import (
 
 
 class ProtocolDeviationViolationAction(ActionWithNotification):
-    reference_model = None  # "my_app.protocolincident"
-    admin_site_name = None  # "my_app_admin"
+    reference_model = "edc_protocol_incident.protocoldeviationviolation"
+    admin_site_name = "edc_protocol_incident_admin"
 
     name = PROTOCOL_DEVIATION_VIOLATION_ACTION
     display_name = "Submit Protocol Deviation / Violation Report"
@@ -25,8 +25,8 @@ class ProtocolDeviationViolationAction(ActionWithNotification):
 
 
 class ProtocolIncidentAction(ActionWithNotification):
-    reference_model = None  # "my_app.protocolincident"
-    admin_site_name = None  # "my_app"
+    reference_model = "edc_protocol_incident.protocolincident"
+    admin_site_name = "edc_protocol_incident_admin"
 
     name = PROTOCOL_INCIDENT_ACTION
     display_name = "Submit Protocol Incident Report"

edc_protocol_incident/auths.py

@@ -14,24 +14,31 @@ from .auth_objects import (
 )
 from .constants import PROTOCOL_DEVIATION_VIOLATION, PROTOCOL_INCIDENT
 
-incident_type = getattr(settings, "EDC_PROTOCOL_VIOLATION_TYPE", PROTOCOL_DEVIATION_VIOLATION)
-
-site_auths.add_group(*protocol_violation_codenames, name=PROTOCOL_VIOLATION)
-site_auths.add_group(*protocol_violation_view_codenames, name=PROTOCOL_VIOLATION_VIEW)
-site_auths.add_group(*protocol_incident_codenames, name=PROTOCOL_INCIDENT)
-site_auths.add_group(*protocol_incident_view_codenames, name=PROTOCOL_INCIDENT_VIEW)
-
-if incident_type == PROTOCOL_DEVIATION_VIOLATION:
-    site_auths.update_role(PROTOCOL_VIOLATION, name=CLINICIAN_ROLE)
-    site_auths.update_role(PROTOCOL_VIOLATION, name=CLINICIAN_SUPER_ROLE)
-    site_auths.update_role(PROTOCOL_VIOLATION_VIEW, name=AUDITOR_ROLE)
-elif incident_type == PROTOCOL_INCIDENT:
-    site_auths.update_role(PROTOCOL_INCIDENT, name=CLINICIAN_ROLE)
-    site_auths.update_role(PROTOCOL_INCIDENT, name=CLINICIAN_SUPER_ROLE)
-    site_auths.update_role(PROTOCOL_INCIDENT_VIEW, name=AUDITOR_ROLE)
-else:
-    raise ValueError(
-        "Invalid value for settings.EDC_PROTOCOL_VIOLATION_TYPE. "
-        f"Expected `{PROTOCOL_INCIDENT}` or `{PROTOCOL_DEVIATION_VIOLATION}`. "
-        f"Got {incident_type}."
+
+def update_site_auths() -> None:
+    incident_type = getattr(
+        settings, "EDC_PROTOCOL_VIOLATION_TYPE", PROTOCOL_DEVIATION_VIOLATION
     )
+
+    site_auths.add_group(*protocol_violation_codenames, name=PROTOCOL_VIOLATION)
+    site_auths.add_group(*protocol_violation_view_codenames, name=PROTOCOL_VIOLATION_VIEW)
+    site_auths.add_group(*protocol_incident_codenames, name=PROTOCOL_INCIDENT)
+    site_auths.add_group(*protocol_incident_view_codenames, name=PROTOCOL_INCIDENT_VIEW)
+
+    if incident_type == PROTOCOL_DEVIATION_VIOLATION:
+        site_auths.update_role(PROTOCOL_VIOLATION, name=CLINICIAN_ROLE)
+        site_auths.update_role(PROTOCOL_VIOLATION, name=CLINICIAN_SUPER_ROLE)
+        site_auths.update_role(PROTOCOL_VIOLATION_VIEW, name=AUDITOR_ROLE)
+    elif incident_type == PROTOCOL_INCIDENT:
+        site_auths.update_role(PROTOCOL_INCIDENT, name=CLINICIAN_ROLE)
+        site_auths.update_role(PROTOCOL_INCIDENT, name=CLINICIAN_SUPER_ROLE)
+        site_auths.update_role(PROTOCOL_INCIDENT_VIEW, name=AUDITOR_ROLE)
+    else:
+        raise ValueError(
+            "Invalid value for settings.EDC_PROTOCOL_VIOLATION_TYPE. "
+            f"Expected `{PROTOCOL_INCIDENT}` or `{PROTOCOL_DEVIATION_VIOLATION}`. "
+            f"Got {incident_type}."
+        )
+
+
+update_site_auths()

edc_pylabels/auths.py

@@ -2,4 +2,9 @@ from edc_auth.site_auths import site_auths
 
 from .auth_objects import PYLABELS, codenames
 
-site_auths.add_group(*codenames, name=PYLABELS, no_delete=False)
+
+def update_site_auths() -> None:
+    site_auths.add_group(*codenames, name=PYLABELS, no_delete=False)
+
+
+update_site_auths()

edc_qareports/auths.py

@@ -9,12 +9,16 @@ from .auth_objects import (
     qa_reports_codenames,
 )
 
-# groups
-site_auths.add_group(*qa_reports_codenames, name=QA_REPORTS)
-site_auths.add_group(*qa_reports_codenames, name=QA_REPORTS_AUDIT, view_only=True)
 
+def update_site_auths() -> None:
+    # groups
+    site_auths.add_group(*qa_reports_codenames, name=QA_REPORTS)
+    site_auths.add_group(*qa_reports_codenames, name=QA_REPORTS_AUDIT, view_only=True)
 
-# roles
-site_auths.add_role(QA_REPORTS, name=QA_REPORTS_ROLE)
-site_auths.add_role(QA_REPORTS, name=QA_REPORTS_SUPER_ROLE)
-site_auths.add_role(QA_REPORTS_AUDIT, name=QA_REPORTS_AUDIT_ROLE)
+    # roles
+    site_auths.add_role(QA_REPORTS, name=QA_REPORTS_ROLE)
+    site_auths.add_role(QA_REPORTS, name=QA_REPORTS_SUPER_ROLE)
+    site_auths.add_role(QA_REPORTS_AUDIT, name=QA_REPORTS_AUDIT_ROLE)
+
+
+update_site_auths()

edc_randomization/admin.py

@@ -1,3 +1,5 @@
+import contextlib
+
 from django.conf import settings
 from django.contrib import admin
 from django.contrib.admin.sites import AlreadyRegistered
@@ -65,7 +67,8 @@ class RandomizationListModelAdmin(TemplatesModelAdminMixin, admin.ModelAdmin):
 
     def get_readonly_fields(self, request, obj=None):
         readonly_fields = super().get_readonly_fields(request, obj=obj)
-        readonly_fields += (
+        readonly_fields = (
+            *readonly_fields,
             "subject_identifier",
             "sid",
             "site_name",
@@ -75,7 +78,8 @@ class RandomizationListModelAdmin(TemplatesModelAdminMixin, admin.ModelAdmin):
             "allocated_datetime",
             "allocated_site",
             "randomizer_name",
-        ) + audit_fields
+            *audit_fields,
+        )
         return tuple(set(readonly_fields))
 
     def get_queryset(self, request):
@@ -90,7 +94,7 @@ class RandomizationListModelAdmin(TemplatesModelAdminMixin, admin.ModelAdmin):
         return qs
 
     def get_list_display(self, request) -> tuple[str, ...]:
-        list_display = [
+        fields = (
             "sid",
             "assignment",
             "site_name",
@@ -98,22 +102,24 @@ class RandomizationListModelAdmin(TemplatesModelAdminMixin, admin.ModelAdmin):
             "allocated_datetime",
             "allocated_site",
             "randomizer_name",
-        ]
+        )
+        if flds := site_randomizers.get_by_model(
+            self.model._meta.label_lower
+        ).get_extra_list_display():
+            fields = list(fields)
+            for pos, fname in flds:
+                fields.insert(pos, fname)
+            fields = tuple(fields)
         if user_is_blinded(request.user.username) or (
             not user_is_blinded(request.user.username)
             and RANDO_UNBLINDED not in [g.name for g in request.user.groups.all()]
         ):
-            list_display.remove("assignment")
-        if flds := site_randomizers.get_by_model(
-            self.model._meta.label_lower
-        ).get_extra_list_display():
-            for pos, fldname in flds:
-                list_display.insert(pos, fldname)
-        return tuple(list_display)
+            fields = tuple([fname for fname in fields if fname != "assignment"])
+        return fields
 
     @staticmethod
     def get_fieldnames(request) -> tuple[str, ...]:
-        fields = [
+        fields = (
             "subject_identifier",
             "sid",
             "assignment",
@@ -122,33 +128,35 @@ class RandomizationListModelAdmin(TemplatesModelAdminMixin, admin.ModelAdmin):
             "allocated_datetime",
             "allocated_site",
             "randomizer_name",
-        ]
+        )
         if user_is_blinded(request.user.username) or (
             not user_is_blinded(request.user.username)
             and RANDO_UNBLINDED not in [g.name for g in request.user.groups.all()]
         ):
-            fields.remove("assignment")
-        return tuple(fields)
+            fields = tuple([fname for fname in fields if fname != "assignment"])
+        return fields
 
     def get_list_filter(self, request) -> tuple[str, ...]:
-        list_filter = [
+        fields = (
             "assignment",
             "allocated_datetime",
             "allocated_site",
             "site_name",
             "randomizer_name",
-        ]
+        )
         if flds := site_randomizers.get_by_model(
             self.model._meta.label_lower
         ).get_extra_list_filter():
-            for pos, fldname in flds:
-                list_filter.insert(pos, fldname)
+            fields = list(fields)
+            for pos, fname in flds:
+                fields.insert(pos, fname)
+            fields = tuple(fields)
         if user_is_blinded(request.user.username) or (
             not user_is_blinded(request.user.username)
             and RANDO_UNBLINDED not in [g.name for g in request.user.groups.all()]
         ):
-            list_filter.remove("assignment")
-        return tuple(list_filter)
+            fields = tuple([fname for fname in fields if fname != "assignment"])
+        return fields
 
 
 def register_admin():
@@ -156,10 +164,8 @@ def register_admin():
     for randomizer_cls in site_randomizers._registry.values():
         model = randomizer_cls.model_cls()
         admin_cls = type(f"{model.__name__}ModelAdmin", (RandomizationListModelAdmin,), {})
-        try:
+        with contextlib.suppress(AlreadyRegistered):
             edc_randomization_admin.register(model, admin_cls)
-        except AlreadyRegistered:
-            pass
 
 
 register_admin()

edc_randomization/auths.py

@@ -9,11 +9,16 @@ from .auth_objects import (
     update_rando_group_permissions,
 )
 
-site_auths.add_post_update_func(
-    "edc_randomization", remove_default_model_permissions_from_edc_permissions
-)
 
-site_auths.add_group(get_rando_permissions_codenames, name=RANDO_BLINDED, view_only=True)
-site_auths.add_group(get_rando_permissions_codenames, name=RANDO_UNBLINDED, view_only=True)
-site_auths.add_post_update_func("edc_randomization", update_rando_group_permissions)
-site_auths.add_post_update_func("edc_randomization", make_randomizationlist_view_only)
+def update_site_auths() -> None:
+    site_auths.add_post_update_func(
+        "edc_randomization", remove_default_model_permissions_from_edc_permissions
+    )
+
+    site_auths.add_group(get_rando_permissions_codenames, name=RANDO_BLINDED, view_only=True)
+    site_auths.add_group(get_rando_permissions_codenames, name=RANDO_UNBLINDED, view_only=True)
+    site_auths.add_post_update_func("edc_randomization", update_rando_group_permissions)
+    site_auths.add_post_update_func("edc_randomization", make_randomizationlist_view_only)
+
+
+update_site_auths()

edc_randomization/randomizer.py

@@ -1,5 +1,6 @@
 from __future__ import annotations
 
+import contextlib
 import warnings
 from datetime import datetime
 from pathlib import Path
@@ -8,6 +9,7 @@ from typing import TYPE_CHECKING, Any
 from django.apps import apps as django_apps
 from django.conf import settings
 from django.core.exceptions import ObjectDoesNotExist, ValidationError
+from django.db.models import Q
 
 from edc_registration.utils import get_registered_subject_model_cls
 
@@ -25,15 +27,15 @@ if TYPE_CHECKING:
     from edc_registration.models import RegisteredSubject
 
 
-class InvalidAssignmentDescriptionMap(Exception):
+class InvalidAssignmentDescriptionMap(Exception):  # noqa: N818
     pass
 
 
-class RandomizationListFileNotFound(Exception):
+class RandomizationListFileNotFound(Exception):  # noqa: N818
     pass
 
 
-class RandomizationListNotLoaded(Exception):
+class RandomizationListNotLoaded(Exception):  # noqa: N818
     pass
 
 
@@ -41,7 +43,7 @@ class RandomizationError(Exception):
     pass
 
 
-class AlreadyRandomized(ValidationError):
+class AlreadyRandomized(ValidationError):  # noqa: N818
     pass
 
 
@@ -113,10 +115,10 @@ class Randomizer:
         subject_identifier: str | None = None,
         identifier_attr: str | None = None,
         identifier_object_name: str | None = None,
-        report_datetime: datetime = None,
-        site: Any = None,
-        user: str = None,
-        **kwargs,
+        report_datetime: datetime | None = None,
+        site: Any | None = None,
+        user: str | None = None,
+        **kwargs,  # noqa: ARG002
     ):
         self._model_obj = None
         self._registration_obj = None
@@ -208,8 +210,10 @@ class Randomizer:
     @property
     def sid(self):
         """Returns the SID."""
-        if self.model_obj.sid is None:
-            raise RandomizationError(f"SID cannot be None. See {self.model_obj}.")
+        if not self.model_obj.sid:
+            raise RandomizationError(
+                f"SID cannot be None. See {self.model_obj}. Got {self.model_obj.sid}"
+            )
         return self.model_obj.sid
 
     @property
@@ -233,7 +237,7 @@ class Randomizer:
         if not self._model_obj:
             try:
                 obj = self.model_cls().objects.get(**self.identifier_opts)
-            except ObjectDoesNotExist:
+            except ObjectDoesNotExist as e:
                 opts = dict(site_name=self.site.name, **self.extra_model_obj_options)
                 self._model_obj = (
                     self.model_cls()
@@ -245,7 +249,7 @@ class Randomizer:
                     fld_str = ", ".join([f"{k}=`{v}`" for k, v in opts.items()])
                     raise AllocationError(
                         f"Randomization failed. No additional SIDs available for {fld_str}."
-                    )
+                    ) from e
             else:
                 raise AlreadyRandomized(
                     f"{self.identifier_object_name.title()} already randomized. "
@@ -282,7 +286,7 @@ class Randomizer:
         Called by `registration_obj`.
         """
         return self.get_registration_model_cls().objects.get(
-            sid__isnull=True, **self.identifier_opts
+            (Q(sid__isnull=True) | Q(sid="")), **self.identifier_opts
         )
 
     @property
@@ -298,14 +302,14 @@ class Randomizer:
         if not self._registration_obj:
             try:
                 self._registration_obj = self.get_unallocated_registration_obj()
-            except ObjectDoesNotExist:
+            except ObjectDoesNotExist as e:
                 try:
                     obj = self.get_registration_model_cls().objects.get(**self.identifier_opts)
-                except ObjectDoesNotExist:
+                except ObjectDoesNotExist as e:
                     raise RandomizationError(
                         f"{self.identifier_object_name.title()} does not exist. "
                         f"Got {getattr(self, self.identifier_attr)}"
-                    )
+                    ) from e
                 else:
                     raise AlreadyRandomized(
                         f"{self.identifier_object_name.title()} already randomized. "
@@ -313,7 +317,7 @@ class Randomizer:
                         f"Got {getattr(obj, self.identifier_attr)} "
                         f"SID={obj.sid}",
                         code=self.get_registration_model_cls()._meta.label_lower,
-                    )
+                    ) from e
         return self._registration_obj
 
     @property
@@ -347,7 +351,7 @@ class Randomizer:
                 "Randomization list file not found. "
                 f"Got `{cls.get_randomizationlist_path()}`. See Randomizer {cls.name}."
             )
-        try:
+        with contextlib.suppress(RandomizationListAlreadyImported):
             result = cls.importer_cls(
                 assignment_map=cls.assignment_map,
                 randomizationlist_path=cls.get_randomizationlist_path(),
@@ -356,8 +360,6 @@ class Randomizer:
                 extra_csv_fieldnames=cls.extra_csv_fieldnames,
                 **kwargs,
             ).import_list(**kwargs)
-        except RandomizationListAlreadyImported:
-            pass
         return result
 
     @classmethod

edc_randomization/utils.py

@@ -1,7 +1,8 @@
 from __future__ import annotations
 
 import csv
-import os
+import sys
+from pathlib import Path
 from typing import Any
 
 from django.conf import settings
@@ -20,13 +21,13 @@ class RandomizationListExporterError(Exception):
     pass
 
 
-class SubjectNotRandomization(Exception):
+class SubjectNotRandomization(Exception):  # noqa: N818
     pass
 
 
 def get_assignment_for_subject(
     subject_identifier: str,
-    randomizer_name: str = None,
+    randomizer_name: str,
     identifier_fld: str | None = None,
 ) -> str:
     """Returns the assignment for a randomized subject.
@@ -42,7 +43,7 @@ def get_assignment_for_subject(
 
 def get_assignment_description_for_subject(
     subject_identifier: str,
-    randomizer_name: str = None,
+    randomizer_name: str,
     identifier_fld: str | None = None,
 ) -> str:
     """Returns the assignment description for a randomized subject.
@@ -60,7 +61,7 @@ def get_assignment_description_for_subject(
 
 def get_object_for_subject(
     subject_identifier: str,
-    randomizer_name: str = None,
+    randomizer_name: str,
     identifier_fld: str | None = None,
     label: str | None = None,
 ) -> Any:
@@ -81,11 +82,11 @@ def get_object_for_subject(
     }
     try:
         obj = randomizer_cls.model_cls().objects.get(**opts)
-    except ObjectDoesNotExist:
+    except ObjectDoesNotExist as e:
         raise SubjectNotRandomization(
             f"{label.title()} not randomized. See Randomizer `{randomizer_name}`. "
             f"Got {identifier_fld}=`{subject_identifier}`."
-        )
+        ) from e
     return obj
 
 
@@ -118,7 +119,7 @@ def generate_fake_randomization_list(
 
     # get site ID and write the file
     site_id = sites.get_by_attr("name", site_name)
-    with open(filename, "a+", newline="") as f:
+    with Path(filename).open("a+", newline="") as f:
         writer = csv.DictWriter(f, fieldnames=["sid", "assignment", "site_name", "country"])
         if write_header:
             writer.writeheader()
@@ -133,7 +134,7 @@ def generate_fake_randomization_list(
                 )
             )
 
-    print(f"(*) Added {slots} slots for {site_name}.")
+    sys.stdout.write(f"(*) Added {slots} slots for {site_name}.\n")
 
 
 def export_randomization_list(
@@ -143,20 +144,20 @@ def export_randomization_list(
 
     try:
         user = get_user_model().objects.get(username=username)
-    except ObjectDoesNotExist:
-        raise RandomizationListExporterError(f"User `{username}` does not exist")
+    except ObjectDoesNotExist as e:
+        raise RandomizationListExporterError(f"User `{username}` does not exist") from e
     if not user.has_perm(randomizer_cls.model_cls()._meta.label_lower.replace(".", ".view_")):
         raise RandomizationListExporterError(
             f"User `{username}` does not have "
             f"permission to view '{randomizer_cls.model_cls()._meta.label_lower}'"
         )
-    path = path or settings.EXPORT_FOLDER
+    path = Path(path or settings.EXPORT_FOLDER)
     timestamp = timezone.now().strftime("%Y%m%d%H%M")
-    filename = os.path.expanduser(
+    filename = Path(
         f"~/{settings.APP_NAME}_{randomizer_cls.name}_"
         f"randomizationlist_exported_{timestamp}.csv"
-    )
-    filename = os.path.join(path, filename)
+    ).expanduser()
+    filename = path / filename
 
     df = (
         read_frame(randomizer_cls.model_cls().objects.all(), verbose=False)
@@ -172,5 +173,5 @@ def export_randomization_list(
         sep="|",
     )
     df.to_csv(**opts)
-    print(filename)
+    sys.stdout.write(f"{filename!s}\n")
     return filename

edc_refusal/auths.py

@@ -3,5 +3,10 @@ from edc_screening.auth_objects import SCREENING, SCREENING_SUPER
 
 from .auth_objects import codenames
 
-site_auths.update_group(*codenames, name=SCREENING, no_delete=True)
-site_auths.update_group(*codenames, name=SCREENING_SUPER)
+
+def update_site_auths() -> None:
+    site_auths.update_group(*codenames, name=SCREENING, no_delete=True)
+    site_auths.update_group(*codenames, name=SCREENING_SUPER)
+
+
+update_site_auths()

edc_refusal/model_mixins.py

@@ -33,7 +33,7 @@ class SubjectRefusalModelMixin(models.Model):
 
     @staticmethod
     def get_search_slug_fields():
-        return ["screening_identifier"]
+        return ("screening_identifier",)
 
     class Meta:
         abstract = True