Skip to content

Fix "ISender#sendMessageOrThrow" closing result that has also been referenced in Session keys#9024

Merged
nielsm5 merged 7 commits intomasterfrom
issue/9015_ForEachChildReturnsEmptyKeys
May 23, 2025
Merged

Fix "ISender#sendMessageOrThrow" closing result that has also been referenced in Session keys#9024
nielsm5 merged 7 commits intomasterfrom
issue/9015_ForEachChildReturnsEmptyKeys

Conversation

@tnleeuw
Copy link
Contributor

@tnleeuw tnleeuw commented May 21, 2025

Related to #9015.

Demonstrate the behaviour of "returnedSessionKeys" with a ForEachChildElementPipe using an IbisLocalSender.

…combination with IbisLocalSender and feature "returnedSessionKeys"
@tnleeuw tnleeuw requested a review from nielsm5 May 21, 2025 13:25
@tnleeuw tnleeuw self-assigned this May 21, 2025
@tnleeuw tnleeuw changed the title Add a test demonstrating the behaviour of ForEachChildElementPipe in combination with IbisLocalSender and feature "returnedSessionKeys" Fix "ISender#sendMessageOrThrow" closing result that has also been referenced in Session keys May 21, 2025
@tnleeuw tnleeuw requested a review from evandongen May 21, 2025 19:56
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ik wou eigenlijk eerst een test voor ForEachChildElementPipe hieraan toevoegen maar realiseerde me al snel dat alle helper-methods die ik nodig zou hebben, in de IbisLocalSenderTest zitten dus ik heb daar de nieuwe tests toegevoegd.

error.setState(PipeLine.ExitState.ERROR);
pl.addPipeLineExit(error);

InputOutputPipeProcessor iopp = SpringUtils.createBean(configuration);
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ik heb de InputOutputPipeProcessor nodig voor de optie storeResultInSessionKey te kunnen gebruiken en zo beter het scenario uit de bug te kunnen reproduceren.

Copy link
Member

Choose a reason for hiding this comment

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

Volgens mij worden de processors automatisch gezet? Als je alle processor dingen hier uit haalt (en Spring niet overschrijft) zou het ook moeten werken.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ik had het eerder nodig. Misschien dat toen de eerste versies van deze test gemaakt werden, de Pipeline niet correct gemaakt werd waardoor het toen nodig was.
Ik zal het zo even uittesten.

Copy link
Contributor Author

@tnleeuw tnleeuw May 23, 2025

Choose a reason for hiding this comment

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

Als het het creeeren van de processors weghaal, gaan heel veel tests fout.

De processors worden wel gemaakt, maar de TransactionAttributePipeLineProcessor gaat fout omdat hij geen TransactionManager heeft.

Als ik niet new TestConfiguration(false) doe maar TransactionManagerType.DATASOURCE.create(false) dan gaan tests op een andere manier fout, ik zie timeout errors in een paar tests.
Dus dat haalt weer andere dingen overhoop.

Dan laat ik het liever even zoals het is.

Heb die tests ook weten te fixen, er was iets wat ik over het hoofd gezien had.

@sonarqubecloud
Copy link

@nielsm5 nielsm5 enabled auto-merge (squash) May 23, 2025 11:29
@nielsm5 nielsm5 merged commit b021800 into master May 23, 2025
33 checks passed
@nielsm5 nielsm5 deleted the issue/9015_ForEachChildReturnsEmptyKeys branch May 23, 2025 11:35
tnleeuw added a commit that referenced this pull request May 28, 2025
…ferenced in Session keys (#9024)

(cherry picked from commit b021800)
tnleeuw added a commit that referenced this pull request May 28, 2025
…ferenced in Session keys (#9024)

(cherry picked from commit b021800)
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.

ForEachChildElementPipe with IbisLocalSender "returnedSessionKeys" results in empty sessionKey in parent adapter

3 participants