Ticket #16173: patch16173.diff

tests/modeltests/many_to_one/tests.py

@@ -6 +6 @@
 from django.core.exceptions import MultipleObjectsReturned
 from django.test import TestCase
 
-from .models import Article, Reporter
+from .models import (Article, Reporter, RelatedModel, SomeModel,
+        SuccessfulExecutedException)
 
 
 class ManyToOneTests(TestCase):
@@ -412 +413 @@
 
         # Same as each other
         self.assertTrue(r1.article_set.__class__ is r2.article_set.__class__)
+
+    def test_reverse_single_related_manager(self):
+        # This unit test prooves if the dummy managers get methode will be
+        # executed. Related to the ticket #16173.
+        related_obj = RelatedModel.default_manager.create(name='bar')
+        obj = SomeModel.objects.create(related=related_obj)
+
+        # reload obj to make sure the
+        obj = SomeModel.objects.all()[0]
+
+        # the get method of the dummy manager must be used
+        with self.assertRaises(SuccessfulExecutedException):
+            obj.related

tests/modeltests/many_to_one/models.py

@@ -25 +25 @@
 
     class Meta:
         ordering = ('headline',)
+
+class SuccessfulExecutedException(Exception):
+    pass
+
+
+class DummyManager(models.Manager):
+    use_for_related_fields = True
+    def get(self, **kwargs):
+        raise SuccessfulExecutedException
+
+
+class RelatedModel(models.Model):
+    name = models.CharField(max_length=32)
+
+    default_manager = DummyManager()
+
+
+class SomeModel(models.Model):
+    related = models.ForeignKey(RelatedModel)