Add tests for merge to operations.

Author: rhysrevans3

stac_fastapi/extensions/tests/test_transaction.py

@@ -54,6 +54,9 @@ def patch_item(
         patch: Union[PartialItem, List[PatchOperation]],
         **kwargs,
     ):
+        if isinstance(patch, PartialItem):
+            patch = patch.operations()
+
         return {
             "path_collection_id": collection_id,
             "path_item_id": item_id,
@@ -78,6 +81,9 @@ def patch_collection(
         patch: Union[PartialCollection, List[PatchOperation]],
         **kwargs,
     ):
+        if isinstance(patch, PartialCollection):
+            patch = patch.operations()
+
         return {
             "path_collection_id": collection_id,
             "patch": patch,
@@ -126,15 +132,18 @@ def test_patch_operation_item(client: TestClient) -> None:
     ]
 
 
-def test_patch_merge_item(client: TestClient, item: Item) -> None:
+def test_patch_merge_item(client: TestClient) -> None:
     response = client.patch(
         "/collections/a-collection/items/an-item",
-        content=json.dumps(item),
+        content=json.dumps({"properties": {"hello": "world", "foo": None}}),
     )
     assert response.is_success, response.text
     assert response.json()["path_collection_id"] == "a-collection"
     assert response.json()["path_item_id"] == "an-item"
-    assert response.json()["patch"]["type"] == "Feature"
+    assert response.json()["patch"] == [
+        {"op": "add", "path": "/properties/hello", "value": "world"},
+        {"op": "remove", "path": "/properties/foo"},
+    ]
 
 
 def test_delete_item(client: TestClient) -> None:
@@ -169,14 +178,17 @@ def test_patch_operation_collection(client: TestClient) -> None:
     ]
 
 
-def test_patch_merge_collection(client: TestClient, collection: Collection) -> None:
+def test_patch_merge_collection(client: TestClient) -> None:
     response = client.patch(
         "/collections/a-collection",
-        content=json.dumps(collection),
+        content=json.dumps({"summaries": {"hello": "world", "foo": None}}),
     )
     assert response.is_success, response.text
     assert response.json()["path_collection_id"] == "a-collection"
-    assert response.json()["patch"]["type"] == "Collection"
+    assert response.json()["patch"] == [
+        {"op": "add", "path": "/summaries/hello", "value": "world"},
+        {"op": "remove", "path": "/summaries/foo"},
+    ]
 
 
 def test_delete_collection(client: TestClient, collection: Collection) -> None:

stac_fastapi/types/stac_fastapi/types/stac.py

@@ -172,17 +172,19 @@ def merge_to_operations(data: Dict) -> List[PatchOperation]:
 
         for key, value in data.copy().items():
             if value is None:
-                operations.append(PatchRemove(op="remove", path=key))
+                operations.append(PatchRemove(op="remove", path=f"/{key}"))
 
             elif isinstance(value, dict):
                 nested_operations = BasePartial.merge_to_operations(value)
 
                 for nested_operation in nested_operations:
-                    nested_operation.path = f"{key}/{nested_operation.path}"
+                    nested_operation.path = f"/{key}{nested_operation.path}"
                     operations.append(nested_operation)
 
             else:
-                operations.append(PatchAddReplaceTest(op="add", path=key, value=value))
+                operations.append(
+                    PatchAddReplaceTest(op="add", path=f"/{key}", value=value)
+                )
 
         return operations