[Backport stable/8.9] test(agentic-ai): wait for active inbound executable to fix flaky A2A e2e tests#7131
Merged
Merged
Conversation
… e2e tests (#7100) * test(agentic-ai): wait for active inbound executable in A2A e2e tests The A2A webhook and polling e2e tests called `waitForElementActivation` which only checks that the BPMN catch event is active in the process. The inbound executable registry, however, processes activation events asynchronously via a scheduled task (fixedDelay = 1000ms). As a result, the inbound subscription could still be registering when the test sent its (delayed) HTTP POST or when the polling task was expected to start, causing flaky failures (issue #7033). Add an `awaitActiveInboundExecutable(elementId)` helper to `InboundConnectorTestHelper` that polls the executable registry until an executable for the element is registered and reports `Health.Status.UP`. Use it in the A2A standalone and AI Agent A2A integration tests after `waitForElementActivation`. Also trigger the import scheduler manually in the standalone polling test so it no longer depends on the default 5s scheduler tick. Closes #7033 * test(agentic-ai): consolidate A2A inbound connector readiness helper Address review feedback: extract the common "trigger import + wait for element + wait for executable" pattern into a single TestUtil.awaitInboundConnectorReady helper, used uniformly for both webhook and polling A2A tests. Trim the inline comments at call sites; the rationale lives in the javadoc. * test(agentic-ai): apply spotless formatting * test(agentic-ai): inline awaitInboundConnectorReady in L4J A2A tests * test(agentic-ai): wait for element activation before triggering import Order the steps in awaitInboundConnectorReady so the BPMN catch event is already active when the executable activates. For polling, this ensures the fetcher's first iteration (initialDelay=0) sees the running process instance instead of running while the instance is still en route to the catch event and missing it for one polling cycle. (cherry picked from commit 60af440)
ztefanie
approved these changes
May 5, 2026
Contributor
|
🎉 This pull request has been included in release Thank you for your contribution! 🚀 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
Backport of #7100 to
stable/8.9.