Skip to content

Throw BpmnError if ExceptionToErrorBoundaryEventTranslationErrorHandler is used in MessageSendTask#510

Merged
hhund merged 3 commits into
developfrom
issue/509_BpmnError-for-MessageSendTaskDelegate
Jun 3, 2026
Merged

Throw BpmnError if ExceptionToErrorBoundaryEventTranslationErrorHandler is used in MessageSendTask#510
hhund merged 3 commits into
developfrom
issue/509_BpmnError-for-MessageSendTaskDelegate

Conversation

@wetret

@wetret wetret commented May 29, 2026

Copy link
Copy Markdown
Member

resolves #509

hhund added 2 commits June 3, 2026 12:08
Consumers of the TestAssertException should be able to perform
instanceof checks, thus the class needs to be public. Only functions in
the PluginTestExecutor need to instantiate TestAssertException.
- Set ID of SendTaskErrorBoundaryEventTestThrow send task for easier
debugging.
- Removed not needed execution listener from
SendTaskErrorBoundaryEventTestThrow send task, mandatory profile field
now set via constant.
- Added missing condition expression to the flow from the exclusive
gateway to the SendTaskErrorBoundaryEventTestThrow send task.
- Added default flows to the exclusive gateways of the v1 and v2 test
BPMNs to catch missing condition expression errors in the future.

@hhund hhund left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I added a missing gateway condition and simplified the new test case, see 6eef8fc
Also fixed an unrelated class/constructor visibility error in PluginTestExecutor via 9e31662

@hhund hhund merged commit b062c7b into develop Jun 3, 2026
20 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

MessageSendTaskDelegate does not throw BPMN Error if ExceptionToErrorBoundaryEventTranslationErrorHandler is used

2 participants