]> review.fuel-infra Code Review - openstack-build/cinder-build.git/commitdiff
Added a test for bad limit param
authorDerek Chiang <derek.chiang@nebula.com>
Wed, 22 May 2013 05:07:20 +0000 (05:07 +0000)
committerDerek Chiang <derek.chiang@nebula.com>
Wed, 22 May 2013 18:11:11 +0000 (18:11 +0000)
Change-Id: Ic9a93779d5623644ccf210de622d59f999571dd2

cinder/tests/api/test_common.py

index f6bdec94c009d1c8f3d9ef482dda0a7f787d7b64..1a8eba04000a0b792df2c137696605fef4c6ccc2 100644 (file)
@@ -98,6 +98,12 @@ class LimiterTest(test.TestCase):
         self.assertEqual(common.limited(self.medium, req), self.medium)
         self.assertEqual(common.limited(self.large, req), self.large[:1000])
 
+    def test_limiter_limit_bad(self):
+        """ Test with a bad limit. """
+        req = webob.Request.blank(u'/?limit=hello')
+        self.assertRaises(
+            webob.exc.HTTPBadRequest, common.limited, self.tiny, req)
+
     def test_limiter_limit_medium(self):
         """ Test limit of 10. """
         req = webob.Request.blank('/?limit=10')