[
https://issues.apache.org/jira/browse/ARIA-155?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15990210#comment-15990210
]
ASF GitHub Bot commented on ARIA-155:
-------------------------------------
Github user mxmrlv commented on a diff in the pull request:
https://github.com/apache/incubator-ariatosca/pull/114#discussion_r114074274
--- Diff: tests/modeling/test_models.py ---
@@ -254,27 +247,26 @@ def test_service_model_creation(self,
service_storage, is_valid, name, created_a
class TestExecution(object):
@pytest.mark.parametrize(
- 'is_valid, created_at, started_at, ended_at, error,
is_system_workflow, inputs, '
+ 'is_valid, created_at, started_at, ended_at, error, inputs, '
'status, workflow_name',
[
- (False, m_cls, now, now, 'error', False, {},
Execution.STARTED, 'wf_name'),
- (False, now, m_cls, now, 'error', False, {},
Execution.STARTED, 'wf_name'),
- (False, now, now, m_cls, 'error', False, {},
Execution.STARTED, 'wf_name'),
- (False, now, now, now, m_cls, False, {}, Execution.STARTED,
'wf_name'),
- (False, now, now, now, 'error', False, m_cls,
Execution.STARTED, 'wf_name'),
- (False, now, now, now, 'error', False, {}, m_cls, 'wf_name'),
- (False, now, now, now, 'error', False, {}, Execution.STARTED,
m_cls),
-
- (True, now, now, now, 'error', False, {}, Execution.STARTED,
'wf_name'),
- (True, now, None, now, 'error', False, {}, Execution.STARTED,
'wf_name'),
- (True, now, now, None, 'error', False, {}, Execution.STARTED,
'wf_name'),
- (True, now, now, now, None, False, {}, Execution.STARTED,
'wf_name'),
- (True, now, now, now, 'error', False, {}, Execution.STARTED,
'wf_name'),
+ (False, m_cls, now, now, 'error', {}, Execution.STARTED,
'wf_name'),
+ (False, now, m_cls, now, 'error', {}, Execution.STARTED,
'wf_name'),
+ (False, now, now, m_cls, 'error', {}, Execution.STARTED,
'wf_name'),
+ (False, now, now, now, m_cls, {}, Execution.STARTED,
'wf_name'),
+ (False, now, now, now, 'error', m_cls, Execution.STARTED,
'wf_name'),
+ (False, now, now, now, 'error', {}, m_cls, 'wf_name'),
+ (False, now, now, now, 'error', {}, Execution.STARTED, m_cls),
+
+ (True, now, now, now, 'error', {}, Execution.STARTED,
'wf_name'),
+ (True, now, None, now, 'error', {}, Execution.STARTED,
'wf_name'),
+ (True, now, now, None, 'error', {}, Execution.STARTED,
'wf_name'),
+ (True, now, now, now, None, {}, Execution.STARTED, 'wf_name'),
+ (True, now, now, now, 'error', {}, Execution.STARTED,
'wf_name'),
--- End diff --
same as 261
> Clean models from unused fields
> -------------------------------
>
> Key: ARIA-155
> URL: https://issues.apache.org/jira/browse/ARIA-155
> Project: AriaTosca
> Issue Type: Task
> Reporter: Ran Ziv
> Assignee: Ran Ziv
> Priority: Minor
>
> Some models have fields which should probably be removed (as they're no
> longer in use or haven't been put to use yet) or rename (due to
> inconsistencies).
> Suggested changes:
> Service changes module:
> - remove module entirely, until we actually implement scaling /
> service-update
> Service:
> - remove {{permalink}}
> - remove {{scaling_groups}}
> Node:
> - remove {{scaling_groups}}
> Operation:
> - rename {{max_retries}} to {{max_attempts}}
> OperationTemplate:
> - rename {{max_retries}} to {{max_attempts}}
> Execution:
> - remove {{is_system_workflow}}
> Task:
> - rename {{retry_count}} to {{attempts_count}}
--
This message was sent by Atlassian JIRA
(v6.3.15#6346)
