ModelForm の単体テストを開発していますが、フィールドに定義されたクエリセット内で選択できない値を渡すと、フォームの ModelChoiceField が invalid_choice ValidationError を発生させていないことがわかりました。
ドキュメントがhttps://docs.djangoproject.com/en/1.4/ref/forms/validation/#form-and-field-validationで言うように、ModelChoiceFieldのto_python()が最初に呼び出されていないことが理由だと思いますまたは to_python() は、フォームとモデルの両方の clean メソッドを呼び出す前に DoesNotExist 例外をキャッチしていません。後者は、以下に含まれる未処理の例外を引き起こします。
# Test Results
E
======================================================================
ERROR: test_clean (tickets.tests.PaymentFormTests)
----------------------------------------------------------------------
Traceback (most recent call last):
File "/home/anand/Ubuntu One/paytickets/park_db/tickets/tests.py", line 187, in test_clean
self.assertFalse(form.is_valid())
File "/usr/local/lib/python2.7/dist-packages/django/forms/forms.py", line 124, in is_valid
return self.is_bound and not bool(self.errors)
File "/usr/local/lib/python2.7/dist-packages/django/forms/forms.py", line 115, in _get_errors
self.full_clean()
File "/usr/local/lib/python2.7/dist-packages/django/forms/forms.py", line 272, in full_clean
self._post_clean()
File "/usr/local/lib/python2.7/dist-packages/django/forms/models.py", line 332, in _post_clean
self.instance.clean()
File "/home/anand/Ubuntu One/paytickets/park_db/tickets/models.py", line 133, in clean
fine = self.ticket.fine_amount
File "/usr/local/lib/python2.7/dist-packages/django/db/models/fields/related.py", line 343, in __get__
raise self.field.rel.to.DoesNotExist
DoesNotExist
ここで、モデルのクリーン メソッドをコメント アウトすると、フォームは期待どおりに動作し、invalid_choice エラーが発生します。これは、モデルの clean メソッドが to_python メソッドの前に呼び出されていることを示しています。
ここで何が起こっているのかを理解するのを手伝ってください。このテストを機能させるにはどうすればよいですか? Django 1.4.3 を使用しています。私のコードは以下の通りです:
# models.py
class Payment(TimeStampedModel): # TimeStampedModel is an abstract base class
user = models.ForeignKey(User, editable=False)
ticket = models.ForeignKey('Ticket', to_field='number')
payment_amount = models.DecimalField(max_digits=7, decimal_places=2)
def clean(self):
fine = self.ticket.fine_amount
payment_sum = self.ticket.sum_payments()
remaining = fine - payment_sum
if self.payment_amount < 1:
msg = 'The minimum payment is $1.'
raise ValidationError(msg)
elif remaining == 0:
msg = 'That ticket has already been fully paid.'
raise ValidationError(msg)
elif remaining < self.payment_amount:
msg = ('You cannot pay more than the oustanding fine: $%0.2f' %
remaining)
raise ValidationError(msg)
def __unicode__(self):
return u'%s; %s; %s' % (self.created, self.ticket, self.payment_amount)
# forms.py
class PaymentForm(forms.ModelForm):
class Meta:
model = Payment
def __init__(self, user=None, *args, **kwargs):
super(PaymentForm, self).__init__(*args, **kwargs)
# passing in user and performing query for ModelChoiceField options
self._user = user
vehicles = self._user.vehicle_set.all().prefetch_related('ticket_set')
ticket_list= []
for i in vehicles:
for j in i.ticket_set.all():
ticket_list.append(j.pk)
self.ticket_queryset = Ticket.objects.filter(pk__in=ticket_list)
self.fields['ticket'].queryset = self.ticket_queryset
def save(self, commit=True):
instance = super(PaymentForm, self).save(commit=False)
instance.user = self._user
if commit:
instance.save()
return instance
# tests.py
class PaymentFormTests(TestCase):
fixtures = ['test_users.json', 'tickets_app.json']
def test_clean(self):
user = User.objects.get(username="test_external_active")
form_data = {
"ticket": 1000007, # this is not a valid ticket number
"payment_amount": 1.00
}
form = PaymentForm(user=user, data=form_data)
self.assertFalse(form.is_valid())