Fix ruff D213 and skip dispatch during fixture loads

- Reformat async_create_notification importer-guard docstring to D213 style
- Skip post_save dispatch when raw=True (loaddata) so the k8s initializer's
  fixture install path doesn't require an available Celery broker. Without
  this guard the unconditional async dispatch tries to enqueue during
  product_type.json load and fails with kombu OperationalError.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

Author: Maffooch

dojo/engagement/signals.py

@@ -17,6 +17,9 @@
 
 @receiver(post_save, sender=Engagement)
 def engagement_post_save(sender, instance, created, **kwargs):
+    # raw=True is set by loaddata; skip dispatch so fixture loading doesn't require a live broker.
+    if kwargs.get("raw"):
+        return
     if created:
         title = _('Engagement created for "%(product)s": %(name)s') % {"product": instance.product, "name": instance.name}
         dojo_dispatch_task(

dojo/product/signals.py

@@ -19,6 +19,9 @@
 
 @receiver(post_save, sender=Product)
 def product_post_save(sender, instance, created, **kwargs):
+    # raw=True is set by loaddata; skip dispatch so fixture loading doesn't require a live broker.
+    if kwargs.get("raw"):
+        return
     if created:
         dojo_dispatch_task(
             async_create_notification,

dojo/product_type/signals.py

@@ -19,6 +19,9 @@
 
 @receiver(post_save, sender=Product_Type)
 def product_type_post_save(sender, instance, created, **kwargs):
+    # raw=True is set by loaddata; skip dispatch so fixture loading doesn't require a live broker.
+    if kwargs.get("raw"):
+        return
     if created:
         dojo_dispatch_task(
             async_create_notification,

unittests/test_notifications.py

@@ -602,7 +602,8 @@ def test_product_type_added_dispatches_async(self, mock_dispatch):
         self.assertNotIn("product_type", kwargs)
 
     def test_importer_dispatch_uses_id_kwargs(self):
-        """Static check that default_importer's test_added dispatch passes *_id kwargs (not model instances).
+        """
+        Static check that default_importer's test_added dispatch passes *_id kwargs (not model instances).
 
         The importer's notification dispatch fires deep inside a full scan-import flow;
         exercising it end-to-end is covered by integration tests. This guard test