From 6aecbc227c69e5a635c7f4e17305d4f4d93ea277 Mon Sep 17 00:00:00 2001 From: Tim Nicholls Date: Mon, 17 Oct 2016 17:01:18 +0100 Subject: [PATCH] Added additional test to test_parameter_tree.py for coverage --- .../odin/testing/adapters/test_parameter_tree.py | 14 +++++++++++--- 1 file changed, 11 insertions(+), 3 deletions(-) diff --git a/server/odin/testing/adapters/test_parameter_tree.py b/server/odin/testing/adapters/test_parameter_tree.py index 461e361b..cbbea7ac 100644 --- a/server/odin/testing/adapters/test_parameter_tree.py +++ b/server/odin/testing/adapters/test_parameter_tree.py @@ -165,6 +165,8 @@ def setup_class(cls): cls.int_rw_value = 9876 cls.int_wo_param = 0 + cls.rw_value_set_called = False + cls.nested_rw_param = 53.752 cls.nested_ro_value = 9.8765 @@ -177,7 +179,7 @@ def setup_class(cls): 'intCallableRwParam': (cls.intCallableRwParamGet, cls.intCallableRwParamSet), 'intCallableRoParam': (cls.intCallableRoParamGet, None), 'intCallableWoParam': (None, cls.intCallableWoParamSet), - 'intCallableRwValue': (cls.int_rw_value, cls.intCallableRoParamSet), + 'intCallableRwValue': (cls.int_rw_value, cls.intCallableRwValueSet), 'branch': nested_tree }) @@ -198,8 +200,8 @@ def intCallableWoParamSet(cls, value): cls.int_wo_param = value @classmethod - def intCallableRoParamSet(cls, value): - pass + def intCallableRwValueSet(cls, value): + cls.rw_value_set_called = True @classmethod def nestedRwParamSet(cls, value): @@ -239,6 +241,12 @@ def test_rw_callable_tree_set_wo_param(self): self.rw_callable_tree.set('intCallableWoParam', new_value) assert_equal(self.int_wo_param, new_value) + def test_rw_callable_tree_set_rw_value(self): + + new_value = 1234 + self.rw_callable_tree.set('intCallableRwValue', new_value) + assert_true(self.rw_value_set_called) + def test_rw_callable_nested_param_get(self): dt_nested_param = self.rw_callable_tree.get('branch/nestedRwParam')