fixed issue with delimiter and added docstring

cloutierMat · cloutierMat · commit 478d6a2d0c33 · 2025-10-16T09:48:24.000-06:00
diff --git a/localstack-core/localstack/utils/json.py b/localstack-core/localstack/utils/json.py
@@ -169,15 +169,24 @@ def extract_jsonpath(value, path):
     return result
 
 
-def assign_to_path(target, path: str, value, delimiter: str = "."):
+def assign_to_path(target: dict, path: str, value: any, delimiter: str = ".") -> dict:
+    """Assign the given value to a dict. If the path doesn't exist in the target dict, it will be created.
+    The delimiter can be used to provide a path with a different delimiter.
+
+    Examples:
+     - assign_to_path({}, "a", "b") => {"a": "b"}
+     - assign_to_path({}, "a.b.c", "d") => {"a": {"b": {"c": "d"}}}
+     - assign_to_path({}, "a.b/c", "d", delimiter="/") => {"a.b": {"c": "d"}}
+
+    """
     parts = path.strip(delimiter).split(delimiter)
 
     if len(parts) == 1:
         target[parts[0]] = value
         return target
 
     path_to_parent = delimiter.join(parts[:-1])
-    parent = extract_from_jsonpointer_path(target, path_to_parent, auto_create=True)
+    parent = extract_from_jsonpointer_path(target, path_to_parent, delimiter, auto_create=True)
     if not isinstance(parent, dict):
         LOG.debug(
             'Unable to find parent (type %s) for path "%s" in object: %s',
diff --git a/tests/unit/utils/test_json.py b/tests/unit/utils/test_json.py
@@ -11,5 +11,17 @@ def test_json_encoder():
 
 def test_assign_to_path_single_path():
     target = {}
-    assign_to_path(target, "foo", "bar")
-    assert target == {"foo": "bar"}
+    assign_to_path(target, "a", "bar")
+    assert target == {"a": "bar"}
+
+
+def test_assign_multi_nested_path():
+    target = {}
+    assign_to_path(target, "a.b.foo", "bar")
+    assert target == {"a": {"b": {"foo": "bar"}}}
+
+
+def test_assign_to_path_mixed_delimiters():
+    target = {}
+    assign_to_path(target, "a.b/c", "d", delimiter="/")
+    assert target == {"a.b": {"c": "d"}}
