From d64f192df188903ea5a1ab8dfd6679e5d52aaf9b Mon Sep 17 00:00:00 2001 From: Stephen Burrows Date: Thu, 15 May 2014 20:12:32 -0700 Subject: Fixed #22628 -- Took initial forms into account when combining FormSet.min_num and FormSet.extra. Forwardport of 79f15ab1ef from stable/1.7.x --- tests/forms_tests/tests/test_formsets.py | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) (limited to 'tests/forms_tests') diff --git a/tests/forms_tests/tests/test_formsets.py b/tests/forms_tests/tests/test_formsets.py index 52dfdf4044..6b82f57343 100644 --- a/tests/forms_tests/tests/test_formsets.py +++ b/tests/forms_tests/tests/test_formsets.py @@ -216,7 +216,7 @@ class FormsFormsetTestCase(TestCase): def test_min_num_displaying_more_than_one_blank_form(self): # We can also display more than 1 empty form passing min_num argument - # to formset_factory. It will increment the extra argument + # to formset_factory. It will (essentially) increment the extra argument ChoiceFormSet = formset_factory(Choice, extra=1, min_num=1) formset = ChoiceFormSet(auto_id=False, prefix='choices') @@ -225,6 +225,10 @@ class FormsFormsetTestCase(TestCase): for form in formset.forms: form_output.append(form.as_ul()) + # Min_num forms are required; extra forms can be empty. + self.assertFalse(formset.forms[0].empty_permitted) + self.assertTrue(formset.forms[1].empty_permitted) + self.assertHTMLEqual('\n'.join(form_output), """
  • Choice:
  • Votes:
  • Choice:
  • -- cgit v1.3