Pulls HTTP methods to class attr

Author: 0bsearch

python_http_client/client.py

@@ -63,6 +63,9 @@ def to_dict(self):
 class Client(object):
     """Quickly and easily access any REST or REST-like API."""
 
+    # These are the supported HTTP verbs
+    methods = {'delete', 'get', 'patch', 'post', 'put'}
+
     def __init__(self,
                  host,
                  request_headers=None,
@@ -89,8 +92,6 @@ def __init__(self,
         self._version = version
         # _url_path keeps track of the dynamically built url
         self._url_path = url_path or []
-        # These are the supported HTTP verbs
-        self.methods = ['delete', 'get', 'patch', 'post', 'put']
         # APPEND SLASH set
         self.append_slash = append_slash
         self.timeout = timeout

tests/test_unit.py

@@ -2,11 +2,13 @@
 import unittest
 
 from python_http_client.client import Client
-from python_http_client.exceptions import (BadRequestsError, HTTPError,
-                                           NotFoundError,
-                                           ServiceUnavailableError,
-                                           UnsupportedMediaTypeError,
-                                           handle_error)
+from python_http_client.exceptions import (
+    BadRequestsError, HTTPError,
+    NotFoundError,
+    ServiceUnavailableError,
+    UnsupportedMediaTypeError,
+    handle_error
+)
 
 try:
     # Python 3
@@ -80,7 +82,7 @@ def test__init__(self):
         self.assertEqual(default_client.host, self.host)
         self.assertEqual(default_client.request_headers, {})
         self.assertIs(default_client.timeout, None)
-        methods = ['delete', 'get', 'patch', 'post', 'put']
+        methods = {'delete', 'get', 'patch', 'post', 'put'}
         self.assertEqual(default_client.methods, methods)
         self.assertIsNone(default_client._version)
         self.assertEqual(default_client._url_path, [])
@@ -93,7 +95,7 @@ def test__init__(self):
                         timeout=10)
         self.assertEqual(client.host, self.host)
         self.assertEqual(client.request_headers, request_headers)
-        methods = ['delete', 'get', 'patch', 'post', 'put']
+        methods = {'delete', 'get', 'patch', 'post', 'put'}
         self.assertEqual(client.methods, methods)
         self.assertEqual(client._version, 3)
         self.assertEqual(client._url_path, [])
@@ -160,7 +162,7 @@ def test__getattr__(self):
         self.assertEqual(client._version, 3)
 
         # Test GET
-        mock_client._url_path + ['test']
+        mock_client._url_path += ['test']
         r = mock_client.get()
         self.assertEqual(r.status_code, 200)