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 2023/01/12 05:44:30 UTC
[GitHub] [activemq-artemis] jbertram opened a new pull request, #4328: ARTEMIS-4127 refactor multiprotocol JMS tests in AMQP package
jbertram opened a new pull request, #4328:
URL: https://github.com/apache/activemq-artemis/pull/4328
Over time org.apache.activemq.artemis.tests.integration.amqp has become home to many multi-protocol JMS tests even though the package is really for AMQP-specific tests. This commit splits those tests out into their own package.
This is a preliminary step to clarify these tests before I add another one for a different issue.
--
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] michaelandrepearce commented on pull request #4328: ARTEMIS-4127 refactor multiprotocol JMS tests in AMQP package
Posted by GitBox <gi...@apache.org>.
michaelandrepearce commented on PR #4328:
URL: https://github.com/apache/activemq-artemis/pull/4328#issuecomment-1379875336
This change looks good, there is test failure, but don't believe its related to this change. But if in another cycle anyone has a chance to look at would be good, bit of a worry.......
Build / Test (17) (pull_request) Failing after 25m
- [INFO]
Error: Failures:
Error: AcknowledgementTest.testDupsOKAcknowledgeQueue:555->ActiveMQServerTestCase.assertRemainingMessages:364 expected:<20> but was:<21>
--
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] clebertsuconic merged pull request #4328: ARTEMIS-4127 refactor multiprotocol JMS tests in AMQP package
Posted by GitBox <gi...@apache.org>.
clebertsuconic merged PR #4328:
URL: https://github.com/apache/activemq-artemis/pull/4328
--
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