diff --git a/src/postorius/forms.py b/src/postorius/forms.py index 47edc9b..68a4075 100644 --- a/src/postorius/forms.py +++ b/src/postorius/forms.py @@ -588,6 +588,7 @@ info = forms.CharField( label=_('Information'), help_text=_('A longer description of this mailing list.'), + required=False, widget=forms.Textarea()) display_name = forms.CharField( label=_('Display name'), diff --git a/src/postorius/tests/test_forms.py b/src/postorius/tests/test_forms.py index 9157c8e..9bd6256 100644 --- a/src/postorius/tests/test_forms.py +++ b/src/postorius/tests/test_forms.py @@ -19,8 +19,9 @@ from django.test import TestCase -from postorius.forms import (ListNew, UserPreferences, DomainNew, - ListSubscribe, ChangeSubscriptionForm) +from postorius.forms import ( + ChangeSubscriptionForm, DomainNew, ListIdentityForm, ListNew, + ListSubscribe, UserPreferences) class UserPreferencesTest(TestCase): @@ -92,7 +93,7 @@ 'advertise': 'abcd', 'description': 'The Most Desirable organization', }) - self.assertTrue(form.is_valid) + self.assertTrue(form.is_valid()) def test_form_fields_list_invalid(self): form = ListNew({ @@ -103,3 +104,15 @@ 'description': 'The Most Desirable organization', }) self.assertFalse(form.is_valid()) + + +class ListIdentityTest(TestCase): + + def test_info_not_required(self): + form = ListIdentityForm({ + 'advertised': 'True', + 'description': 'The Most Desirable organization', + 'display_name': 'Most Desirable', + 'subject_prefix': '[Most Desirable] ', + }, mlist=None) + self.assertTrue(form.is_valid(), form.errors)