Merge pull request #853 from pnijhara/pnijhara-patch-1

Fix some code quality and bug-risk issues

Author: timothycrosley

.deepsource.toml

@@ -0,0 +1,15 @@
+version = 1
+
+test_patterns = ["tests/**"]
+
+exclude_patterns = [
+    "examples/**",
+    "benchmarks/**"
+]
+
+[[analyzers]]
+name = "python"
+enabled = true
+
+  [analyzers.meta]
+  runtime_version = "3.x.x"

hug/api.py

@@ -173,7 +173,7 @@ def add_exception_handler(self, exception_type, error_handler, versions=(None,))
 
         for version in versions:
             placement = self._exception_handlers.setdefault(version, OrderedDict())
-            placement[exception_type] = (error_handler,) + placement.get(exception_type, tuple())
+            placement[exception_type] = (error_handler,) + placement.get(exception_type, ())
 
     def extend(self, http_api, route="", base_url="", **kwargs):
         """Adds handlers from a different Hug API to this one - to create a single API"""

hug/interface.py

@@ -186,7 +186,7 @@ def __init__(self, route, function):
             self.parameters = tuple(route["parameters"])
             self.all_parameters = set(route["parameters"])
             self.required = tuple(
-                [parameter for parameter in self.parameters if parameter not in self.defaults]
+                parameter for parameter in self.parameters if parameter not in self.defaults
             )
 
         if "map_params" in route:
@@ -204,10 +204,8 @@ def __init__(self, route, function):
                     self.all_parameters.add(interface_name)
                 if internal_name in self.required:
                     self.required = tuple(
-                        [
-                            interface_name if param == internal_name else param
-                            for param in self.required
-                        ]
+                        interface_name if param == internal_name else param
+                        for param in self.required
                     )
 
             reverse_mapping = {