You are viewing a plain text version of this content. The canonical link for it is here.

Posted to gitbox@activemq.apache.org by GitBox <gi...@apache.org> on 2022/12/21 18:07:10 UTC

[GitHub] [activemq-artemis] jbertram opened a new pull request, #4319: ARTEMIS-3707 fixing tests

jbertram opened a new pull request, #4319:
URL: https://github.com/apache/activemq-artemis/pull/4319

   A handful of tests started to fail after the original fix was committed. This commit fixes those failures mainly by using a mock `TransactionSynchronizationRegistry`.
   
   I changed `o.a.a.a.r.ActiveMQRAManagedConnection#checkTransactionActive` slightly because `getTransactionStatus` will never return `null` unlike `getTransaction` would. The semantics should still be the same, though.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: gitbox-unsubscribe@activemq.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [activemq-artemis] jbertram merged pull request #4319: ARTEMIS-3707 fixing tests

Posted by GitBox <gi...@apache.org>.

jbertram merged PR #4319:
URL: https://github.com/apache/activemq-artemis/pull/4319


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: gitbox-unsubscribe@activemq.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org