clebertsuconic commented on a change in pull request #3801:
URL: https://github.com/apache/activemq-artemis/pull/3801#discussion_r733709420
##########
File path:
tests/integration-tests/src/test/java/org/apache/activemq/artemis/tests/integration/management/AddressControlTest.java
##########
@@ -580,6 +591,107 @@ public void testPurge() throws Exception {
Wait.assertEquals(0L, () -> addressControl.getMessageCount(), 2000, 100);
}
+ @Test
+ public void testReplayWithoutDate() throws Exception {
+ testReplaySimple(false);
+ }
+
+ @Test
+ public void testReplayWithDate() throws Exception {
+ testReplaySimple(true);
+ }
+
+ private void testReplaySimple(boolean useDate) throws Exception {
+ ActiveMQServerControl serverControl = createManagementControl();
Review comment:
the point I made about copying, wasn't to be literal :)
you have to use the ActiveMQAddressControl on this... you are not testing
your new code, you are testing the old code on this case.
Everything else seems spot on.
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
