Merge pull request #139 from Kinto/prabhakar267-107-not-raise-Not-found-error-401

Prabhakar267 107 not raise not found error 401

Author: Natim

kinto_http/__init__.py

@@ -255,6 +255,13 @@ def get_bucket(self, bucket=None):
         try:
             resp, _ = self.session.request('get', endpoint)
         except KintoException as e:
+            error_resp_code = e.response.status_code
+            if error_resp_code == 401:
+                msg = ("Unauthorized. Please authenticate or make sure the bucket "
+                       "can be read anonymously.")
+                e = KintoException(msg, e)
+                raise e
+
             raise BucketNotFound(bucket or self._bucket_name, e)
         return resp
 

kinto_http/tests/test_client.py

@@ -285,6 +285,23 @@ def test_unexisting_bucket_raises(self):
         self.assertEquals(e.request, mock.sentinel.request)
         self.assertEquals(e.message, 'test')
 
+    def test_unauthorized_raises_a_kinto_exception(self):
+        # Make the next call to sess.request raise a 401.
+        exception = KintoException()
+        exception.response = mock.MagicMock()
+        exception.response.status_code = 401
+        exception.request = mock.sentinel.request
+        self.session.request.side_effect = exception
+
+        with self.assertRaises(KintoException) as cm:
+            self.client.get_bucket('test')
+        e = cm.exception
+        self.assertEquals(e.response, exception.response)
+        self.assertEquals(e.request, mock.sentinel.request)
+        self.assertEquals(e.message,
+                          "Unauthorized. Please authenticate or make sure the bucket "
+                          "can be read anonymously.")
+
     def test_http_500_raises_an_error(self):
         exception = KintoException()
         exception.response = mock.MagicMock()