linting fixes

Author: jonhealy1

stac_fastapi/core/stac_fastapi/core/core.py

@@ -906,7 +906,7 @@ def bulk_item_insert(
         return f"Successfully added {len(processed_items)} Items."
 
 
-_DEFAULT_QUERYABLES = {
+_DEFAULT_QUERYABLES: Dict[str, Dict[str, Any]] = {
     "id": {
         "description": "ID",
         "$ref": "https://schemas.stacspec.org/v1.0.0/item-spec/json-schema/item.json#/definitions/core/allOf/2/properties/id",
@@ -1013,7 +1013,7 @@ async def get_queryables(
         if not collection_id:
             return queryables
 
-        properties = {}
+        properties: Dict[str, Any] = queryables["properties"]
         queryables.update(
             {
                 "properties": properties,

stac_fastapi/core/stac_fastapi/core/database_logic.py

@@ -1,3 +1,5 @@
+"""Database logic core."""
+
 import os
 from functools import lru_cache
 from typing import Any, Dict, List, Optional, Protocol

stac_fastapi/core/stac_fastapi/core/extensions/query.py

@@ -8,7 +8,7 @@
 from dataclasses import dataclass
 from enum import auto
 from types import DynamicClassAttribute
-from typing import Any, Callable, Dict, Optional, Union
+from typing import Any, Callable, Dict, Optional
 
 from pydantic import BaseModel, root_validator
 from stac_pydantic.utils import AutoValueEnum

stac_fastapi/elasticsearch/stac_fastapi/elasticsearch/database_logic.py

@@ -273,7 +273,7 @@ async def get_one_item(self, collection_id: str, item_id: str) -> Dict:
             )
         except exceptions.NotFoundError:
             raise NotFoundError(
-                f"Item {item_id} does not exist in Collection {collection_id}"
+                f"Item {item_id} does not exist inside Collection {collection_id}"
             )
         return item["_source"]
 
@@ -826,7 +826,7 @@ async def update_collection(
                     "source": {"index": f"{ITEMS_INDEX_PREFIX}{collection_id}"},
                     "script": {
                         "lang": "painless",
-                        "source": f"""ctx._id = ctx._id.replace('{collection_id}', '{collection["id"]}'); ctx._source.collection = '{collection["id"]}' ;""",
+                        "source": f"""ctx._id = ctx._id.replace('{collection_id}', '{collection["id"]}'); ctx._source.collection = '{collection["id"]}' ;""",  # noqa: E702
                     },
                 },
                 wait_for_completion=True,

stac_fastapi/opensearch/stac_fastapi/opensearch/database_logic.py

@@ -302,7 +302,7 @@ async def get_one_item(self, collection_id: str, item_id: str) -> Dict:
             )
         except exceptions.NotFoundError:
             raise NotFoundError(
-                f"Item {item_id} does not exist in Collection {collection_id}"
+                f"Item {item_id} does not exist inside Collection {collection_id}"
             )
         return item["_source"]
 
@@ -858,7 +858,7 @@ async def update_collection(
                     "source": {"index": f"{ITEMS_INDEX_PREFIX}{collection_id}"},
                     "script": {
                         "lang": "painless",
-                        "source": f"""ctx._id = ctx._id.replace('{collection_id}', '{collection["id"]}'); ctx._source.collection = '{collection["id"]}' ;""",
+                        "source": f"""ctx._id = ctx._id.replace('{collection_id}', '{collection["id"]}'); ctx._source.collection = '{collection["id"]}' ;""",  # noqa: E702
                     },
                 },
                 wait_for_completion=True,

stac_fastapi/tests/rate_limit/test_rate_limit.py

@@ -18,7 +18,7 @@ async def test_rate_limit(app_client_rate_limit: AsyncClient, ctx):
         except RateLimitExceeded:
             status_code = 429
 
-        logger.info(f"Request {i+1}: Status code {status_code}")
+        logger.info(f"Request {i + 1}: Status code {status_code}")
         assert (
             status_code == expected_status_code
         ), f"Expected status code {expected_status_code}, but got {status_code}"
@@ -32,7 +32,7 @@ async def test_rate_limit_no_limit(app_client: AsyncClient, ctx):
         response = await app_client.get("/collections")
         status_code = response.status_code
 
-        logger.info(f"Request {i+1}: Status code {status_code}")
+        logger.info(f"Request {i + 1}: Status code {status_code}")
         assert (
             status_code == expected_status_code
         ), f"Expected status code {expected_status_code}, but got {status_code}"