Merge "Separate Forbidden exception from Unauthorized"

tempest_lib/common/rest_client.py

@@ -423,9 +423,12 @@ class RestClient(object):
         else:
             raise exceptions.InvalidContentType(str(resp.status))
 
-        if resp.status == 401 or resp.status == 403:
+        if resp.status == 401:
             raise exceptions.Unauthorized(resp_body)
 
+        if resp.status == 403:
+            raise exceptions.Forbidden(resp_body)
+
         if resp.status == 404:
             raise exceptions.NotFound(resp_body)
 

tempest_lib/exceptions.py

@@ -62,6 +62,10 @@ class Unauthorized(RestClientException):
     message = 'Unauthorized'
 
 
+class Forbidden(RestClientException):
+    message = "Forbidden"
+
+
 class TimeoutException(RestClientException):
     message = "Request timed out"
 

tempest_lib/tests/test_rest_client.py

@@ -315,7 +315,7 @@ class TestRestClientErrorCheckerJSON(base.TestCase):
                           **self.set_data("401"))
 
     def test_response_403(self):
-        self.assertRaises(exceptions.Unauthorized,
+        self.assertRaises(exceptions.Forbidden,
                           self.rest_client._error_checker,
                           **self.set_data("403"))