3 files changed, 54 insertions, 9 deletions
diff --git a/src/mailman/docs/NEWS.rst b/src/mailman/docs/NEWS.rst
index 35709846e..17d5bdc4b 100644
--- a/src/mailman/docs/NEWS.rst
+++ b/src/mailman/docs/NEWS.rst
@@ -23,6 +23,7 @@ Architecture
    It now properly supports email verification and subscription confirmation
    by the user, and approval by the moderator using unique tokens.
    ``IMailingList`` objects now have a ``subscription_policy`` attribute.
+   (LP: #1095552)
 
 Bugs
 ----
diff --git a/src/mailman/rest/tests/test_basic.py b/src/mailman/rest/tests/test_basic.py
new file mode 100644
index 000000000..839ce5b97
--- /dev/null
+++ b/src/mailman/rest/tests/test_basic.py
@@ -0,0 +1,53 @@
+# Copyright (C) 2015 by the Free Software Foundation, Inc.
+#
+# This file is part of GNU Mailman.
+#
+# GNU Mailman is free software: you can redistribute it and/or modify it under
+# the terms of the GNU General Public License as published by the Free
+# Software Foundation, either version 3 of the License, or (at your option)
+# any later version.
+#
+# GNU Mailman is distributed in the hope that it will be useful, but WITHOUT
+# ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
+# FITNESS FOR A PARTICULAR PURPOSE.  See the GNU General Public License for
+# more details.
+#
+# You should have received a copy of the GNU General Public License along with
+# GNU Mailman.  If not, see <http://www.gnu.org/licenses/>.
+
+"""Test basic functionality of the REST API.
+
+For example, test the integration between Mailman and Falcon.
+"""
+
+__all__ = [
+    'TestBasicREST',
+    ]
+
+
+import unittest
+
+from mailman.app.lifecycle import create_list
+from mailman.database.transaction import transaction
+from mailman.testing.helpers import call_api
+from mailman.testing.layers import RESTLayer
+
+
+
+class TestBasicREST(unittest.TestCase):
+    """Test basic REST integration and functionality."""
+
+    layer = RESTLayer
+
+    def setUp(self):
+        with transaction():
+            self._mlist = create_list('test@example.com')
+
+    def test_comma_fields(self):
+        resource, response = call_api(
+            'http://localhost:9001/3.0/lists/test@example.com/config',
+            dict(description='A description with , to check stuff'),
+            method='PATCH')
+        # This fails with Falcon 0.2; passes with Falcon 0.3.
+        self.assertEqual(self._mlist.description,
+                         'A description with , to check stuff')
diff --git a/src/mailman/rest/tests/test_listconf.py b/src/mailman/rest/tests/test_listconf.py
index 0c36ef30a..860adac57 100644
--- a/src/mailman/rest/tests/test_listconf.py
+++ b/src/mailman/rest/tests/test_listconf.py
@@ -92,7 +92,6 @@ class TestConfiguration(unittest.TestCase):
         self.assertEqual(set(IAcceptableAliasSet(self._mlist).aliases),
                          set(aliases))
 
-
     def test_patch_subscription_policy(self):
         # The new subscription_policy value can be patched.
         #
@@ -109,11 +108,3 @@ class TestConfiguration(unittest.TestCase):
         # And now we verify that it has the requested setting.
         self.assertEqual(self._mlist.subscription_policy,
                          SubscriptionPolicy.confirm_then_moderate)
-
-    @unittest.expectedFailure
-    def test_bad_description_update(self):
-        resource, response = call_api(
-            'http://localhost:9001/3.0/lists/test@example.com/config',
-            dict(description='A description with , to check stuff'), 'PATCH')
-        self.assertEqual(self._mlist.description,
-                         'A description with , to check stuff')