django 5.1 adaptions

ca/django_ca/tests/test_acme.py

@@ -31,6 +31,9 @@
 from django.urls.exceptions import NoReverseMatch
 from django.utils.crypto import get_random_string
 
+import pytest
+from pytest_django.fixtures import SettingsWrapper
+
 from django_ca import urls
 from django_ca.acme import validation
 from django_ca.acme.constants import IdentifierType, Status
@@ -42,47 +45,17 @@
 ]
 
 
-class URLPatternTestCase(TestCase):
-    """Test that URL patterns are not enabled when CA_ENABLE_ACME."""
+def assert_no_reverse_match(
+    name: str, args: Optional[typing.Sequence[Any]] = None, kwargs: Optional[dict[str, Any]] = None
+) -> None:
+    """Context manager asserting that the given URL pattern is **not** found."""
+    urlname = name
+    if ":" in urlname:
+        _namespace, urlname = name.split(":", 1)
 
-    @contextmanager
-    def reload_urlconf(self) -> Iterator[None]:
-        """Context manager to reload the current URL configuration."""
-        reload(urls)
-        try:
-            with self.settings(ROOT_URLCONF=__name__):
-                yield
-        finally:
-            reload(urls)
-
-    def assertNoReverseMatch(  # pylint: disable=invalid-name
-        self,
-        name: str,
-        args: Optional[typing.Sequence[Any]] = None,
-        kwargs: Optional[dict[str, Any]] = None,
-    ) -> None:
-        """Context manager asserting that the given URL pattern is **not** found."""
-        urlname = name
-        if ":" in name:
-            _namespace, urlname = name.split(":", 1)
-
-        msg = f"Reverse for '{urlname}' not found. '{urlname}' is not a valid view function or pattern name."
-        with self.assertRaisesRegex(NoReverseMatch, msg):
-            reverse(name, args=args, kwargs=kwargs)
-
-    @override_settings(CA_ENABLE_ACME=False, CA_ENABLE_REST_API=False)
-    def test_disabled(self) -> None:
-        """Test that resolving URLs does **NOT** work if disabled."""
-        with self.reload_urlconf():
-            self.assertNoReverseMatch("django_ca:acme-directory")
-            self.assertNoReverseMatch("django_ca:acme-directory", kwargs={"serial": "AB:CD"})
-            self.assertNoReverseMatch("django_ca:acme-new-nonce", kwargs={"serial": "AB:CD"})
-
-    def test_enabled(self) -> None:
-        """Test that resolving URLs work if enabled."""
-        reverse("django_ca:acme-directory")
-        reverse("django_ca:acme-directory", kwargs={"serial": "AB:CD"})
-        reverse("django_ca:acme-new-nonce", kwargs={"serial": "AB:CD"})
+    msg = f"Reverse for '{urlname}' not found. '{urlname}' is not a valid view function or pattern name."
+    with pytest.raises(NoReverseMatch, match=msg):
+        reverse(name, args=args, kwargs=kwargs)
 
 
 class TestConstantsTestCase(TestCase):

ca/django_ca/tests/test_utils.py

@@ -30,6 +30,7 @@
 from cryptography.x509.name import _ASN1Type
 from cryptography.x509.oid import NameOID
 
+import django
 from django.test import TestCase, override_settings
 
 import pytest
@@ -81,6 +82,7 @@ def test_read_file(tmpcadir: Path) -> None:
     assert read_file(path) == data
 
 
+@pytest.mark.skipif(django.VERSION >= (5, 1), reason="get_storage_class() is removed in Django 5.1")
 def test_deprecated_storage_configuration(settings: SettingsWrapper) -> None:
     """Test that using a deprecated storage configuration emits a warning."""
     settings.STORAGES = {

ca/django_ca/urls.py

@@ -40,13 +40,13 @@
     path("crl/ca/<hex:serial>/", views.CertificateRevocationListView.as_view(scope="ca"), name="ca-crl"),
 ]
 
-if model_settings.CA_ENABLE_REST_API is True:
+if model_settings.CA_ENABLE_REST_API is True:  # pragma: no branch
     from django_ca.api.endpoints import api
 
     urlpatterns.append(path("api/", api.urls))
 
 
-if model_settings.CA_ENABLE_ACME:
+if model_settings.CA_ENABLE_ACME:  # pragma: no branch
     from django_ca.acme import views as acme_views
 
     # NOTE: Some functions depend on the fact that ALL ACME urls have a "serial" kwarg

ca/django_ca/utils.py

@@ -36,7 +36,7 @@
 from cryptography.x509.oid import NameOID
 
 from django.conf import global_settings, settings
-from django.core.files.storage import InvalidStorageError, Storage, get_storage_class, storages
+from django.core.files.storage import InvalidStorageError, Storage, storages
 from django.utils import timezone
 
 from django_ca import constants
@@ -973,6 +973,10 @@ def get_storage() -> Storage:
             RemovedInDjangoCA200Warning,
             stacklevel=2,
         )
+
+        # Lazy import as this function is removed in Django 5.0.
+        from django.core.files.storage import get_storage_class
+
         ca_file_storage = getattr(settings, "CA_FILE_STORAGE", global_settings.DEFAULT_FILE_STORAGE)
         ca_file_storage_kwargs = getattr(
             settings,
@@ -983,6 +987,7 @@ def get_storage() -> Storage:
                 "directory_permissions_mode": 0o700,
             },
         )
+
         ca_storage_cls = get_storage_class(ca_file_storage)
         return ca_storage_cls(**ca_file_storage_kwargs)
 
@@ -1000,10 +1005,7 @@ def read_file(path: str) -> bytes:
 
 
 def split_str(val: str, sep: str) -> Iterator[str]:
-    """Split a character on the given set of characters.
-
-    This function is now only used in a migration and will be moved there in 2.0.
-    """
+    """Split a character on the given set of characters."""
     lex = shlex.shlex(val, posix=True)
     lex.commenters = ""
     lex.whitespace = sep