即使引发了 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")。

看起来 ValidationErrorTestCase 评估期间被忽略了。

*测试输出:

.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 的文档。应该将其添加到表单中。)