Skip to content

Commit be60c8e

Browse files
committed
Adapts codebase to >= 2.7
1 parent 20315da commit be60c8e

File tree

2 files changed

+9
-6
lines changed

2 files changed

+9
-6
lines changed

python_http_client/client.py

Lines changed: 6 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -64,7 +64,7 @@ class Client(object):
6464
"""Quickly and easily access any REST or REST-like API."""
6565

6666
# These are the supported HTTP verbs
67-
methods = set(('delete', 'get', 'patch', 'post', 'put'))
67+
methods = {'delete', 'get', 'patch', 'post', 'put'}
6868

6969
def __init__(self,
7070
host,
@@ -242,7 +242,8 @@ def http_request(
242242
# Don't serialize to a JSON formatted str
243243
# if we don't have a JSON Content-Type
244244
if 'Content-Type' in self.request_headers and \
245-
self.request_headers['Content-Type'] != 'application/json':
245+
self.request_headers['Content-Type'] != \
246+
'application/json':
246247
data = request_body.encode('utf-8')
247248
else:
248249
self.request_headers.setdefault(
@@ -257,7 +258,9 @@ def http_request(
257258
)
258259
request.get_method = lambda: method
259260

260-
return Response(self._make_request(opener, request, timeout=timeout))
261+
return Response(
262+
self._make_request(opener, request, timeout=timeout)
263+
)
261264
return http_request
262265
else:
263266
# Add a segment to the URL

tests/test_unit.py

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -81,7 +81,7 @@ def test__init__(self):
8181
self.assertEqual(default_client.host, self.host)
8282
self.assertEqual(default_client.request_headers, {})
8383
self.assertIs(default_client.timeout, None)
84-
methods = set(('delete', 'get', 'patch', 'post', 'put'))
84+
methods = {'delete', 'get', 'patch', 'post', 'put'}
8585
self.assertEqual(default_client.methods, methods)
8686
self.assertIsNone(default_client._version)
8787
self.assertEqual(default_client._url_path, [])
@@ -94,7 +94,7 @@ def test__init__(self):
9494
timeout=10)
9595
self.assertEqual(client.host, self.host)
9696
self.assertEqual(client.request_headers, request_headers)
97-
methods = set(('delete', 'get', 'patch', 'post', 'put'))
97+
methods = {'delete', 'get', 'patch', 'post', 'put'}
9898
self.assertEqual(client.methods, methods)
9999
self.assertEqual(client._version, 3)
100100
self.assertEqual(client._url_path, [])
@@ -125,7 +125,7 @@ def test__urllib_headers(self, maker):
125125
self.client._update_headers({'X-test': 'Test'})
126126
self.client.get()
127127
request = maker.call_args[0][1]
128-
self.assertTrue('X-test' in request.headers)
128+
self.assertIn('X-test', request.headers)
129129

130130
@mock.patch('python_http_client.client.Client._make_request')
131131
def test__urllib_method(self, maker):

0 commit comments

Comments
 (0)