diff options
| -rw-r--r-- | src/mailman/docs/NEWS.rst | 2 | ||||
| -rw-r--r-- | src/mailman/rest/tests/test_root.py | 74 |
2 files changed, 44 insertions, 32 deletions
diff --git a/src/mailman/docs/NEWS.rst b/src/mailman/docs/NEWS.rst index 301221ce1..53a24cbe2 100644 --- a/src/mailman/docs/NEWS.rst +++ b/src/mailman/docs/NEWS.rst @@ -173,6 +173,8 @@ REST * Expose `goodbye_message_uri` in the REST API. Given by Harshit Bansal. * New subscription requests are rejected if there is already one pending. With thanks to Anirudh Dahiya. (Closes #199) + * Expose the system pipelines and chains via ``<api>/system/pipelines`` and + ``<api>/system/chains`` respectively. Given by Simon Hanna. (Closes #66) Other ----- diff --git a/src/mailman/rest/tests/test_root.py b/src/mailman/rest/tests/test_root.py index 164f27f91..dc3acce20 100644 --- a/src/mailman/rest/tests/test_root.py +++ b/src/mailman/rest/tests/test_root.py @@ -63,38 +63,6 @@ class TestRoot(unittest.TestCase): call_api('http://localhost:9001/3.0/system/foo') self.assertEqual(cm.exception.code, 404) - def test_system_pipelines_are_exposed(self): - json, response = call_api('http://localhost:9001/3.0/system/pipelines') - self.assertEqual(json['pipelines'], sorted(config.pipelines)) - - def test_system_pipelines_are_read_only(self): - with self.assertRaises(HTTPError) as cm: - call_api('http://localhost:9001/3.0/system/pipelines', { - 'pipelines': ['pipeline_1', 'pipeline_2'] - }, method='PATCH') - self.assertEqual(cm.exception.code, 405) - with self.assertRaises(HTTPError) as cm: - call_api('http://localhost:9001/3.0/system/pipelines', { - 'pipelines': ['pipeline_1', 'pipeline_2'] - }, method='PUT') - self.assertEqual(cm.exception.code, 405) - - def test_system_chains_are_exposed(self): - json, response = call_api('http://localhost:9001/3.0/system/chains') - self.assertEqual(json['chains'], sorted(config.chains)) - - def test_system_chains_are_read_only(self): - with self.assertRaises(HTTPError) as cm: - call_api('http://localhost:9001/3.0/system/chains', { - 'chains': ['chain_1', 'chain_2'] - }, method='PATCH') - self.assertEqual(cm.exception.code, 405) - with self.assertRaises(HTTPError) as cm: - call_api('http://localhost:9001/3.0/system/chains', { - 'chains': ['chain_1', 'chain_2'] - }, method='PUT') - self.assertEqual(cm.exception.code, 405) - def test_system_preferences_are_read_only(self): # /system/preferences are read-only. with self.assertRaises(HTTPError) as cm: @@ -157,3 +125,45 @@ class TestRoot(unittest.TestCase): call_api('http://localhost:9001/3.0/reserved/uids/assigned', method='DELETE') self.assertEqual(cm.exception.code, 404) + + def test_system_pipelines_are_exposed(self): + json, response = call_api('http://localhost:9001/3.0/system/pipelines') + self.assertEqual(json['pipelines'], sorted(config.pipelines)) + + def test_system_pipelines_bad_request(self): + with self.assertRaises(HTTPError) as cm: + call_api('http://localhost:9001/3.0/system/pipelines/bogus') + self.assertEqual(cm.exception.code, 400) + + def test_system_pipelines_are_read_only(self): + with self.assertRaises(HTTPError) as cm: + call_api('http://localhost:9001/3.0/system/pipelines', { + 'pipelines': ['pipeline_1', 'pipeline_2'] + }, method='PATCH') + self.assertEqual(cm.exception.code, 405) + with self.assertRaises(HTTPError) as cm: + call_api('http://localhost:9001/3.0/system/pipelines', { + 'pipelines': ['pipeline_1', 'pipeline_2'] + }, method='PUT') + self.assertEqual(cm.exception.code, 405) + + def test_system_chains_are_exposed(self): + json, response = call_api('http://localhost:9001/3.0/system/chains') + self.assertEqual(json['chains'], sorted(config.chains)) + + def test_system_chains_bad_request(self): + with self.assertRaises(HTTPError) as cm: + call_api('http://localhost:9001/3.0/system/chains/bogus') + self.assertEqual(cm.exception.code, 400) + + def test_system_chains_are_read_only(self): + with self.assertRaises(HTTPError) as cm: + call_api('http://localhost:9001/3.0/system/chains', { + 'chains': ['chain_1', 'chain_2'] + }, method='PATCH') + self.assertEqual(cm.exception.code, 405) + with self.assertRaises(HTTPError) as cm: + call_api('http://localhost:9001/3.0/system/chains', { + 'chains': ['chain_1', 'chain_2'] + }, method='PUT') + self.assertEqual(cm.exception.code, 405) |