fix: value->variant conversion for empty dict (#2739)

* fix: value->variant conversion for empty dict

* fix: test

* fix: test

* fix: add test

src/ansys/fluent/core/services/datamodel_se.py

@@ -295,6 +295,7 @@ def _convert_value_to_variant(val: _TValue, var: Variant) -> None:
             item_var = var.variant_vector_state.item.add()
             _convert_value_to_variant(item, item_var)
     elif isinstance(val, dict):
+        var.variant_map_state.SetInParent()
         for k, v in val.items():
             _convert_value_to_variant(v, var.variant_map_state.item[k])
 

tests/test_data_model_cache.py

@@ -49,7 +49,8 @@ def test_data_model_cache():
         ({"r1": {}}, "r1", {"A": [3.0, 6.0]}, [], {"r1": {"A": [3.0, 6.0]}}),
         ({"r1": {}}, "r1", {"A": ["ab", "cd"]}, [], {"r1": {"A": ["ab", "cd"]}}),
         ({"r1": {"A": {}}}, "r1", {"A": {"B": 5}}, [], {"r1": {"A": {"B": 5}}}),
-        ({"r1": {"A": 5}}, "r1", {"A": {}}, [], {"r1": {"A": None}}),
+        ({"r1": {"A": 5}}, "r1", {"A": {}}, [], {"r1": {"A": 5}}),
+        ({"r1": {"A": 5}}, "r1", {"A": None}, [], {"r1": {"A": None}}),
         (
             {"r1": {"A": {}}},
             "r1",

tests/test_datamodel_service.py

@@ -7,6 +7,7 @@
 from util.solver_workflow import new_solver_session  # noqa: F401
 
 from ansys.api.fluent.v0 import datamodel_se_pb2
+from ansys.api.fluent.v0.variant_pb2 import Variant
 import ansys.fluent.core as pyfluent
 from ansys.fluent.core import examples
 from ansys.fluent.core.services.datamodel_se import (
@@ -16,13 +17,36 @@
     PyNamedObjectContainer,
     PyTextual,
     ReadOnlyObjectError,
+    _convert_value_to_variant,
     _convert_variant_to_value,
     convert_path_to_se_path,
 )
 from ansys.fluent.core.streaming_services.datamodel_streaming import DatamodelStream
 from ansys.fluent.core.utils.execution import timeout_loop
 
 
+@pytest.mark.parametrize(
+    "value,expected",
+    [
+        (None, None),
+        (False, False),
+        (True, True),
+        (1, 1),
+        (1.0, 1.0),
+        ("abc", "abc"),
+        ((1, 2, 3), [1, 2, 3]),
+        ([1, 2, 3], [1, 2, 3]),
+        ({"a": 5}, {"a": 5}),
+        ({"a": [1, 2, 3]}, {"a": [1, 2, 3]}),
+        ({"a": {"b": 5}}, {"a": {"b": 5}}),
+    ],
+)
+def test_convert_value_to_variant_to_value(value, expected):
+    variant = Variant()
+    _convert_value_to_variant(value, variant)
+    assert expected == _convert_variant_to_value(variant)
+
+
 @pytest.mark.fluent_version(">=23.2")
 @pytest.mark.codegen_required
 def test_event_subscription(new_mesh_session):