Reset command set between different patches.

Author: rhysrevans3

stac_fastapi/core/stac_fastapi/core/models/patch.py

@@ -17,6 +17,10 @@ class ESCommandSet:
 
     dict_: Dict[str, None] = {}
 
+    def __init__(self):
+        """Initialise ESCommandSet instance."""
+        self.dict_ = {}
+
     def add(self, value: str):
         """Add command.
 

stac_fastapi/core/stac_fastapi/core/utilities.py

@@ -278,29 +278,6 @@ def test_commands(
     )
 
 
-def commands_to_source(commands: List[str]) -> str:
-    """Convert list of commands to Elasticsearch script source.
-
-    Args:
-        commands (List[str]): List of Elasticearch commands
-
-    Returns:
-        str: Elasticsearch script source
-    """
-    seen: Set[str] = set()
-    seen_add = seen.add
-    # regex = re.compile(r"([^.' ]*:[^.' ]*)[. ;]")
-    source = ""
-
-    # filter duplicate lines
-    for command in commands:
-        if command not in seen:
-            seen_add(command)
-            source += command
-
-    return source
-
-
 def operations_to_script(operations: List) -> Dict:
     """Convert list of operation to painless script.
 

stac_fastapi/tests/clients/test_elasticsearch.py

@@ -320,7 +320,7 @@ async def test_json_patch_item_replace(ctx, core_client, txn_client):
             {"op": "replace", "path": "/properties/gsd", "value": 100}
         ),
         PatchAddReplaceTest.model_validate(
-            {"op": "replace", "path": "/properties/proj:epsg", "value": "world"}
+            {"op": "replace", "path": "/properties/proj:epsg", "value": 12345}
         ),
         PatchAddReplaceTest.model_validate(
             {"op": "replace", "path": "/properties/area/1", "value": 50}
@@ -339,7 +339,7 @@ async def test_json_patch_item_replace(ctx, core_client, txn_client):
     )
 
     assert updated_item["properties"]["gsd"] == 100
-    assert updated_item["properties"]["proj:epsg"] == "world"
+    assert updated_item["properties"]["proj:epsg"] == 12345
     assert updated_item["properties"]["area"] == [2500, 50]