The TxInflowUnitTestCase is currently failing because of a bug in the subordinate transaction implementation.

The test creates an inflow through the JCA WorkManager, enlists a single resource and then executes a one phase commit via the XATerminator. The logic in the one phase commit code is, unfortunately, incorrect.

The code attempts to invoke end on all suspended resources and, if this succeeds, marks the transaction for rollback. The result is that all successful, subordinate, one phase transactions are rolled back.

The logic should be reversed so that a failure marks the transaction for rollback.