Skip to content

Commit

Permalink
Check for child workflow already started error when setting env.testE…
Browse files Browse the repository at this point in the history
…rror (#1109)
  • Loading branch information
cdavis-joy committed May 25, 2023
1 parent ed13374 commit fc09967
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion internal/internal_workflow_testsuite.go
Original file line number Diff line number Diff line change
Expand Up @@ -872,7 +872,8 @@ func (env *testWorkflowEnvironmentImpl) Complete(result *commonpb.Payloads, err
var timeoutErr *TimeoutError
var workflowPanicErr *workflowPanicError
var workflowExecutionAlreadyStartedErr *serviceerror.WorkflowExecutionAlreadyStarted
if errors.As(err, &canceledErr) || errors.As(err, &continueAsNewErr) || errors.As(err, &timeoutErr) || errors.As(err, &workflowExecutionAlreadyStartedErr) {
var childWorkflowExecutionAlreadyStartedErr *ChildWorkflowExecutionAlreadyStartedError
if errors.As(err, &canceledErr) || errors.As(err, &continueAsNewErr) || errors.As(err, &timeoutErr) || errors.As(err, &workflowExecutionAlreadyStartedErr) || errors.As(err, &childWorkflowExecutionAlreadyStartedErr) {
env.testError = err
} else if errors.As(err, &workflowPanicErr) {
env.testError = newPanicError(workflowPanicErr.value, workflowPanicErr.stackTrace)
Expand Down

0 comments on commit fc09967

Please sign in to comment.