即使引发了 ValidationError,表单在单元测试中仍然有效
Form valid in unit test even though ValidationError was raised
我正在测试处理无效表单数据的视图。在我的测试用例中,我正在提交缺少字段的表单,并期望视图通过显示错误消息来处理它。这是我表格中 clean
的相关片段:
表格:
def clean(self):
first_name = self.cleaned_data.get('first_name', '')
last_name = self.cleaned_data.get('last_name', '')
if not first_name or not last_name:
print('Form is invalid')
# This is always executed.
raise ValidationError('All fields are required.')
# other stuff
ValidationError
总是被引发,然而,form.is_valid()
returns True
尽管如此:
查看:
form = forms.EditProfileForm(request.POST, instance=user)
if form.is_valid():
print('Form is valid')
# Update user data.
# Always executed even though error was raised.
这会导致测试失败,因为我断言表单验证将失败:
测试用例:
def test_invalid_data(self):
formdata = {
'first_name': 'Bruno',
'last_name': '', # Missing data.
'email': 'bruno@schulz.pl',
'password': 'metamorphosis'
}
request = self.factory.post('/accounts/profile', formdata)
request.user = self.user
setup_messages(request)
response = ProfileView.as_view()(request)
self.assertEqual(response.status_code, 200)
self.assertNotEqual(self.user.first_name, formdata['first_name']) # Fails.
self.assertNotEqual(self.user.last_name, formdata['last_name'])
self.assertNotEqual(self.user.email, formdata['email'])
self.assertNotEqual(self.user.username, formdata['email'])
表单验证在实时服务器上正常工作(已测试 "by hand")。
看起来 ValidationError
在 TestCase
评估期间被忽略了。
*测试输出:
.Form is invalid
F..Form is valid
.
======================================================================
FAIL: test_invalid_data (.tests.ProfileTestCase)
----------------------------------------------------------------------
Traceback (most recent call last):
File "/apps/accounts/tests.py", line 114, in test_invalid_data
self.assertNotEqual(self.user.first_name, formdata['first_name'])
AssertionError: 'Bruno' == 'Bruno'
您没有显示完整视图,但通常在 post 成功后 return redirect('/success-url/')
。如果您的验证错误被忽略,那么视图将重定向状态代码 302,并且您的测试将在前面的行 self.assertEqual(response.status_code, 200)
上失败
您可以使用 form = response.context['form']
从视图访问表单。如果您在测试中检查 form.is_valid()
或 form.errors
,您将看到您的 ValidationError
没有被忽略。
您的问题是您的 assertNotEqual
检查没有测试您认为的内容。当您验证模型表单时,实例会被修改。如果要查看数据库中用户是否被修改,需要refresh it from the database first.
self.user.refresh_from_db()
尝试使用 self.add_error('last_name', your_validation_error)
而不是仅仅引发错误:
def clean(self):
first_name = self.cleaned_data.get('first_name', '')
last_name = self.cleaned_data.get('last_name', '')
if not first_name:
print('Form is invalid: missing first name')
# define but don't raise the error
missing_first_name = ValidationError('First name field is required.')
self.add_error('first_name', missing_first_name)
if not last_name:
print('Form is invalid: missing last name')
# define but don't raise error
missing_last_name = ValidationError('Last name field is required.')
self.add_error('last_name', missing_last_name)
# you could also add a string instead of an Error, but a ValidationError is recommended:
# self.add_error('last_name', 'the last name field is missing')
# other stuff
现在 form.is_valid()
将 return 错误。
This SO answer led me to the solution, and here are the add_error docs。您应该使用 self.add_error
将错误添加到表单中,而不是引发验证错误。
(我确实发现了 this example in the docs 错误被引发而不是传递给 self.add_error
,这可能是您混淆的根源,但那是为了清理特定字段。它不是form.clean()
方法。我假设您可以捕获该错误,然后用 self.add_error
添加它...但这是一个需要解决的不同问题。关键是,我没有在form.clean()
应该引发 ValidationError 的文档。应该将其添加到表单中。)
我正在测试处理无效表单数据的视图。在我的测试用例中,我正在提交缺少字段的表单,并期望视图通过显示错误消息来处理它。这是我表格中 clean
的相关片段:
表格:
def clean(self):
first_name = self.cleaned_data.get('first_name', '')
last_name = self.cleaned_data.get('last_name', '')
if not first_name or not last_name:
print('Form is invalid')
# This is always executed.
raise ValidationError('All fields are required.')
# other stuff
ValidationError
总是被引发,然而,form.is_valid()
returns True
尽管如此:
查看:
form = forms.EditProfileForm(request.POST, instance=user)
if form.is_valid():
print('Form is valid')
# Update user data.
# Always executed even though error was raised.
这会导致测试失败,因为我断言表单验证将失败:
测试用例:
def test_invalid_data(self):
formdata = {
'first_name': 'Bruno',
'last_name': '', # Missing data.
'email': 'bruno@schulz.pl',
'password': 'metamorphosis'
}
request = self.factory.post('/accounts/profile', formdata)
request.user = self.user
setup_messages(request)
response = ProfileView.as_view()(request)
self.assertEqual(response.status_code, 200)
self.assertNotEqual(self.user.first_name, formdata['first_name']) # Fails.
self.assertNotEqual(self.user.last_name, formdata['last_name'])
self.assertNotEqual(self.user.email, formdata['email'])
self.assertNotEqual(self.user.username, formdata['email'])
表单验证在实时服务器上正常工作(已测试 "by hand")。
看起来 ValidationError
在 TestCase
评估期间被忽略了。
*测试输出:
.Form is invalid
F..Form is valid
.
======================================================================
FAIL: test_invalid_data (.tests.ProfileTestCase)
----------------------------------------------------------------------
Traceback (most recent call last):
File "/apps/accounts/tests.py", line 114, in test_invalid_data
self.assertNotEqual(self.user.first_name, formdata['first_name'])
AssertionError: 'Bruno' == 'Bruno'
您没有显示完整视图,但通常在 post 成功后 return redirect('/success-url/')
。如果您的验证错误被忽略,那么视图将重定向状态代码 302,并且您的测试将在前面的行 self.assertEqual(response.status_code, 200)
您可以使用 form = response.context['form']
从视图访问表单。如果您在测试中检查 form.is_valid()
或 form.errors
,您将看到您的 ValidationError
没有被忽略。
您的问题是您的 assertNotEqual
检查没有测试您认为的内容。当您验证模型表单时,实例会被修改。如果要查看数据库中用户是否被修改,需要refresh it from the database first.
self.user.refresh_from_db()
尝试使用 self.add_error('last_name', your_validation_error)
而不是仅仅引发错误:
def clean(self):
first_name = self.cleaned_data.get('first_name', '')
last_name = self.cleaned_data.get('last_name', '')
if not first_name:
print('Form is invalid: missing first name')
# define but don't raise the error
missing_first_name = ValidationError('First name field is required.')
self.add_error('first_name', missing_first_name)
if not last_name:
print('Form is invalid: missing last name')
# define but don't raise error
missing_last_name = ValidationError('Last name field is required.')
self.add_error('last_name', missing_last_name)
# you could also add a string instead of an Error, but a ValidationError is recommended:
# self.add_error('last_name', 'the last name field is missing')
# other stuff
现在 form.is_valid()
将 return 错误。
This SO answer led me to the solution, and here are the add_error docs。您应该使用 self.add_error
将错误添加到表单中,而不是引发验证错误。
(我确实发现了 this example in the docs 错误被引发而不是传递给 self.add_error
,这可能是您混淆的根源,但那是为了清理特定字段。它不是form.clean()
方法。我假设您可以捕获该错误,然后用 self.add_error
添加它...但这是一个需要解决的不同问题。关键是,我没有在form.clean()
应该引发 ValidationError 的文档。应该将其添加到表单中。)