Skip to content

Commit

Permalink
Adding GCCL header for HTTP APIs. (#3046)
Browse files Browse the repository at this point in the history
  • Loading branch information
dhermes authored and lukesneeringer committed Feb 22, 2017
1 parent 4dee135 commit e730b60
Show file tree
Hide file tree
Showing 3 changed files with 45 additions and 0 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -14,6 +14,10 @@

"""Google Cloud Translation API wrapper."""


from pkg_resources import get_distribution
__version__ = get_distribution('google-cloud-translate').version

from google.cloud.translate.client import BASE
from google.cloud.translate.client import Client
from google.cloud.translate.client import NMT
Original file line number Diff line number Diff line change
Expand Up @@ -16,6 +16,11 @@

from google.cloud import _http

from google.cloud.translate import __version__


_CLIENT_INFO = _http.CLIENT_INFO_TEMPLATE.format(__version__)


class Connection(_http.JSONConnection):
"""A connection to Google Cloud Translation API via the JSON REST API.
Expand All @@ -32,3 +37,7 @@ class Connection(_http.JSONConnection):

API_URL_TEMPLATE = '{api_base_url}/language/translate/{api_version}{path}'
"""A template for the URL of a particular API call."""

_EXTRA_HEADERS = {
_http.CLIENT_INFO_HEADER: _CLIENT_INFO,
}
32 changes: 32 additions & 0 deletions packages/google-cloud-translate/unit_tests/test__http.py
Original file line number Diff line number Diff line change
Expand Up @@ -14,6 +14,8 @@

import unittest

import mock


class TestConnection(unittest.TestCase):

Expand Down Expand Up @@ -52,3 +54,33 @@ def test_build_api_url_w_extra_query_params(self):
self.assertEqual(path, expected_path)
params = parse_qsl(qs)
self.assertEqual(params, query_params)

def test_extra_headers(self):
from google.cloud import _http as base_http
from google.cloud.translate import _http as MUT

http = mock.Mock(spec=['request'])
response = mock.Mock(status=200, spec=['status'])
data = b'brent-spiner'
http.request.return_value = response, data
client = mock.Mock(_http=http, spec=['_http'])

conn = self._make_one(client)
req_data = 'req-data-boring'
result = conn.api_request(
'GET', '/rainbow', data=req_data, expect_json=False)
self.assertEqual(result, data)

expected_headers = {
'Content-Length': str(len(req_data)),
'Accept-Encoding': 'gzip',
base_http.CLIENT_INFO_HEADER: MUT._CLIENT_INFO,
'User-Agent': conn.USER_AGENT,
}
expected_uri = conn.build_api_url('/rainbow')
http.request.assert_called_once_with(
body=req_data,
headers=expected_headers,
method='GET',
uri=expected_uri,
)

0 comments on commit e730b60

Please sign in to comment.