From dcfafea408a6eec2d35f9bc2f7aaff724ceee36f Mon Sep 17 00:00:00 2001 From: Karen Tracey Date: Wed, 5 Nov 2008 19:55:51 +0000 Subject: [1.0.X] Fixed #9218 -- Simplified the fix from #9039 and added tests to ensure this case doesn't break again (and that the simplification didn't break anything). [9341] from trunk. Also updated svnmerge metadata. git-svn-id: http://code.djangoproject.com/svn/django/branches/releases/1.0.X@9342 bcc190cf-cafb-0310-a4f2-bffc1f526a37 --- tests/modeltests/model_forms/models.py | 30 ++++++++++++++++++++++++++++++ 1 file changed, 30 insertions(+) (limited to 'tests/modeltests/model_forms') diff --git a/tests/modeltests/model_forms/models.py b/tests/modeltests/model_forms/models.py index 01728a997e..0489ea81d8 100644 --- a/tests/modeltests/model_forms/models.py +++ b/tests/modeltests/model_forms/models.py @@ -158,6 +158,15 @@ class Book(models.Model): class Meta: unique_together = ('title', 'author') + +class ExplicitPK(models.Model): + key = models.CharField(max_length=20, primary_key=True) + desc = models.CharField(max_length=20, blank=True, unique=True) + class Meta: + unique_together = ('key', 'desc') + + def __unicode__(self): + return self.key __test__ = {'API_TESTS': """ >>> from django import forms @@ -1247,6 +1256,27 @@ True >>> form.is_valid() True +# Test for primary_key being in the form and failing validation. +>>> class ExplicitPKForm(ModelForm): +... class Meta: +... model = ExplicitPK +... fields = ('key', 'desc',) +>>> form = ExplicitPKForm({'key': u'', 'desc': u'' }) +>>> form.is_valid() +False + +# Ensure keys and blank character strings are tested for uniqueness. +>>> form = ExplicitPKForm({'key': u'key1', 'desc': u''}) +>>> form.is_valid() +True +>>> form.save() + +>>> form = ExplicitPKForm({'key': u'key1', 'desc': u''}) +>>> form.is_valid() +False +>>> form.errors +{'__all__': [u'Explicit pk with this Key and Desc already exists.'], 'key': [u'Explicit pk with this Key already exists.'], 'desc': [u'Explicit pk with this Desc already exists.']} + # Choices on CharField and IntegerField >>> class ArticleForm(ModelForm): ... class Meta: -- cgit v1.3