From a7ed09d13d9532089bd2380edab1df5df96082a6 Mon Sep 17 00:00:00 2001 From: David Cramer Date: Mon, 14 Jan 2013 13:18:24 -0800 Subject: [PATCH] Improve test to ensure that post_delete was actually called --- tests/modeltests/delete/tests.py | 7 +++++++ 1 file changed, 7 insertions(+) diff --git a/tests/modeltests/delete/tests.py b/tests/modeltests/delete/tests.py index 99981df0185..0a3ddcfc2e2 100644 --- a/tests/modeltests/delete/tests.py +++ b/tests/modeltests/delete/tests.py @@ -230,8 +230,12 @@ class DeletionTests(TestCase): models.signals.post_delete.disconnect(log_pre_delete) def test_relational_post_delete_signals_happen_before_parent_object(self): + deletions = [] + def log_post_delete(instance, **kwargs): self.assertTrue(R.objects.filter(pk=instance.r_id)) + self.assertEquals(type(instance), S) + deletions.append(instance.id) r = R.objects.create(pk=1) S.objects.create(pk=1, r=r) @@ -243,6 +247,9 @@ class DeletionTests(TestCase): finally: models.signals.post_delete.disconnect(log_post_delete) + self.assertEquals(len(deletions), 1) + self.assertEquals(deletions[0], 1) + @skipUnlessDBFeature("can_defer_constraint_checks") def test_can_defer_constraint_checks(self): u = User.objects.create(