Test all depends of fields

Since 4ceb8ad07bd5 the depends attribute contains only some of the depending
fields. Tests should include the computed depends.
This commit is contained in:
Cédric Krier committed 2023-01-27 14:04:37 +01:00
1 parent e90e274a82
commit 3be0472924
2 files changed
+8 -2

No files matched your search

+3 -1
View File
@@ -142,7 +142,9 @@ def _iter_eval_fields(value):
def get_eval_fields(value):
"Return fields evaluated"
return set(_iter_eval_fields(value))
fields = set(_iter_eval_fields(value))
fields.discard('context') # TODO: remove when context is renamed
return fields
def instanciate_values(Target, value, **extra):
+5 -1
View File
@@ -452,7 +452,11 @@ class ModuleTestCase(unittest.TestCase):
for attribute in [
'depends', 'on_change', 'on_change_with',
'selection_change_with', 'autocomplete']:
depends = getattr(field, attribute, [])
depends = getattr(field, attribute, set())
if attribute == 'depends':
depends |= field.display_depends
depends |= field.edition_depends
depends |= field.validation_depends
qualname = '"%s"."%s"."%s"' % (mname, fname, attribute)
for depend in depends:
test_depend_exists(model, depend, qualname)