Merge pull request #19 from sshedmake/fixes_for_zanata-python-client

FIXES rhbz 1207303, return code now is 1 if push fails
https://bugzilla.redhat.com/show_bug.cgi?id=1207303

zanataclient/zanatalib/error.py

@@ -28,7 +28,7 @@
         "ProjectExistException", "UnAvaliableResourceException",
         "UnAvaliablePOTException", "BadRequestBodyException",
         "SameNameDocumentException","InternalServerError",
-        "NotAllowedException", "UnavailableServiceError",
+        "NotAllowedException", "UnavailableServiceError", "ForbiddenException",
         "UnexpectedStatusException"
 )
 class ZanataException(Exception):
@@ -80,5 +80,8 @@ class SameNameDocumentException(ZanataException):
 class NotAllowedException(ZanataException):
     pass
 
+class ForbiddenException(ZanataException):
+    pass
+
 class UnexpectedStatusException(ZanataException):
     pass

zanataclient/zanatalib/service.py

@@ -8,9 +8,11 @@
 from error import ProjectExistException
 from error import NotAllowedException
 from error import SameNameDocumentException
+from error import ForbiddenException
 from rest.client import RestClient
 
 import json
+import sys
 
 __all__ = (
         "Service",
@@ -59,6 +61,13 @@ def messages(self,res,content,extra_msg=None):
         elif res['status'] == '503':
             raise UnavailableServiceError('Error 503', 
                                           'Service Temporarily Unavailable, stop processing!')
+        elif res['status'] == '403':
+            try:
+               raise ForbiddenException('Error 403', 'You are authenticated but do not have the permission for the requested resource.')
+            except ForbiddenException as e:
+                print '', e
+            finally:
+                sys.exit(1)
         else:
             raise UnexpectedStatusException('Error', 
                                             'Unexpected Status (%s), failed to push: %s' % (res['status'], extra_msg or ""))