diff options
author | Jenkins <jenkins@review.openstack.org> | 2016-03-14 10:50:12 +0000 |
---|---|---|
committer | Gerrit Code Review <review@openstack.org> | 2016-03-14 10:50:12 +0000 |
commit | 844bf19be0309e0e741d67f9bf33e20426697abe (patch) | |
tree | 2836b0ba5c96c3591397e994e7edf86625ff4933 | |
parent | 78869fbccaffdf625fc1ec4ed3108bb7de892720 (diff) | |
parent | c8ff0e7f9672298859298960084721ee520fa060 (diff) | |
download | python-ceilometerclient-844bf19be0309e0e741d67f9bf33e20426697abe.tar.gz |
Merge "remove default value of repeat-actions"
-rw-r--r-- | ceilometerclient/tests/unit/test_shell.py | 19 | ||||
-rw-r--r-- | ceilometerclient/tests/unit/v2/test_shell.py | 12 | ||||
-rw-r--r-- | ceilometerclient/v2/shell.py | 1 |
3 files changed, 31 insertions, 1 deletions
diff --git a/ceilometerclient/tests/unit/test_shell.py b/ceilometerclient/tests/unit/test_shell.py index 46b640a..6323dd5 100644 --- a/ceilometerclient/tests/unit/test_shell.py +++ b/ceilometerclient/tests/unit/test_shell.py @@ -266,3 +266,22 @@ class ShellEndpointTest(ShellTestBase): self._test_endpoint_and_token('--os-auth-token', '--os-endpoint') self._test_endpoint_and_token('--os-token', '--ceilometer-url') self._test_endpoint_and_token('--os-token', '--os-endpoint') + + +class ShellAlarmUpdateRepeatAction(ShellTestBase): + @mock.patch('ceilometerclient.v2.alarms.AlarmManager.update') + @mock.patch('ceilometerclient.v2.client.Client._get_alarm_client', + mock.Mock()) + def test_repeat_action_not_specified(self, mocked): + self.make_env(FAKE_V2_ENV) + + def _test(method): + args = ['--debug', method, '--state', 'alarm', '123'] + ceilometer_shell.main(args) + args, kwargs = mocked.call_args + self.assertEqual(None, kwargs.get('repeat_actions')) + + _test('alarm-update') + _test('alarm-threshold-update') + _test('alarm-combination-update') + _test('alarm-event-update') diff --git a/ceilometerclient/tests/unit/v2/test_shell.py b/ceilometerclient/tests/unit/v2/test_shell.py index 8f518b2..de2b5db 100644 --- a/ceilometerclient/tests/unit/v2/test_shell.py +++ b/ceilometerclient/tests/unit/v2/test_shell.py @@ -266,6 +266,18 @@ class ShellAlarmCommandTest(utils.BaseTestCase): method = ceilometer_shell.do_alarm_threshold_update self._do_test_alarm_update_repeat_actions(method, False) + def test_alarm_event_upadte_repeat_action_untouched(self): + method = ceilometer_shell.do_alarm_event_update + self._do_test_alarm_update_repeat_actions(method, None) + + def test_alarm_event_upadte_repeat_action_set(self): + method = ceilometer_shell.do_alarm_event_update + self._do_test_alarm_update_repeat_actions(method, True) + + def test_alarm_event_upadte_repeat_action_clear(self): + method = ceilometer_shell.do_alarm_event_update + self._do_test_alarm_update_repeat_actions(method, False) + @mock.patch('sys.stdout', new=six.StringIO()) def test_alarm_threshold_create_args(self): argv = ['alarm-threshold-create'] + self.THRESHOLD_ALARM_CLI_ARGS diff --git a/ceilometerclient/v2/shell.py b/ceilometerclient/v2/shell.py index 09c2e95..771ea4d 100644 --- a/ceilometerclient/v2/shell.py +++ b/ceilometerclient/v2/shell.py @@ -503,7 +503,6 @@ def common_alarm_arguments(create=False): '[timezone=<IANA Timezone>]]')) @utils.arg('--repeat-actions', dest='repeat_actions', metavar='{True|False}', type=strutils.bool_from_string, - default=False, help=('True if actions should be repeatedly notified ' 'while alarm remains in target state.')) @functools.wraps(func) |