clean up test

Author: jonhealy1

CHANGELOG.md

@@ -9,6 +9,7 @@ and this project adheres to [Semantic Versioning](http://semver.org/spec/v2.0.0.
 
 ### Added
 
+- GET `/collections` collection search free text extension ex. `/collections?q=sentinel`. [#466](https://github.com/stac-utils/stac-fastapi-elasticsearch-opensearch/pull/466)
 - Added `USE_DATETIME` environment variable to configure datetime search behavior in SFEOS. [#452](https://github.com/stac-utils/stac-fastapi-elasticsearch-opensearch/pull/452)
 - GET `/collections` collection search sort extension ex. `/collections?sortby=+id`. [#456](https://github.com/stac-utils/stac-fastapi-elasticsearch-opensearch/pull/456)
 - GET `/collections` collection search fields extension ex. `/collections?fields=id,title`. [#465](https://github.com/stac-utils/stac-fastapi-elasticsearch-opensearch/pull/465)

README.md

@@ -126,8 +126,15 @@ SFEOS implements extended capabilities for the `/collections` endpoint, allowing
   - Example: `/collections?fields=id,title,description`
   - This helps reduce payload size when only certain fields are needed
 
+- **Free Text Search**: Search across collection text fields using the `q` parameter
+  - Example: `/collections?q=landsat`
+  - Searches across multiple text fields including title, description, and keywords
+  - Supports partial word matching and relevance-based sorting
+
 These extensions make it easier to build user interfaces that display and navigate through collections efficiently.
 
+> **Configuration**: Collection search extensions (sorting, field selection, and free text search) can be disabled by setting the `ENABLE_COLLECTIONS_SEARCH` environment variable to `false`. By default, these extensions are enabled.
+
 > **Note**: Sorting is only available on fields that are indexed for sorting in Elasticsearch/OpenSearch. With the default mappings, you can sort on:
 > - `id` (keyword field)
 > - `extent.temporal.interval` (date field)

stac_fastapi/tests/api/test_api_search_collections.py

@@ -226,90 +226,3 @@ async def test_collections_free_text_search_get(app_client, txn_client, load_tes
     # Should only find the landsat collection
     assert len(found_collections) == 1
     assert found_collections[0]["id"] == f"{test_prefix}-modis"
-
-
-# @pytest.mark.asyncio
-# async def test_collections_free_text_search_post(app_client, txn_client, load_test_data):
-#     """Verify POST /collections-search honors the q parameter for free text search."""
-#     # Create multiple collections with different content
-#     base_collection = load_test_data("test_collection.json")
-
-#     # Use unique prefixes to avoid conflicts between tests
-#     test_prefix = f"q-post-{uuid.uuid4().hex[:8]}"
-
-#     # Create collections with different content to test free text search
-#     test_collections = [
-#         {
-#             "id": f"{test_prefix}-sentinel",
-#             "title": "Sentinel-2 Collection",
-#             "description": "Collection of Sentinel-2 data",
-#             "summaries": {"platform": ["sentinel-2a", "sentinel-2b"]}
-#         },
-#         {
-#             "id": f"{test_prefix}-landsat",
-#             "title": "Landsat Collection",
-#             "description": "Collection of Landsat data",
-#             "summaries": {"platform": ["landsat-8", "landsat-9"]}
-#         },
-#         {
-#             "id": f"{test_prefix}-modis",
-#             "title": "MODIS Collection",
-#             "description": "Collection of MODIS data",
-#             "summaries": {"platform": ["terra", "aqua"]}
-#         }
-#     ]
-
-#     for i, coll in enumerate(test_collections):
-#         test_collection = base_collection.copy()
-#         test_collection["id"] = coll["id"]
-#         test_collection["title"] = coll["title"]
-#         test_collection["description"] = coll["description"]
-#         test_collection["summaries"] = coll["summaries"]
-#         await create_collection(txn_client, test_collection)
-
-#     # Test free text search for "sentinel" via POST
-#     resp = await app_client.post(
-#         "/collections-search",
-#         json={"q": ["sentinel"]},
-#     )
-#     assert resp.status_code == 200
-#     resp_json = resp.json()
-
-#     # Filter collections to only include the ones we created for this test
-#     found_collections = [c for c in resp_json["collections"] if c["id"].startswith(test_prefix)]
-
-#     # Should only find the sentinel collection
-#     assert len(found_collections) == 1
-#     assert found_collections[0]["id"] == f"{test_prefix}-sentinel"
-
-#     # Test free text search for "landsat" via POST
-#     resp = await app_client.post(
-#         "/collections-search",
-#         json={"q": ["landsat"]},
-#     )
-#     assert resp.status_code == 200
-#     resp_json = resp.json()
-
-#     # Filter collections to only include the ones we created for this test
-#     found_collections = [c for c in resp_json["collections"] if c["id"].startswith(test_prefix)]
-
-#     # Should only find the landsat collection
-#     assert len(found_collections) == 1
-#     assert found_collections[0]["id"] == f"{test_prefix}-landsat"
-
-#     # Test free text search with multiple terms
-#     resp = await app_client.post(
-#         "/collections-search",
-#         json={"q": ["sentinel", "modis"]},
-#     )
-#     assert resp.status_code == 200
-#     resp_json = resp.json()
-
-#     # Filter collections to only include the ones we created for this test
-#     found_collections = [c for c in resp_json["collections"] if c["id"].startswith(test_prefix)]
-#     found_ids = [c["id"] for c in found_collections]
-
-#     # Should find both sentinel and modis collections
-#     assert len(found_collections) == 2
-#     assert f"{test_prefix}-sentinel" in found_ids
-#     assert f"{test_prefix}-modis" in found_ids