Merge lp:~prakhar126/postorius/postorius into lp:postorius

[pjoshi]

Adding test cases for creating new list.

[Terri (terriko)]

I'm getting the following error:

  File "/Users/terri/Code/mailman/postorius/src/postorius/tests/test_forms.py", line 58
    'mail_host': 'mailman.most-desirable.org'
               ^
SyntaxError: invalid syntax

If you could tidy that up, that would be great!

review: Needs Fixing

Unmerged revisions

204. By pjoshi on 2015-03-10

adding test case for new list form

=== modified file 'src/postorius/templates/postorius/user_tasks.html'
--- src/postorius/templates/postorius/user_tasks.html	2014-10-15 20:59:52 +0000
+++ src/postorius/templates/postorius/user_tasks.html	2015-03-10 12:00:51 +0000
@@ -4,5 +4,7 @@
 
 {% block main %}
     <h1>{% trans "To Do" %}</h1>
-    <p>{% trans "There are no pending tasks at the present time." %}</p>
+    {% for tasks in memberships %}
+    	<p> {% trans tasks %}</p>
+    {% endfor %}
 {% endblock main %}

=== modified file 'src/postorius/tests/test_forms.py'
--- src/postorius/tests/test_forms.py	2015-02-09 14:35:44 +0000
+++ src/postorius/tests/test_forms.py	2015-03-10 12:00:51 +0000
@@ -49,3 +49,26 @@
             'contact_address': '[e-mail]',
         })
         self.assertFalse(form.is_valid())
+
+class ListNewTest(unittest.TestCase):
+
+    def test_form_fields_list(self):
+        form = ListNew({
+            'listname': 'xyz'
+            'mail_host': 'mailman.most-desirable.org'
+            'list_owner': '[e-mail]'
+            'advertise': 'abcd'
+            'description': 'The Most Desirable organization'
+        })
+        self.assertTrue(form.is_valid)
+
+    def test_form_fields_list_invalid(self):
+        form = ListNew({
+            'listname': ''xy#z'
+            'mail_host': 'mailman.most-desirable.org'
+            'list_owner': 'mailman.most-desirable.org'
+            'advertise': 'abcd'
+            'description': 'The Most Desirable organization'
+        })
+        self.assertFalse(form.is_valid)
+        
\ No newline at end of file