openstack/deb-python-mistralclient
Code Issues Proposed changes

Merge "Add cancelled state to action executions"

Browse Source
This commit is contained in:
Jenkins
2016-11-17 07:10:32 +00:00
committed by Gerrit Code Review
parent cbe80a9675 190a0931da
commit c366a61ff9
2 changed files with 42 additions and 14 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
mistralclient/commands/v2/action_executions.py
View File

@@ -1,5 +1,5 @@
# Copyright 2014 - Mirantis, Inc. # Copyright 2014 - Mirantis, Inc.
# All Rights Reserved # Copyright 2016 - Brocade Communications Systems, Inc.
# #
# Licensed under the Apache License, Version 2.0 (the "License"); you may # Licensed under the Apache License, Version 2.0 (the "License"); you may
# not use this file except in compliance with the License. You may obtain # not use this file except in compliance with the License. You may obtain
@@ -228,7 +228,7 @@ class Update(command.ShowOne):
parser.add_argument( parser.add_argument(
'--state', '--state',
dest='state', dest='state',
choices=['IDLE', 'RUNNING', 'SUCCESS', 'ERROR'], choices=['IDLE', 'RUNNING', 'SUCCESS', 'ERROR', 'CANCELLED'],
help='Action execution state') help='Action execution state')
parser.add_argument( parser.add_argument(
'--output', '--output',

52
mistralclient/tests/unit/v2/test_cli_action_execs.py
View File

@@ -1,5 +1,5 @@
# Copyright 2014 Mirantis, Inc. # Copyright 2014 - Mirantis, Inc.
# All Rights Reserved # Copyright 2016 - Brocade Communications Systems, Inc.
# #
# Licensed under the Apache License, Version 2.0 (the "License"); you may # Licensed under the Apache License, Version 2.0 (the "License"); you may
# not use this file except in compliance with the License. You may obtain # not use this file except in compliance with the License. You may obtain
@@ -14,6 +14,7 @@
# under the License. # under the License.
# #
import copy
import json import json
import mock import mock
@@ -120,18 +121,45 @@ class TestCLIActionExecutions(base.BaseCommandTest):
) )
def test_update(self): def test_update(self):
self.client.action_executions.update.return_value = ACTION_EX states = ['IDLE', 'RUNNING', 'SUCCESS', 'ERROR', 'CANCELLED']
result = self.call( for state in states:
action_ex_cmd.Update, action_ex_dict = copy.deepcopy(ACTION_EX_DICT)
app_args=['id', '--state', 'ERROR'] action_ex_dict['state'] = state
) action_ex_dict['state_info'] = 'testing update'
action_ex_obj = action_ex.ActionExecution(mock, action_ex_dict)
self.client.action_executions.update.return_value = action_ex_obj
self.assertEqual( result = self.call(
('123', 'some', 'thing', 'task1', '1-2-3-4', 'RUNNING', action_ex_cmd.Update,
'RUNNING somehow.', True, '1', '1'), app_args=['id', '--state', state]
result[1] )
)
expected_result = (
action_ex_dict['id'],
action_ex_dict['name'],
action_ex_dict['workflow_name'],
action_ex_dict['task_name'],
action_ex_dict['task_execution_id'],
action_ex_dict['state'],
action_ex_dict['state_info'],
action_ex_dict['accepted'],
action_ex_dict['created_at'],
action_ex_dict['updated_at']
)
self.assertEqual(expected_result, result[1])
def test_update_invalid_state(self):
states = ['PAUSED', 'WAITING', 'DELAYED']
for state in states:
self.assertRaises(
SystemExit,
self.call,
action_ex_cmd.Update,
app_args=['id', '--state', state]
)
def test_list(self): def test_list(self):
self.client.action_executions.list.return_value = [ACTION_EX] self.client.action_executions.list.return_value = [ACTION_EX]