【问题标题】:Form valid in unit test even though ValidationError was raised即使引发了 ValidationError,表单在单元测试中也有效
【发布时间】:2018-01-18 17:53:26
【问题描述】:

我正在测试处理无效表单数据的视图。在我的测试用例中,我正在提交缺少字段的表单,并希望视图通过显示错误消息来处理它。这是我的表单中来自clean 的相关sn-p:

表格:

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() 仍然返回 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'])

表单验证在实时服务器上正常工作(“手动”测试)。

TestCase 评估期间,ValidationError 似乎被忽略了。

*测试输出:

.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'

【问题讨论】:

    标签: python django unit-testing django-forms django-testing


    【解决方案1】:

    您尚未展示您的完整视图,但通常您会在成功发布后展示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()
    

    【讨论】:

    • "然后视图将重定向状态码 302' - Bingo,是的,我忘记了这个断言。我发现 is_valid()-branch 实际上没有执行,我正在查看输出不同的测试(duh)(我在描述中省略了大部分代码)。尽管如此,用户实例在其他测试中发生了更改,因此在不刷新它的情况下断言必须失败。应用refresh_from_db解决了这个问题。非常感谢!
    【解决方案2】:

    尝试使用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() 将返回 False。

    This SO answer 引导我找到解决方案,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 的示例。它应该将其添加到表单中。)

    【讨论】:

      猜你喜欢
      • 2022-11-06
      • 2010-11-10
      • 1970-01-01
      • 2018-11-10
      • 1970-01-01
      • 2013-09-28
      • 2021-05-25
      • 1970-01-01
      • 1970-01-01
      相关资源
      最近更新 更多