diff options
author | Baptiste Mispelon <bmispelon@gmail.com> | 2021-12-10 12:22:23 +0100 |
---|---|---|
committer | Mariusz Felisiak <felisiak.mariusz@gmail.com> | 2022-04-06 07:58:52 +0200 |
commit | 50e1e7ef8ef63271717f8bcab76d7151ccf4bb18 (patch) | |
tree | 888beb2bff18679025536e574a0bc02a53f9d552 /tests/test_client | |
parent | 1a7d75cf77639e450854d9bcf9518664f755eb04 (diff) | |
download | django-50e1e7ef8ef63271717f8bcab76d7151ccf4bb18.tar.gz |
Fixed #33348 -- Changed SimpleTestCase.assertFormError()/assertFormsetErrors() to take form/formset.
Instead of taking a response object and a context name for
the form/formset, the two methods now take the object directly.
Diffstat (limited to 'tests/test_client')
-rw-r--r-- | tests/test_client/tests.py | 24 |
1 files changed, 14 insertions, 10 deletions
diff --git a/tests/test_client/tests.py b/tests/test_client/tests.py index 8fcf3e046d..f13b05f25b 100644 --- a/tests/test_client/tests.py +++ b/tests/test_client/tests.py @@ -437,10 +437,10 @@ class ClientTest(TestCase): response = self.client.post("/form_view/", post_data) self.assertContains(response, "This field is required.", 3) self.assertTemplateUsed(response, "Invalid POST Template") - - self.assertFormError(response, "form", "email", "This field is required.") - self.assertFormError(response, "form", "single", "This field is required.") - self.assertFormError(response, "form", "multi", "This field is required.") + form = response.context["form"] + self.assertFormError(form, "email", "This field is required.") + self.assertFormError(form, "single", "This field is required.") + self.assertFormError(form, "multi", "This field is required.") def test_form_error(self): "POST erroneous data to a form" @@ -455,7 +455,9 @@ class ClientTest(TestCase): self.assertEqual(response.status_code, 200) self.assertTemplateUsed(response, "Invalid POST Template") - self.assertFormError(response, "form", "email", "Enter a valid email address.") + self.assertFormError( + response.context["form"], "email", "Enter a valid email address." + ) def test_valid_form_with_template(self): "POST valid data to a form using multiple templates" @@ -480,10 +482,10 @@ class ClientTest(TestCase): self.assertTemplateUsed(response, "form_view.html") self.assertTemplateUsed(response, "base.html") self.assertTemplateNotUsed(response, "Invalid POST Template") - - self.assertFormError(response, "form", "email", "This field is required.") - self.assertFormError(response, "form", "single", "This field is required.") - self.assertFormError(response, "form", "multi", "This field is required.") + form = response.context["form"] + self.assertFormError(form, "email", "This field is required.") + self.assertFormError(form, "single", "This field is required.") + self.assertFormError(form, "multi", "This field is required.") def test_form_error_with_template(self): "POST erroneous data to a form using multiple templates" @@ -500,7 +502,9 @@ class ClientTest(TestCase): self.assertTemplateUsed(response, "base.html") self.assertTemplateNotUsed(response, "Invalid POST Template") - self.assertFormError(response, "form", "email", "Enter a valid email address.") + self.assertFormError( + response.context["form"], "email", "Enter a valid email address." + ) def test_unknown_page(self): "GET an invalid URL" |