[fix] Invalidate DeviceGroup cache when Cert.organization is None

Author: pandafy

openwisp_controller/config/handlers.py

@@ -105,7 +105,9 @@ def devicegroup_delete_handler(instance, **kwargs):
         if not instance.common_name or organization is None:
             return
         kwargs["common_name"] = instance.common_name
-        kwargs["organization_slug"] = organization.slug
+        organization = instance.organization
+        if organization is not None:
+            kwargs["organization_slug"] = organization.slug
     else:
         kwargs["organization_id"] = instance.organization_id
     instance_id = instance.id

openwisp_controller/pki/tests/test_api.py

@@ -1,5 +1,8 @@
 from django.test import TestCase, TransactionTestCase
 from django.urls import reverse
+from openwisp_users.tests.test_api import AuthenticationMixin
+from openwisp_users.tests.utils import TestOrganizationMixin
+from openwisp_utils.tests import AssertNumQueriesSubTestMixin, capture_any_output
 from packaging.version import parse as parse_version
 from rest_framework import VERSION as REST_FRAMEWORK_VERSION
 from swapper import load_model
@@ -247,7 +250,7 @@ def test_cert_detail_api(self):
     def test_cert_delete_api(self):
         cert1 = self._create_cert(name="cert1")
         path = reverse("pki_api:cert_detail", args=[cert1.pk])
-        with self.assertNumQueries(6):
+        with self.assertNumQueries(5):
             r = self.client.delete(path)
         self.assertEqual(r.status_code, 204)
         self.assertEqual(Cert.objects.count(), 0)