Format fix

Author: tcalmant

docs/conf.py

@@ -64,7 +64,13 @@
 # List of patterns, relative to source directory, that match files and
 # directories to ignore when looking for source files.
 # This pattern also affects html_static_path and html_extra_path.
-exclude_patterns = ["_build", "Thumbs.db", ".DS_Store", ".gitignore", "requirements.txt"]
+exclude_patterns = [
+    "_build",
+    "Thumbs.db",
+    ".DS_Store",
+    ".gitignore",
+    "requirements.txt",
+]
 
 # The name of the Pygments (syntax highlighting) style to use.
 pygments_style = None

jsonrpclib/jsonclass.py

@@ -307,8 +307,9 @@ def load(obj, classes=None):
             )
     else:
         raise TranslationError(
-            "Constructor args must be a dict or a list, "
-            "not {0}".format(type(params).__name__)
+            "Constructor args must be a dict or a list, not {0}".format(
+                type(params).__name__
+            )
         )
 
     # Remove the class information, as it must be ignored during the

jsonrpclib/jsonrpc.py

@@ -236,7 +236,9 @@ def close(self):
 
 
 class TransportMixIn(object):
-    """ Just extends the XML-RPC transport where necessary. """
+    """
+    Just extends the XML-RPC transport where necessary.
+    """
 
     # for Python 2.7 support
     _connection = None
@@ -305,7 +307,7 @@ def emit_additional_headers(self, connection):
             # Not available this version of Python (should not happen)
             pass
         else:
-            for (key, value) in extra_headers:
+            for key, value in extra_headers:
                 additional_headers[key] = value
 
         # Prepare the merged dictionary
