Merge pull request #94 from stac-utils/fix_intersection

Fix search request intersection, add test

Author: jonhealy1

CHANGELOG.md

@@ -15,6 +15,8 @@ and this project adheres to [Semantic Versioning](http://semver.org/spec/v2.0.0.
 
 ### Fixed
 
+- Fixed search intersects query
+
 ### Changed
 
 - Elasticsearch index mappings updated to be more thorough.

stac_fastapi/elasticsearch/stac_fastapi/elasticsearch/core.py

@@ -244,7 +244,7 @@ async def post_search(
             search = self.database.apply_bbox_filter(search=search, bbox=bbox)
 
         if search_request.intersects:
-            self.database.apply_intersects_filter(
+            search = self.database.apply_intersects_filter(
                 search=search, intersects=search_request.intersects
             )
 

stac_fastapi/elasticsearch/tests/api/test_api.py

@@ -201,6 +201,21 @@ async def test_search_point_intersects(app_client, ctx):
     assert len(resp_json["features"]) == 1
 
 
+async def test_search_point_does_not_intersect(app_client, ctx):
+    point = [15.04, -3.14]
+    intersects = {"type": "Point", "coordinates": point}
+
+    params = {
+        "intersects": intersects,
+        "collections": [ctx.item["collection"]],
+    }
+    resp = await app_client.post("/search", json=params)
+
+    assert resp.status_code == 200
+    resp_json = resp.json()
+    assert len(resp_json["features"]) == 0
+
+
 async def test_datetime_non_interval(app_client, ctx):
     dt_formats = [
         "2020-02-12T12:30:22+00:00",