Include TestContext in log message when starting a new transaction in TransactionalTestExecutionListener [SPR-11323]

[spring-projects-issues]

Sam Brannen opened SPR-11323 and commented

Status Quo

TransactionalTestExecutionListener.endTransaction() includes the TestContext in trace logging output; however, TransactionalTestExecutionListener.startNewTransaction() currently does not include it in its info logging output, thus making it more difficult to debug.

Deliverables

1. Ensure that startNewTransaction() also includes the current TestContext in logging output.

---

Affects: 3.0 GA

Issue Links:

• AbstractTransactionalTestNGSpringContextTests not working as expected when an EJB with TransactionAttribute.REQUIRES_NEW is encountered [SPR-6132] #10800 AbstractTransactionalTestNGSpringContextTests not working as expected when an EJB with TransactionAttribute.REQUIRES_NEW is encountered

Referenced from: commits 6e30851

[spring-projects-issues]

Sam Brannen commented

Resolving this issue will make it easier to analyze issues such as those described in #10800.

[spring-projects-issues]

Sam Brannen commented

Completed as described in the comments for GitHub commit 6e30851:

*Improve logging in TransactionalTEL*

This commit makes the logging in TransactionalTestExecutionListener
consistent for both starting and ending transactions. Specifically,
the current TestContext is now included in the informational log
statement when starting a new transaction.