diff --git a/senlinclient/tests/unit/v1/test_shell.py b/senlinclient/tests/unit/v1/test_shell.py index 8cd23e54..31c4374c 100644 --- a/senlinclient/tests/unit/v1/test_shell.py +++ b/senlinclient/tests/unit/v1/test_shell.py @@ -511,7 +511,7 @@ class ShellTest(testtools.TestCase): 'type': 'webhook', 'cluster': 'cluster1', 'action': 'CLUSTER_SCALE_IN', - 'params': {} + 'params': [] } args = self._make_args(args) params = { @@ -535,7 +535,7 @@ class ShellTest(testtools.TestCase): 'type': 'webhook', 'cluster': None, 'action': None, - 'params': {} + 'params': [] } args = self._make_args(args) ex = self.assertRaises(exc.CommandError, @@ -552,7 +552,7 @@ class ShellTest(testtools.TestCase): 'type': 'foo', 'cluster': None, 'action': None, - 'params': {} + 'params': [] } args = self._make_args(args) params = { @@ -576,13 +576,13 @@ class ShellTest(testtools.TestCase): 'name': 'receiver2', 'id': 'receiver_id', 'action': 'CLUSTER_SCALE_OUT', - 'params': {'count': '2'} + 'params': ['key1=value1;key2=value2'] } args = self._make_args(args) params = { 'name': 'receiver2', 'action': 'CLUSTER_SCALE_OUT', - 'params': {'count': '2'} + 'params': {'key1': 'value1', 'key2': 'value2'} } receiver = mock.Mock() receiver.id = 'receiver_id' diff --git a/senlinclient/v1/receiver.py b/senlinclient/v1/receiver.py index dbc43f56..67ad8a95 100644 --- a/senlinclient/v1/receiver.py +++ b/senlinclient/v1/receiver.py @@ -214,7 +214,7 @@ class CreateReceiver(command.ShowOne): class UpdateReceiver(command.ShowOne): - """Create a receiver.""" + """Update a receiver.""" log = logging.getLogger(__name__ + ".UpdateReceiver") diff --git a/senlinclient/v1/shell.py b/senlinclient/v1/shell.py index ac5380a3..51f4172e 100644 --- a/senlinclient/v1/shell.py +++ b/senlinclient/v1/shell.py @@ -1584,7 +1584,7 @@ def do_receiver_update(service, args): params = { 'name': args.name, 'action': args.action, - 'params': args.params + 'params': utils.format_parameters(args.params) } # Find the receiver first, we need its id