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 2019/03/21 09:35:42 UTC
[GitHub] [activemq-artemis] gaohoward commented on a change in pull request
#2587: ARTEMIS-2280 Add new test for moving message to FQQN
gaohoward commented on a change in pull request #2587: ARTEMIS-2280 Add new test for moving message to FQQN
URL: https://github.com/apache/activemq-artemis/pull/2587#discussion_r267676395
##########
File path: tests/integration-tests/src/test/java/org/apache/activemq/artemis/tests/integration/management/QueueControlTest.java
##########
@@ -1329,6 +1329,64 @@ public void testMoveMessage() throws Exception {
session.deleteQueue(otherQueue);
}
+ /**
+ * Moving message from another address to a single "child" queue of a multicast address
+ *
+ * <address name="ErrorQueue">
+ * <anycast>
+ * <queue name="ErrorQueue" />
+ * </anycast>
+ * </address>
+ * <address name="parent.addr.1">
+ * <multicast>
+ * <queue name="child.queue.1" />
+ * <queue name="child.queue.2" />
+ * <queue name="child.queue.3" />
Review comment:
Base on the test, this queue is never created/used. Should be removed.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services