From e95e6425ac354b8fb0c24580b740dbdc2ea72bd6 Mon Sep 17 00:00:00 2001 From: Baptiste Mispelon Date: Sat, 11 Dec 2021 20:17:29 +0100 Subject: Refs #24121 -- Added __repr__() to BaseFormSet. --- tests/forms_tests/tests/test_formsets.py | 63 ++++++++++++++++++++++++++++++++ 1 file changed, 63 insertions(+) (limited to 'tests/forms_tests') diff --git a/tests/forms_tests/tests/test_formsets.py b/tests/forms_tests/tests/test_formsets.py index 1705d8bb6b..50342cfaa9 100644 --- a/tests/forms_tests/tests/test_formsets.py +++ b/tests/forms_tests/tests/test_formsets.py @@ -25,6 +25,12 @@ class Choice(Form): ChoiceFormSet = formset_factory(Choice) +class ChoiceFormsetWithNonFormError(ChoiceFormSet): + def clean(self): + super().clean() + raise ValidationError('non-form error') + + class FavoriteDrinkForm(Form): name = CharField() @@ -1328,6 +1334,63 @@ class FormsFormsetTestCase(SimpleTestCase): self.assertEqual(formset.non_form_errors().renderer, renderer) self.assertEqual(formset.empty_form.renderer, renderer) + def test_repr(self): + valid_formset = self.make_choiceformset([('test', 1)]) + valid_formset.full_clean() + invalid_formset = self.make_choiceformset([('test', '')]) + invalid_formset.full_clean() + partially_invalid_formset = self.make_choiceformset( + [('test', '1'), ('test', '')], + ) + partially_invalid_formset.full_clean() + invalid_formset_non_form_errors_only = self.make_choiceformset( + [('test', '')], + formset_class=ChoiceFormsetWithNonFormError, + ) + invalid_formset_non_form_errors_only.full_clean() + + cases = [ + ( + self.make_choiceformset(), + '', + ), + ( + self.make_choiceformset( + formset_class=formset_factory(Choice, extra=10), + ), + '', + ), + ( + self.make_choiceformset([]), + '', + ), + ( + self.make_choiceformset([('test', 1)]), + '', + ), + (valid_formset, ''), + (invalid_formset, ''), + ( + partially_invalid_formset, + '', + ), + ( + invalid_formset_non_form_errors_only, + '', + ), + ] + for formset, expected_repr in cases: + with self.subTest(expected_repr=expected_repr): + self.assertEqual(repr(formset), expected_repr) + + def test_repr_do_not_trigger_validation(self): + formset = self.make_choiceformset([('test', 1)]) + with mock.patch.object(formset, 'full_clean') as mocked_full_clean: + repr(formset) + mocked_full_clean.assert_not_called() + formset.is_valid() + mocked_full_clean.assert_called() + @jinja2_tests class Jinja2FormsFormsetTestCase(FormsFormsetTestCase): -- cgit v1.3