New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[JBPM-9459] Nested process instances abort doesn't work when a task in event subprocess is marked with "Is Async" as true #1797
Conversation
98954e3
to
6792e1e
Compare
Jenkins retest this |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me, just only a few changes in the test case.
jbpm-flow/src/main/java/org/jbpm/workflow/instance/node/EventSubProcessNodeInstance.java
Outdated
Show resolved
Hide resolved
jbpm-test-coverage/src/test/resources/org/jbpm/test/functional/async/Exception.bpmn2
Outdated
Show resolved
Hide resolved
</bpmn2:assignment> | ||
</bpmn2:dataInputAssociation> | ||
</bpmn2:task> | ||
<bpmn2:task id="_1DBE0765-593C-45A8-B3AA-337A64D3062E" drools:taskName="Rest" name="REST"> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There's no need for a second task call actually. This can be removed.
jbpm-test-coverage/src/test/resources/org/jbpm/test/functional/async/Level1P.bpmn2
Outdated
Show resolved
Hide resolved
jbpm-test-coverage/src/test/resources/org/jbpm/test/functional/async/Level2C.bpmn2
Outdated
Show resolved
Hide resolved
jbpm-test-coverage/src/test/java/org/jbpm/test/functional/async/AsyncAdHocSubprocessTest.java
Outdated
Show resolved
Hide resolved
jbpm-test-coverage/src/test/java/org/jbpm/test/functional/async/AsyncAdHocSubprocessTest.java
Outdated
Show resolved
Hide resolved
jbpm-test-coverage/src/test/java/org/jbpm/test/functional/async/AsyncAdHocSubprocessTest.java
Show resolved
Hide resolved
jbpm-test-coverage/src/test/java/org/jbpm/test/functional/async/AsyncAdHocSubprocessTest.java
Outdated
Show resolved
Hide resolved
jbpm-test-coverage/src/test/java/org/jbpm/test/functional/async/AsyncAdHocSubprocessTest.java
Outdated
Show resolved
Hide resolved
@elguardian - I can see there're some failures in the CI tests. I noticed it's also failing in my local environment with this PR, just for you to know. |
…n event subprocess is marked with "Is Async" as true
6792e1e
to
cf058d3
Compare
@afalhambra fixed... some minor issue with the PR. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great work, just wait for the CI to go green
Kudos, SonarCloud Quality Gate passed! 0 Bugs |
Jira: https://issues.redhat.com/browse/JBPM-9459