Yasuke/moto
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Add WorkflowExecution.open to clarify code in some places

Browse Source
This commit is contained in:
Jean-Baptiste Barth 2015-11-04 22:25:27 +01:00
parent f38d23e483
commit 9c3996ff58
3 changed files with 8 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
moto/swf/models/__init__.py
View File

@ -243,7 +243,7 @@ class SWFBackend(BaseBackend):
raise SWFValidationException("Invalid token")
# decision task found, but WorflowExecution is CLOSED
wfe = decision_task.workflow_execution
if wfe.execution_status != "OPEN":
if not wfe.open:
raise SWFUnknownResourceFault(
"execution",
"WorkflowExecution=[workflowId={0}, runId={1}]".format(
@ -331,7 +331,7 @@ class SWFBackend(BaseBackend):
raise SWFValidationException("Invalid token")
# activity task found, but WorflowExecution is CLOSED
wfe = activity_task.workflow_execution
if wfe.execution_status != "OPEN":
if not wfe.open:
raise SWFUnknownResourceFault(
"execution",
"WorkflowExecution=[workflowId={0}, runId={1}]".format(

2
moto/swf/models/domain.py
View File

@ -83,7 +83,7 @@ class Domain(object):
if w.workflow_id == workflow_id and w.run_id == run_id]
else:
_all = [w for w in self.workflow_executions
if w.workflow_id == workflow_id and w.execution_status == "OPEN"]
if w.workflow_id == workflow_id and w.open]
# reduce
wfe = _all[0] if _all else None
# raise if closed / none

6
moto/swf/models/workflow_execution.py
View File

@ -523,7 +523,7 @@ class WorkflowExecution(object):
self.close_cause = "OPERATOR_INITIATED"
def has_timedout(self):
if self.execution_status != "OPEN" or not self.start_timestamp:
if not self.open or not self.start_timestamp:
return False
# TODO: handle the "NONE" case
start_to_close_timeout = self.start_timestamp + \
@ -535,3 +535,7 @@ class WorkflowExecution(object):
self.execution_status = "CLOSED"
self.close_status = "TIMED_OUT"
self.timeout_type = "START_TO_CLOSE"
@property
def open(self):
return self.execution_status == "OPEN"