UniAsserter - ignore the assertFailedWith() if a previous stage fails

[mkouba]

• resolves UniAsserter - assertFailedWith() does not always assert the failure from the supplied Uni #29614

[geoand]

I am not sure I understand what this does exactly. Can you elaborate?

[mkouba]

I am not sure I understand what this does exactly. Can you elaborate?

I can try ;-). So previously we transformed the previous Uni (U1) to a new Uni provided by the supplier (U2) and then transformed the Uni based on the logic in the onItemOrFailure() callback into U3. I.e. U1 -> U2 -> (onItemOrFailure() + assert) -> U3. As a a result, when U1 failed we received the exception in the (onItemOrFailure() + assert) part.

After this change, we do something like U1 -> (U2 -> (onItemOrFailure() + assert) -> U3). And so if U1 fails then the (onItemOrFailure() + assert) part is not invoked at all.

Does it make any sense? I'm still not very sure when it comes to Mutiny but at least the tests don't fail :D

[geoand]

Thanks for the explanation.

I think it makes sense.

[quarkus-bot]

Failing Jobs - Building 53795e7

Status	Name	Step	Failures	Logs	Raw logs
✔️	JVM Tests - JDK 11
✔️	JVM Tests - JDK 17
✖	JVM Tests - JDK 17 MacOS M1	Set up runner	⚠️ Check →	Logs	Raw logs
✔️	JVM Tests - JDK 18