Merge pull request #113 from maxmind/greg/run-black

Run new black version against repo

Author: shadromani

minfraud/webservice.py

@@ -132,18 +132,18 @@ def _exception_for_4xx_status(
                 uri,
                 raw_body,
             )
-        else:
-            if "code" in decoded_body and "error" in decoded_body:
-                return self._exception_for_web_service_error(
-                    decoded_body.get("error"), decoded_body.get("code"), status, uri
-                )
-            return HTTPError(
-                "Error response contains JSON but it does not "
-                + f"specify code or error keys: {raw_body}",
-                status,
-                uri,
-                raw_body,
+
+        if "code" in decoded_body and "error" in decoded_body:
+            return self._exception_for_web_service_error(
+                decoded_body.get("error"), decoded_body.get("code"), status, uri
             )
+        return HTTPError(
+            "Error response contains JSON but it does not "
+            + f"specify code or error keys: {raw_body}",
+            status,
+            uri,
+            raw_body,
+        )
 
     @staticmethod
     def _exception_for_web_service_error(

pyproject.toml

@@ -0,0 +1,6 @@
+# We should probably migrate most of setup.cfg here
+
+[tool.black]
+# src is showing up in our GitHub linting builds. It seems to
+# contain deps.
+extend-exclude = '^/src/'

tests/test_models.py

@@ -218,7 +218,6 @@ def test_score_ip_address(self):
         self.assertEqual(99, address.risk)
 
     def test_ip_address_locales(self):
-
         loc = IPAddress(
             {
                 "_locales": ["fr"],

tests/test_webservice.py

@@ -217,7 +217,6 @@ def test_200_on_request_with_nones(self):
 
     @httprettified
     def test_200_with_email_hashing(self):
-
         uri = "/".join([self.base_uri, self.type])
 
         httpretty.register_uri(