fix: add dependency for calculation in choice values

tests/test_choice_option.py

@@ -162,6 +162,10 @@ def test_choiceoption_calc_opt_function(config_type):
     cfg = get_config(cfg, config_type)
     assert cfg.option('choice').owner.isdefault()
     #
+    dep = cfg.option('str').dependencies()
+    assert len(dep) == 1
+    assert dep[0].path() == 'choice'
+    #
     cfg.option('choice').value.set('val1')
     assert cfg.option('choice').owner.get() == owner
     #

tiramisu/option/choiceoption.py

@@ -21,11 +21,12 @@
 """ChoiceOption
 """
 from typing import Any
+from itertools import chain
 
 from ..setting import undefined
 from ..i18n import _
 from .option import Option
-from ..autolib import Calculation, get_calculated_value
+from ..autolib import Calculation, get_calculated_value, ParamOption
 from ..error import ConfigError, display_list
 
 
@@ -42,7 +43,11 @@ class ChoiceOption(Option):
         """
         :param values: is a list of values the option can possibly take
         """
-        if not isinstance(values, (Calculation, tuple)):
+        if isinstance(values, Calculation):
+            for param in chain(values.params.args, values.params.kwargs.values()):
+                if isinstance(param, ParamOption):
+                    param.option._add_dependency(self)
+        elif not isinstance(values, tuple):
             raise TypeError(
                 _("values must be a tuple or a calculation for {0}").format(name)
             )