Enthuware

In option 2, instead of

call ut.getStatus() != Status.STATUS_ROLLEDBACK at INSERT CODE HERE.

shouldn't it be:

call ut.getStatus() != Status.STATUS_MARKED_ROLLBACK at INSERT CODE HERE.

Because anotherMethod() merely marked the txn for rollback instead of actually rolling back the txn...

I think STATUS_ROLLEDBACK is fine as well because the API for STATUS_ROLLEDBACK says,

A transaction is associated with the target object and the outcome has been determined to be rollback. It is likely that heuristics exist; otherwise, the transaction would have been destroyed and NoTransaction returned.

First option is right:

if (!ut.getRollbackOnly()) {
// executes only if anotherMethod has NOT set the transaction for a rollback
}

Am I missing something?

UT does not have such method

getRollbackOnly() method is reserved for container managed transaction, ejbContext.getRollBackOnly().

Just tried it with Glassfish and the status that it gets in this situation is Status.STATUS_MARKED_ROLLBACK.
And if the client only marks as rollback, then it can never be Status.STATUS_ROLLEDBACK at this point because no commit or rollback was attempted. The transaction is either active or marked for rollback.

This has now been fixed.
Thank you for your feedback!