1 files changed, 199 insertions, 46 deletions

diff --git a/src/mailman/rest/tests/test_listconf.py b/src/mailman/rest/tests/test_listconf.py
index 37eec5f26..92e616ef7 100644
--- a/src/mailman/rest/tests/test_listconf.py
+++ b/src/mailman/rest/tests/test_listconf.py
@@ -30,6 +30,51 @@ from mailman.interfaces.mailinglist import (
     IAcceptableAliasSet, SubscriptionPolicy)
 from mailman.testing.helpers import call_api
 from mailman.testing.layers import RESTLayer
+from urllib.error import HTTPError
+
+
+# The representation of the listconf resource as a dictionary.  This is
+# required when PUTting to the list's configuration resource.
+RESOURCE = dict(
+    acceptable_aliases=[
+        'ant@example.com',
+        'bee@example.com',
+        'cat@example.com',
+        ],
+    admin_immed_notify=False,
+    admin_notify_mchanges=True,
+    administrivia=False,
+    advertised=False,
+    anonymous_list=True,
+    archive_policy='never',
+    autorespond_owner='respond_and_discard',
+    autorespond_postings='respond_and_continue',
+    autorespond_requests='respond_and_discard',
+    autoresponse_grace_period='45d',
+    autoresponse_owner_text='the owner',
+    autoresponse_postings_text='the mailing list',
+    autoresponse_request_text='the robot',
+    display_name='Fnords',
+    description='This is my mailing list',
+    include_rfc2369_headers=False,
+    allow_list_posts=False,
+    #digest_send_periodic='yes',
+    digest_size_threshold=10.5,
+    #digest_volume_frequency=1,
+    posting_pipeline='virgin',
+    filter_content=True,
+    first_strip_reply_to=True,
+    convert_html_to_plaintext=True,
+    collapse_alternatives=False,
+    reply_goes_to_list='point_to_list',
+    reply_to_address='bee@example.com',
+    send_welcome_message=False,
+    subject_prefix='[ant]',
+    subscription_policy='confirm_then_moderate',
+    welcome_message_uri='mailman:///welcome.txt',
+    default_member_action='hold',
+    default_nonmember_action='discard',
+    )
 
 
 
@@ -40,73 +85,181 @@ class TestConfiguration(unittest.TestCase):
 
     def setUp(self):
         with transaction():
-            self._mlist = create_list('test@example.com')
+            self._mlist = create_list('ant@example.com')
+
+    def test_get_missing_attribute(self):
+        with self.assertRaises(HTTPError) as cm:
+            call_api(
+                'http://localhost:9001/3.0/lists/ant.example.com/config/bogus')
+        self.assertEqual(cm.exception.code, 404)
+        self.assertEqual(cm.exception.reason, b'Unknown attribute: bogus')
 
     def test_put_configuration(self):
