Commit af088b8d authored by Aurélien Bompard's avatar Aurélien Bompard

The MailingList.info field is not required

Fixes #155
parent c95e5ca1
......@@ -588,6 +588,7 @@ class ListIdentityForm(ListSettingsForm):
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'),
......
......@@ -19,8 +19,9 @@ from __future__ import absolute_import, unicode_literals
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 @@ class ListNewTest(TestCase):
'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 @@ class ListNewTest(TestCase):
'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)
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment