Skip to content

Commit

Permalink
Added additional test to test_parameter_tree.py for coverage
Browse files Browse the repository at this point in the history
  • Loading branch information
timcnicholls committed Oct 17, 2016
1 parent b3c6437 commit 6aecbc2
Showing 1 changed file with 11 additions and 3 deletions.
14 changes: 11 additions & 3 deletions server/odin/testing/adapters/test_parameter_tree.py
Original file line number Diff line number Diff line change
Expand Up @@ -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

Expand All @@ -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
})

Expand All @@ -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):
Expand Down Expand Up @@ -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')
Expand Down

0 comments on commit 6aecbc2

Please sign in to comment.