String Formatting and PEP-8 fixes

Author: vkmrishad

README.md

@@ -78,7 +78,7 @@ print response.body
 import python_http_client
 global_headers = {"Authorization": "Basic XXXXXXX"}
 client = Client(host='base_url', request_headers=global_headers)
-query_params={"hello":0, "world":1}
+query_params={"hello": 0, "world": 1}
 request_headers={"X-Test": "test"}
 data={"some": 1, "awesome": 2, "data": 3}
 response = client.your.api._(param).call.post(request_body=data,

python_http_client/client.py

@@ -128,7 +128,7 @@ def _build_url(self, query_params):
         if self._version:
             url = self._build_versioned_url(url)
         else:
-            url = self.host + url
+            url = '{}{}'.format(self.host, url)
         return url
 
     def _update_headers(self, request_headers):
@@ -147,7 +147,7 @@ def _build_client(self, name=None):
         :type name: string
         :return: A Client object
         """
-        url_path = self._url_path + [name] if name else self._url_path
+        url_path = '{}{}'.format(self._url_path, [name]) if name else self._url_path
         return Client(host=self.host,
                       version=self._version,
                       request_headers=self.request_headers,

tests/test_repofiles.py

@@ -31,4 +31,5 @@ def _all_file(self, files):
     def test_file_existence(self):
         missing = list(filter(self._all_file, self.FILES))
         self.assertEqual(len(missing), 0,
-                         "Files %s aren't found" % str(missing))
+                         "Files {} aren't found".format(missing)
+                         )

tests/test_unit.py

@@ -70,7 +70,7 @@ def setUp(self):
         self.api_key = "SENDGRID_API_KEY"
         self.request_headers = {
             'Content-Type': 'application/json',
-            'Authorization': 'Bearer ' + self.api_key
+            'Authorization': 'Bearer {}'.format(self.api_key)
         }
         self.client = Client(host=self.host,
                              request_headers=self.request_headers,