Skip to content

Commit

Permalink
Expose RetryError exceptions.
Browse files Browse the repository at this point in the history
  • Loading branch information
travis-ci committed Mar 1, 2017
1 parent c090fb5 commit f5af6bc
Show file tree
Hide file tree
Showing 2 changed files with 34 additions and 1 deletion.
9 changes: 8 additions & 1 deletion vision/google/cloud/vision/_gax.py
Original file line number Diff line number Diff line change
Expand Up @@ -14,6 +14,8 @@

"""GAX Client for interacting with the Google Cloud Vision API."""

from google.gax.errors import RetryError

from google.cloud.gapic.vision.v1 import image_annotator_client
from google.cloud.proto.vision.v1 import image_annotator_pb2

Expand Down Expand Up @@ -66,7 +68,12 @@ def annotate(self, images=None, requests_pb=None):
requests = requests_pb

annotator_client = self._annotator_client
responses = annotator_client.batch_annotate_images(requests).responses
try:
api_result = annotator_client.batch_annotate_images(requests)
responses = api_result.responses
except RetryError as exception:
raise exception.cause.exception()

return [Annotations.from_pb(response) for response in responses]


Expand Down
26 changes: 26 additions & 0 deletions vision/unit_tests/test__gax.py
Original file line number Diff line number Diff line change
Expand Up @@ -221,6 +221,32 @@ def test_annotate_with_pb_requests_results(self):
self.assertIsInstance(annotation, Annotations)
gax_api._annotator_client.batch_annotate_images.assert_called()

def test_handle_retry_error(self):
from grpc._channel import _Rendezvous
from google.gax.errors import GaxError
from google.gax.errors import RetryError
from google.cloud.proto.vision.v1 import image_annotator_pb2

client = mock.Mock(spec_set=['_credentials'])
request = image_annotator_pb2.AnnotateImageRequest()

# In the case of a RetryError being raised, we want to get to the root
# exception. i.e. RetryError.cause.exception()
mock_rendezvous = mock.Mock(spec=_Rendezvous)
mock_rendezvous.exception.return_value = GaxError('gax')

with mock.patch('google.cloud.vision._gax.image_annotator_client.'
'ImageAnnotatorClient'):
gax_api = self._make_one(client)

gax_api._annotator_client = mock.Mock(
spec_set=['batch_annotate_images'])

gax_api._annotator_client.batch_annotate_images = mock.Mock(
side_effect=RetryError('retry', cause=mock_rendezvous))
with self.assertRaises(GaxError):
gax_api.annotate(requests_pb=[request])


class Test__to_gapic_feature(unittest.TestCase):
def _call_fut(self, feature):
Expand Down

0 comments on commit f5af6bc

Please sign in to comment.