@@ -1235,9 +1237,7 @@ def dump(
         If a method, and params are not in a listish or a Fault,
         error out.
         """
-        raise TypeError(
-            "Params must be a dict, list, tuple " "or Fault instance."
-        )
+        raise TypeError("Params must be a dict, list, tuple or Fault instance.")
 
     # Prepares the JSON-RPC content
     payload = Payload(rpcid=rpcid, version=version)
@@ -1250,8 +1250,7 @@ def dump(
     if not isinstance(methodname, utils.STRING_TYPES) and not is_response:
         # Neither a request nor a response
         raise ValueError(
-            "Method name must be a string, or is_response "
-            "must be set to True."
+            "Method name must be a string, or is_response must be set to True."
         )
 
     if config.use_jsonclass:

tests/test_compatibility.py

@@ -59,7 +59,7 @@ def tearDown(self):
 
     # Version 2.0 Tests
     def test_positional(self):
-        """ Positional arguments in a single call """
+        """Positional arguments in a single call"""
         result = self.client.subtract(23, 42)
         self.assertTrue(result == -19)
         result = self.client.subtract(42, 23)
@@ -77,7 +77,7 @@ def test_positional(self):
         self.assertTrue(response == verify_response)
 
     def test_named(self):
-        """ Named arguments in a single call """
+        """Named arguments in a single call"""
         result = self.client.subtract(subtrahend=23, minuend=42)
         self.assertTrue(result == 19)
         result = self.client.subtract(minuend=42, subtrahend=23)
@@ -95,7 +95,7 @@ def test_named(self):
         self.assertTrue(response == verify_response)
 
     def test_notification(self):
-        """ Testing a notification (response should be null) """
+        """Testing a notification (response should be null)"""
         result = self.client._notify.update(1, 2, 3, 4, 5)
         self.assertTrue(result is None)
         request = json.loads(self.history.request)
@@ -110,7 +110,7 @@ def test_notification(self):
         self.assertTrue(response == verify_response)
 
     def test_non_existent_method(self):
-        """ Testing behaviour when calling a non-existent method """
+        """Testing behaviour when calling a non-existent method"""
         self.assertRaises(jsonrpclib.ProtocolError, self.client.foobar)
         request = json.loads(self.history.request)
         response = json.loads(self.history.response)
@@ -128,14 +128,14 @@ def test_non_existent_method(self):
         self.assertTrue(response == verify_response)
 
     def test_special_method(self):
-        """ Tests behaviour on dunder methods """
+        """Tests behaviour on dunder methods"""
         self.assertRaises(
             AttributeError, getattr, self.client, "__special_method__"
         )
         self.assertIsNone(self.history.request)
 
     def test_invalid_json(self):
-        """ Tests behaviour on invalid JSON request """
+        """Tests behaviour on invalid JSON request"""
         invalid_json = (
             '{"jsonrpc": "2.0", "method": "foobar, ' + '"params": "bar", "baz]'
         )
@@ -149,7 +149,7 @@ def test_invalid_json(self):
         self.assertTrue(response == verify_response)
 
     def test_invalid_request(self):
-        """ Tests incomplete request """
+        """Tests incomplete request"""
         invalid_request = '{"jsonrpc": "2.0", "method": 1, "params": "bar"}'
         self.client._run_request(invalid_request)
         response = json.loads(self.history.response)
@@ -161,7 +161,7 @@ def test_invalid_request(self):
         self.assertTrue(response == verify_response)
 
     def test_batch_invalid_json(self):
-        """ Tests invalid JSON request on batch call """
+        """Tests invalid JSON request on batch call"""
         invalid_request = (
             '[ {"jsonrpc": "2.0", "method": "sum", '
             + '"params": [1,2,4], "id": "1"},{"jsonrpc": "2.0", "method" ]'
@@ -176,7 +176,7 @@ def test_batch_invalid_json(self):
         self.assertTrue(response == verify_response)
 
     def test_empty_array(self):
-        """ Tests empty array as request """
+        """Tests empty array as request"""
         invalid_request = "[]"
         self.client._run_request(invalid_request)
         response = json.loads(self.history.response)
@@ -188,7 +188,7 @@ def test_empty_array(self):
         self.assertTrue(response == verify_response)
 
     def test_nonempty_array(self):
-        """ Tests array as request """
+        """Tests array as request"""
         invalid_request = "[1,2]"
         request_obj = json.loads(invalid_request)
         self.client._run_request(invalid_request)
@@ -203,7 +203,7 @@ def test_nonempty_array(self):
             self.assertTrue(resp == verify_resp)
 
     def test_batch(self):
-        """ Tests batch call """
+        """Tests batch call"""
         multicall = jsonrpclib.MultiCall(self.client)
         multicall.sum(1, 2, 4)
         multicall._notify.notify_hello(7)
@@ -272,7 +272,7 @@ def test_batch(self):
             self.assertTrue(response == verify_response)
 
     def test_batch_notifications(self):
-        """ Tests batch notifications """
+        """Tests batch notifications"""
         multicall = jsonrpclib.MultiCall(self.client)
         multicall._notify.notify_sum(1, 2, 4)
         multicall._notify.notify_hello(7)
@@ -290,7 +290,8 @@ def test_batch_notifications(self):
         self.assertTrue(self.history.response == "")
 
     def test_url_query_string(self):
-        """ Tests if the query string arguments are kept """
+        """Tests if the query string arguments are kept"""
+
         # Prepare a simple server
         class ReqHandler(BaseHTTPRequestHandler):
             """

tests/test_server.py

@@ -22,14 +22,14 @@
 
 
 def add(a, b):
-    """ Basic addition """
+    """Basic addition"""
     return a + b
 
 
 def sleep(t):
     start = time.time()
     while time.time() - start < t:
-        time.sleep(.1)
+        time.sleep(0.1)
 
 
 class PooledServerTests(unittest.TestCase):
@@ -71,8 +71,12 @@ def test_default_pool(self, pool=None, max_time=3):
                 self.assertEqual(result, rand1 + rand2)
 
             # Check pauses (using different clients)
-            threads = [threading.Thread(target=ServerProxy(
-                target_url).sleep, args=(1,)) for _ in range(5)]
+            threads = [
+                threading.Thread(
+                    target=ServerProxy(target_url).sleep, args=(1,)
+                )
+                for _ in range(5)
+            ]
             start_time = time.time()
             for thread in threads:
                 thread.start()

tests/test_threadpool.py

@@ -385,6 +385,7 @@ def testException(self):
         """
         Tests if an exception is correctly hidden
         """
+
         # Define the exception
         def thrower(ex):
             raise ex