update black

Pipfile

@@ -19,7 +19,7 @@ lovely-pytest-docker = "*"
 freezegun = "*"
 ipython = "*"
 responses = "*"
-black = "==19.10b0"
+black = "==20.8b1"
 respx = "*"
 pylint = "*"
 

idunn/api/instant_answer.py

@@ -38,7 +38,9 @@ class InstantAnswerResult(BaseModel):
         ),
         example=(2.32, 48.85, 2.367, 48.866),
     )
-    maps_url: HttpUrl = Field(description="Direct URL to the result(s) on Qwant Maps.",)
+    maps_url: HttpUrl = Field(
+        description="Direct URL to the result(s) on Qwant Maps.",
+    )
     maps_frame_url: HttpUrl = Field(
         description="URL to the map displaying the results on Qwant Maps, with no user interface. "
         "This URL can be used to display an `<iframe>`."

idunn/api/places_list.py

@@ -193,7 +193,10 @@ async def get_places_bbox(
     )
 
     return PlacesBboxResponse(
-        places=result_places, source=params.source, bbox=results_bbox, bbox_extended=bbox_extended,
+        places=result_places,
+        source=params.source,
+        bbox=results_bbox,
+        bbox_extended=bbox_extended,
     )
 
 
@@ -220,7 +223,10 @@ async def _fetch_places_list(params: PlacesQueryParam):
         raw_filters = [f for c in params.category for f in c.raw_filters()]
 
     bbox_places = await run_in_threadpool(
-        fetch_es_pois, raw_filters=raw_filters, bbox=params.bbox, max_size=params.size,
+        fetch_es_pois,
+        raw_filters=raw_filters,
+        bbox=params.bbox,
+        max_size=params.size,
     )
     return [POI(p["_source"]) for p in bbox_places]
 

idunn/api/status.py

@@ -7,8 +7,7 @@
 
 
 def get_status():
-    """Returns the status of the elastic cluster
-    """
+    """Returns the status of the elastic cluster"""
     es = get_elasticsearch()
     try:
         cluster_health = es.cluster.health()

idunn/blocks/wikipedia.py

@@ -48,7 +48,9 @@ def wrapped_f(*args, **kwargs):
                 except pybreaker.CircuitBreakerError:
                     prometheus.exception("CircuitBreakerError")
                     logger.error(
-                        "Got CircuitBreakerError in %s", f.__name__, exc_info=True,
+                        "Got CircuitBreakerError in %s",
+                        f.__name__,
+                        exc_info=True,
                     )
                 except HTTPError:
                     prometheus.exception("HTTPError")

idunn/datasources/pages_jaunes.py

@@ -140,7 +140,8 @@ def get_places_from_url(self, url, params=None, size=10):
             and res.context.pages.next_page_url
         ):
             pois += self.get_places_from_url(
-                res.context.pages.next_page_url, size=size - len(pois),
+                res.context.pages.next_page_url,
+                size=size - len(pois),
             )
 
         return pois

idunn/directions/client.py

@@ -174,7 +174,8 @@ def directions_combigo(self, start, end, mode, lang):
     def get_directions(self, from_place, to_place, mode, lang, params: QueryParams):
         if not DirectionsClient.is_in_allowed_zone(mode, from_place, to_place):
             raise HTTPException(
-                status_code=422, detail="requested path is not inside an allowed area",
+                status_code=422,
+                detail="requested path is not inside an allowed area",
             )
 
         method = self.directions_qwant

idunn/geocoder/nlu_client.py

@@ -74,7 +74,7 @@ async def nlu_classifier(self, text):
 
     @staticmethod
     def regex_classifier(text, is_brand=False):
-        """ Match text with a category, using 'regex'
+        """Match text with a category, using 'regex'
         >>> NLU_Helper.regex_classifier("restau").value
         'restaurant'
         >>> NLU_Helper.regex_classifier("pub").value
@@ -98,7 +98,7 @@ async def classify_category(self, text, is_brand=False):
 
     @classmethod
     def fuzzy_match(cls, query, bragi_res):
-        """ Does the response match the query reasonably well ?
+        """Does the response match the query reasonably well ?
         >>> NLU_Helper.fuzzy_match("bastille", {"name": "Beuzeville-la-Bastille"})
         False
         >>> NLU_Helper.fuzzy_match("paris 20", {"name": "Paris 20e Arrondissement"})

idunn/utils/logging.py

@@ -23,7 +23,11 @@ def get_logging_dict(settings):
         },
         "loggers": {
             "uvicorn": {"level": "INFO", "handlers": ["console"], "propagate": False},
-            "uvicorn.access": {"level": "WARNING", "handlers": ["console"], "propagate": False,},
+            "uvicorn.access": {
+                "level": "WARNING",
+                "handlers": ["console"],
+                "propagate": False,
+            },
             "gunicorn": {"level": "INFO", "handlers": ["console"], "propagate": False},
             # "gunicorn.error" logger needs to be defined explicitly,
             # as it will be used by UvicornWorker to configure its own error logger

idunn/utils/prometheus.py

@@ -52,15 +52,19 @@ def expose_metrics_multiprocess():
 
 
 REQUEST_DURATION = Histogram(
-    "http_request_duration_seconds", "Time spent processing a request.", ["method", "handler"],
+    "http_request_duration_seconds",
+    "Time spent processing a request.",
+    ["method", "handler"],
 )
 REQUEST_COUNT = Counter(
     "http_requests_total",
     "Request count by method, handler and response code.",
     ["method", "handler", "code"],
 )
 REQUESTS_INPROGRESS = Gauge(
-    "http_requests_inprogress", "Requests in progress by method and handler", ["method", "handler"],
+    "http_requests_inprogress",
+    "Requests in progress by method and handler",
+    ["method", "handler"],
 )
 
 

idunn/utils/result_filter.py

@@ -157,7 +157,9 @@ def coverage(terms):
 
     if not check_coverage:
         logger.info(
-            "Removed `%s` from results because query `%s` is not accurate enough", name, query,
+            "Removed `%s` from results because query `%s` is not accurate enough",
+            name,
+            query,
         )
         return False
 

tests/test_accessibility.py

@@ -13,7 +13,16 @@ def test_accessibility_block():
 
 
 def test_accessibility_unknown():
-    web_block = AccessibilityBlock.from_es(POI({"properties": {"wheelchair": "toto",}}), lang="en")
+    web_block = AccessibilityBlock.from_es(
+        POI(
+            {
+                "properties": {
+                    "wheelchair": "toto",
+                }
+            }
+        ),
+        lang="en",
+    )
     assert web_block is None