diff options
author | Florian Apolloner <florian@apolloner.eu> | 2013-02-26 09:53:47 +0100 |
---|---|---|
committer | Florian Apolloner <florian@apolloner.eu> | 2013-02-26 14:36:57 +0100 |
commit | 89f40e36246100df6a11316c31a76712ebc6c501 (patch) | |
tree | 6e65639683ddaf2027908d1ecb1739e0e2ff853b /tests/null_queries | |
parent | b3d2ccb5bfbaf6e7fe1f98843baaa48c35a70950 (diff) | |
download | django-89f40e36246100df6a11316c31a76712ebc6c501.tar.gz |
Merged regressiontests and modeltests into the test root.
Diffstat (limited to 'tests/null_queries')
-rw-r--r-- | tests/null_queries/__init__.py | 0 | ||||
-rw-r--r-- | tests/null_queries/models.py | 32 | ||||
-rw-r--r-- | tests/null_queries/tests.py | 83 |
3 files changed, 115 insertions, 0 deletions
diff --git a/tests/null_queries/__init__.py b/tests/null_queries/__init__.py new file mode 100644 index 0000000000..e69de29bb2 --- /dev/null +++ b/tests/null_queries/__init__.py diff --git a/tests/null_queries/models.py b/tests/null_queries/models.py new file mode 100644 index 0000000000..9070dd4873 --- /dev/null +++ b/tests/null_queries/models.py @@ -0,0 +1,32 @@ +from __future__ import unicode_literals + +from django.db import models +from django.utils.encoding import python_2_unicode_compatible + + +@python_2_unicode_compatible +class Poll(models.Model): + question = models.CharField(max_length=200) + + def __str__(self): + return "Q: %s " % self.question + +@python_2_unicode_compatible +class Choice(models.Model): + poll = models.ForeignKey(Poll) + choice = models.CharField(max_length=200) + + def __str__(self): + return "Choice: %s in poll %s" % (self.choice, self.poll) + +# A set of models with an inner one pointing to two outer ones. +class OuterA(models.Model): + pass + +class OuterB(models.Model): + data = models.CharField(max_length=10) + +class Inner(models.Model): + first = models.ForeignKey(OuterA) + # second would clash with the __second lookup. + third = models.ForeignKey(OuterB, null=True) diff --git a/tests/null_queries/tests.py b/tests/null_queries/tests.py new file mode 100644 index 0000000000..93e72d55d8 --- /dev/null +++ b/tests/null_queries/tests.py @@ -0,0 +1,83 @@ +from __future__ import absolute_import + +from django.test import TestCase +from django.core.exceptions import FieldError + +from .models import Poll, Choice, OuterA, Inner, OuterB + + +class NullQueriesTests(TestCase): + + def test_none_as_null(self): + """ + Regression test for the use of None as a query value. + + None is interpreted as an SQL NULL, but only in __exact queries. + Set up some initial polls and choices + """ + p1 = Poll(question='Why?') + p1.save() + c1 = Choice(poll=p1, choice='Because.') + c1.save() + c2 = Choice(poll=p1, choice='Why Not?') + c2.save() + + # Exact query with value None returns nothing ("is NULL" in sql, + # but every 'id' field has a value). + self.assertQuerysetEqual(Choice.objects.filter(choice__exact=None), []) + + # Excluding the previous result returns everything. + self.assertQuerysetEqual( + Choice.objects.exclude(choice=None).order_by('id'), + [ + '<Choice: Choice: Because. in poll Q: Why? >', + '<Choice: Choice: Why Not? in poll Q: Why? >' + ] + ) + + # Valid query, but fails because foo isn't a keyword + self.assertRaises(FieldError, Choice.objects.filter, foo__exact=None) + + # Can't use None on anything other than __exact + self.assertRaises(ValueError, Choice.objects.filter, id__gt=None) + + # Can't use None on anything other than __exact + self.assertRaises(ValueError, Choice.objects.filter, foo__gt=None) + + # Related managers use __exact=None implicitly if the object hasn't been saved. + p2 = Poll(question="How?") + self.assertEqual(repr(p2.choice_set.all()), '[]') + + def test_reverse_relations(self): + """ + Querying across reverse relations and then another relation should + insert outer joins correctly so as not to exclude results. + """ + obj = OuterA.objects.create() + self.assertQuerysetEqual( + OuterA.objects.filter(inner__third=None), + ['<OuterA: OuterA object>'] + ) + self.assertQuerysetEqual( + OuterA.objects.filter(inner__third__data=None), + ['<OuterA: OuterA object>'] + ) + + inner_obj = Inner.objects.create(first=obj) + self.assertQuerysetEqual( + Inner.objects.filter(first__inner__third=None), + ['<Inner: Inner object>'] + ) + + # Ticket #13815: check if <reverse>_isnull=False does not produce + # faulty empty lists + objB = OuterB.objects.create(data="reverse") + self.assertQuerysetEqual( + OuterB.objects.filter(inner__isnull=False), + [] + ) + Inner.objects.create(first=obj) + self.assertQuerysetEqual( + OuterB.objects.exclude(inner__isnull=False), + ['<OuterB: OuterB object>'] + ) |