[-] Fixes issue w/ post_delete / Adds delete test

Author: jsoa

easyaudit/signals/model_signals.py

@@ -302,6 +302,9 @@ def post_delete(sender, instance, using, **kwargs):
                 user = None
             c_t = ContentType.objects.get_for_model(instance)
 
+            # object id to be used later
+            obj_id = instance.pk
+
             def crud_flow():
                 try:
                     with transaction.atomic(using=DATABASE_ALIAS):
@@ -311,7 +314,7 @@ def crud_flow():
                             'object_repr': str(instance),
                             'object_json_repr': object_json_repr,
                             'content_type_id': c_t.id,
-                            'object_id': instance.pk,
+                            'object_id': obj_id,
                             'user_id': getattr(user, 'id', None),
                             'datetime': timezone.now(),
                             'user_pk_as_string': str(user.pk) if user else user

easyaudit/tests/test_app/tests.py

@@ -90,6 +90,16 @@ def test_fake_update(self):
         obj.save()
         self.assertEqual(2, crud_event_qs.count())
 
+    def test_delete(self):
+        obj = self.Model.objects.create()
+        crud_event_qs = CRUDEvent.objects.filter(object_id=obj.id, content_type=ContentType.objects.get_for_model(obj))
+        self.assertEqual(1, crud_event_qs.count())
+
+        obj_id = obj.pk
+        obj.delete()
+        crud_event_qs = CRUDEvent.objects.filter(object_id=obj_id, content_type=ContentType.objects.get_for_model(obj))
+        self.assertEqual(2, crud_event_qs.count())
+
 
 class TestAuditUUIDModels(TestAuditModels):
     Model = TestUUIDModel