-        aliases = [
-            'ant@example.com',
-            'bee@example.com',
-            'cat@example.com',
-            ]
         # When using PUT, all writable attributes must be included.
         resource, response = call_api(
-            'http://localhost:9001/3.0/lists/test@example.com/config',
-            dict(
-                acceptable_aliases=aliases,
-                admin_immed_notify=False,
-                admin_notify_mchanges=True,
-                administrivia=False,
-                advertised=False,
-                anonymous_list=True,
-                archive_policy='never',
-                autorespond_owner='respond_and_discard',
-                autorespond_postings='respond_and_continue',
-                autorespond_requests='respond_and_discard',
-                autoresponse_grace_period='45d',
-                autoresponse_owner_text='the owner',
-                autoresponse_postings_text='the mailing list',
-                autoresponse_request_text='the robot',
-                display_name='Fnords',
-                description='This is my mailing list',
-                include_rfc2369_headers=False,
-                allow_list_posts=False,
-                digest_send_periodic='yes',
-                digest_size_threshold=10.5,
-                digest_volume_frequency=1,
-                posting_pipeline='virgin',
-                filter_content=True,
-                first_strip_reply_to=True,
-                convert_html_to_plaintext=True,
-                collapse_alternatives=False,
-                reply_goes_to_list='point_to_list',
-                reply_to_address='bee@example.com',
-                send_welcome_message=False,
-                subject_prefix='[ant]',
-                subscription_policy='confirm_then_moderate',
-                welcome_message_uri='mailman:///welcome.txt',
-                default_member_action='hold',
-                default_nonmember_action='discard',
-                ),
+            'http://localhost:9001/3.0/lists/ant.example.com/config',
+            RESOURCE,
             'PUT')
         self.assertEqual(response.status, 204)
         self.assertEqual(self._mlist.display_name, 'Fnords')
         # All three acceptable aliases were set.
         self.assertEqual(set(IAcceptableAliasSet(self._mlist).aliases),
-                         set(aliases))
+                         set(RESOURCE['acceptable_aliases']))
+
+    def test_put_attribute(self):
+        resource, response = call_api(
+            'http://localhost:9001/3.0/lists/ant.example.com'
+            '/config/reply_to_address')
+        self.assertEqual(resource['reply_to_address'], '')
+        resource, response = call_api(
+            'http://localhost:9001/3.0/lists/ant.example.com'
+            '/config/reply_to_address',
+            dict(reply_to_address='bar@ant.example.com'),
+            'PUT')
+        self.assertEqual(response.status, 204)
+        resource, response = call_api(
+            'http://localhost:9001/3.0/lists/ant.example.com'
+            '/config/reply_to_address')
+        self.assertEqual(resource['reply_to_address'], 'bar@ant.example.com')
+
+    def test_put_extra_attribute(self):
+        bogus_resource = RESOURCE.copy()
+        bogus_resource['bogus'] = 'yes'
+        with self.assertRaises(HTTPError) as cm:
+            call_api(
+                'http://localhost:9001/3.0/lists/ant.example.com/config',
+                bogus_resource,
+                'PUT')
+        self.assertEqual(cm.exception.code, 400)
+        self.assertEqual(cm.exception.reason, b'Unexpected parameters: bogus')
+
+    def test_put_attribute_mismatch(self):
+        resource, response = call_api(
+            'http://localhost:9001/3.0/lists/ant.example.com'
+            '/config/reply_to_address')
+        self.assertEqual(resource['reply_to_address'], '')
+        with self.assertRaises(HTTPError) as cm:
+            resource, response = call_api(
+                'http://localhost:9001/3.0/lists/ant.example.com'
+                '/config/reply_to_address',
+                dict(display_name='bar@ant.example.com'),
+                'PUT')
+        self.assertEqual(cm.exception.code, 400)
+        self.assertEqual(cm.exception.reason,
+                         b'Unexpected parameters: display_name')
+
+    def test_put_attribute_double(self):
+        with self.assertRaises(HTTPError) as cm:
+            resource, response = call_api(
+                'http://localhost:9001/3.0/lists/ant.example.com'
+                '/config/reply_to_address',
+                dict(display_name='bar@ant.example.com',
+                     reply_to_address='foo@example.com'),
+                'PUT')
+        self.assertEqual(cm.exception.code, 400)
+        self.assertEqual(cm.exception.reason,
+                         b'Unexpected parameters: display_name')
+
+    def test_put_read_only_attribute(self):
+        with self.assertRaises(HTTPError) as cm:
+            call_api('http://localhost:9001/3.0/lists/ant.example.com'
+                     '/config/mail_host',
+                     dict(mail_host='foo.example.com'),
+                    'PUT')
+        self.assertEqual(cm.exception.code, 400)
+        self.assertEqual(cm.exception.reason,
+                         b'Read-only attribute: mail_host')
+
+    def test_put_missing_attribute(self):
+        with self.assertRaises(HTTPError) as cm:
+            call_api(
+                'http://localhost:9001/3.0/lists/ant.example.com/config/bogus',
+                dict(bogus='no matter'),
+                'PUT')
+        self.assertEqual(cm.exception.code, 404)
+        self.assertEqual(cm.exception.reason, b'Unknown attribute: bogus')
 
     def test_patch_subscription_policy(self):
         # The new subscription_policy value can be patched.
         #
         # To start with, the subscription policy is confirm by default.
         resource, response = call_api(
-            'http://localhost:9001/3.0/lists/test@example.com/config')
+            'http://localhost:9001/3.0/lists/ant.example.com/config')
         self.assertEqual(resource['subscription_policy'], 'confirm')
         # Let's patch it to do some moderation.
         resource, response = call_api(
-            'http://localhost:9001/3.0/lists/test@example.com/config', dict(
+            'http://localhost:9001/3.0/lists/ant.example.com/config', dict(
                 subscription_policy='confirm_then_moderate'),
             method='PATCH')
         self.assertEqual(response.status, 204)
         # And now we verify that it has the requested setting.
         self.assertEqual(self._mlist.subscription_policy,
                          SubscriptionPolicy.confirm_then_moderate)
+
+    ## def test_digest_volume_frequency(self):
+    ##     resource, response = call_api(
+    ##         'http://localhost:9001/3.0/lists/test.example.com/config',
+    ##         dict(digest_volume_frequency=DigestFrequency.never),
+    ##         'PATCH')
+    ##     self.assertEqual(self._mlist.digest_volume_frequency,
+    ##                      DigestFrequency.never)
+
+    def test_patch_attribute_double(self):
+        with self.assertRaises(HTTPError) as cm:
+            resource, response = call_api(
+                'http://localhost:9001/3.0/lists/ant.example.com'
+                '/config/reply_to_address',
+                dict(display_name='bar@ant.example.com',
+                     reply_to_address='foo'),
+                'PATCH')
+        self.assertEqual(cm.exception.code, 400)
+        self.assertEqual(cm.exception.reason, b'Expected 1 attribute, got 2')
+
+    def test_unknown_patch_attribute(self):
+        with self.assertRaises(HTTPError) as cm:
+            call_api('http://localhost:9001/3.0/lists/ant.example.com/config',
+                    dict(bogus=1),
+                    'PATCH')
+        self.assertEqual(cm.exception.code, 404)
+        self.assertEqual(cm.exception.reason, b'Unknown attribute: bogus')
+
+    def test_read_only_patch_attribute(self):
+        with self.assertRaises(HTTPError) as cm:
+            call_api('http://localhost:9001/3.0/lists/ant.example.com'
+                     '/config/mail_host',
+                     dict(mail_host='foo.example.com'),
+                    'PATCH')
+        self.assertEqual(cm.exception.code, 400)
+        self.assertEqual(cm.exception.reason,
+                         b'Read-only attribute: mail_host')
+
+    def test_patch_missing_attribute(self):
+        with self.assertRaises(HTTPError) as cm:
+            call_api(
+                'http://localhost:9001/3.0/lists/ant.example.com/config/bogus',
+                dict(bogus='no matter'),
+                'PATCH')
+        self.assertEqual(cm.exception.code, 404)
+        self.assertEqual(cm.exception.reason, b'Unknown attribute: bogus')
+
+    def test_patch_bad_value(self):
+        with self.assertRaises(HTTPError) as cm:
+            call_api(
+                'http://localhost:9001/3.0/lists/ant.example.com/config'
+                '/archive_policy',
+                dict(archive_policy='not a valid archive policy'),
+                'PATCH')
+        self.assertEqual(cm.exception.code, 400)
+        self.assertEqual(cm.exception.reason,
+                         b'Cannot convert parameters: archive_policy')
+
+    def test_bad_pipeline_name(self):
+        with self.assertRaises(HTTPError) as cm:
+            call_api(
+                'http://localhost:9001/3.0/lists/ant.example.com/config'
+                '/posting_pipeline',
+                dict(posting_pipeline='not a valid pipeline'),
+                'PATCH')
+        self.assertEqual(cm.exception.code, 400)
+        self.assertEqual(cm.exception.reason,
+                         b'Cannot convert parameters: posting_pipeline')