Disable atomic requests for health-check view

health_check/views.py

@@ -1,5 +1,6 @@
 import re
 
+from django.db import transaction
 from django.http import HttpResponse, JsonResponse
 from django.utils.decorators import method_decorator
 from django.views.decorators.cache import never_cache
@@ -79,6 +80,7 @@ def __lt__(self, other):
         return self.weight.__lt__(other.weight)
 
 
+@method_decorator(transaction.non_atomic_requests, name="dispatch")
 class MainView(CheckMixin, TemplateView):
     template_name = "health_check/index.html"
 

tests/test_views.py

@@ -1,6 +1,8 @@
 import json
+from unittest.mock import Mock
 
 import pytest
+from django.db import DatabaseError
 from django.urls import reverse
 
 from health_check.backends import BaseHealthCheckBackend
@@ -309,3 +311,15 @@ def run_check(self):
         assert response.status_code == 500, response.content.decode("utf-8")
         assert response["content-type"] == "application/json"
         assert "JSON Error" in json.loads(response.content.decode("utf-8"))[JSONErrorBackend().identifier()]
+
+    @pytest.mark.django_db(transaction=True)
+    def test_non_native_atomic_request(self, settings, monkeypatch, client):
+        # See also: https://github.com/revsys/django-health-check/pull/469
+        settings.DATABASES["default"]["ATOMIC_REQUESTS"] = True
+        # disable the ensure_connection
+        monkeypatch.setattr(
+            "django.db.backends.base.base.BaseDatabaseWrapper.ensure_connection", Mock(side_effect=DatabaseError())
+        )
+        response = client.get(self.url)
+        assert response.status_code == 500
+        assert b"<title>System status</title>" in response.content