fix: return 404 for missing collection on /items

sqlalchemy

Author: gadomski

stac_fastapi/sqlalchemy/stac_fastapi/sqlalchemy/core.py

@@ -110,6 +110,12 @@ def item_collection(
         """Read an item collection from the database."""
         base_url = str(kwargs["request"].base_url)
         with self.session.reader.context_session() as session:
+            # Look up the collection first to get a 404 if it doesn't exist
+            collection = self._lookup_id(collection_id, self.collection_table, session)
+            if collection is None:
+                return NotFoundError(
+                    f"collection with id={collection_id} does not exist"
+                )
             query = (
                 session.query(self.item_table)
                 .join(self.collection_table)

stac_fastapi/sqlalchemy/tests/resources/test_item.py

@@ -688,7 +688,7 @@ def test_item_search_get_query_extension(app_client, load_test_data):
 def test_get_missing_item_collection(app_client):
     """Test reading a collection which does not exist"""
     resp = app_client.get("/collections/invalid-collection/items")
-    assert resp.status_code == 200
+    assert resp.status_code == 404
 
 
 def test_pagination_item_collection(app_client, load_test_data):