Add missing tests of app initialization

mozilla-services · Sep 29, 2015 · e8a2e21 · e8a2e21
1 parent 3fb98e4
commit e8a2e21
Showing 1 changed file with 13 additions and 2 deletions.
diff --git a/cliquet/tests/resource/test_views_cors.py b/cliquet/tests/resource/test_views_cors.py
@@ -181,7 +181,7 @@ def test_present_on_unknown_url(self):
             'Alert', 'Backoff', 'Retry-After', 'Content-Length'], status=404)
 
 
-class CORSMaxAgeTest(BaseWebTest):
+class CORSMaxAgeTest(BaseWebTest, unittest.TestCase):
     def setUp(self):
         super(CORSMaxAgeTest, self).setUp()
         self.headers.update({
@@ -190,5 +190,16 @@ def setUp(self):
         })
 
     def test_cors_max_age_is_3600_seconds_by_default(self):
-        resp = self.app.options('/', headers=self.headers)
+        app = self.get_test_app()
+        resp = app.options('/', headers=self.headers)
         self.assertEqual(int(resp.headers['Access-Control-Max-Age']), 3600)
+
+    def test_cors_max_age_can_be_specified_in_settings(self):
+        app = self.get_test_app({'cliquet.cors_max_age': '42'})
+        resp = app.options('/', headers=self.headers)
+        self.assertEqual(int(resp.headers['Access-Control-Max-Age']), 42)
+
+    def test_cors_max_age_is_disabled_if_unset(self):
+        app = self.get_test_app({'cliquet.cors_max_age': ''})
+        resp = app.options('/', headers=self.headers)
+        self.assertNotIn('Access-Control-Max-Age', resp.headers)