From a5870ee2ab151d8618f6a8dbaa7d67c7abd7bd19 Mon Sep 17 00:00:00 2001 From: Emmanuel Garette Date: Mon, 24 Nov 2025 20:06:54 +0100 Subject: [PATCH] fix: force_store_value in dynamic family --- tests/test_dyn_optiondescription.py | 11 +++++++++++ tiramisu/option/optiondescription.py | 2 +- 2 files changed, 12 insertions(+), 1 deletion(-) diff --git a/tests/test_dyn_optiondescription.py b/tests/test_dyn_optiondescription.py index f39473d..04755f8 100644 --- a/tests/test_dyn_optiondescription.py +++ b/tests/test_dyn_optiondescription.py @@ -344,6 +344,17 @@ def test_prop_dyndescription_force_store_value(): # assert not list_sessions() +def test_prop_dyndescription_force_store_value_disabled(): + st = StrOption('st', '', properties=('force_store_value', 'disabled')) + dod = DynOptionDescription('dod', '', [st], identifiers=Calculation(return_list)) + od = OptionDescription('od', '', [dod]) + od2 = OptionDescription('od', '', [od]) + cfg = Config(od2) + cfg.property.read_write() + assert parse_od_get(cfg.value.get()) == {} +# assert not list_sessions() + + def test_prop_dyndescription_force_store_value_calculation_prefix(): lst = StrOption('lst', '', ['val1', 'val2'], multi=True) st = StrOption('st', '', Calculation(return_list, Params(ParamIdentifier())) , properties=('force_store_value',)) diff --git a/tiramisu/option/optiondescription.py b/tiramisu/option/optiondescription.py index 9fdd0df..eb8d671 100644 --- a/tiramisu/option/optiondescription.py +++ b/tiramisu/option/optiondescription.py @@ -176,8 +176,8 @@ class CacheOptionDescription(BaseOption): parent.get_child( doption, None, - True, name=name, + validate_properties=False, ) ) parents = new_parents