openstack/deb-python-senlinclient
Code Issues Proposed changes

Fix update_policy parameter passing error

Browse Source 
Closes-Bug: #1534072
Change-Id: I98079458b4d1f04ba64bbd40e4640b8547bc1af8
This commit is contained in:
EldonZhao 2016-01-15 16:08:19 +08:00
parent 9118fff840
commit 3576f5d65b
2 changed files with 2 additions and 4 deletions
senlinclient
tests/unit/v1
test_shell.py
v1
shell.py

3
senlinclient/tests/unit/v1/test_shell.py

@ -557,7 +557,6 @@ class ShellTest(testtools.TestCase):
args = self._make_args(args)
params = {
'name': 'deletion_policy',
'id': 'policy_id'
}
policy = mock.Mock()
service.get_policy.return_value = policy
@ -565,7 +564,7 @@ class ShellTest(testtools.TestCase):
sh.do_policy_update(service, args)
service.get_policy.assert_called_once_with('policy_id')
service.update_policy.assert_called_once_with(
'policy_id', params)
'policy_id', **params)
mock_show(service, policy_id=policy.id)
def test_do_policy_delete(self):

3
senlinclient/v1/shell.py

@ -329,8 +329,7 @@ def do_policy_update(service, args):
policy = service.get_policy(args.id)
if policy is not None:
params['id'] = policy.id
service.update_policy(policy.id, params)
service.update_policy(policy.id, **params)
_show_policy(service, policy_id=policy.id)