clebertsuconic commented on a change in pull request #3801:
URL: https://github.com/apache/activemq-artemis/pull/3801#discussion_r733710363



##########
File path: 
tests/integration-tests/src/test/java/org/apache/activemq/artemis/tests/integration/management/AddressControlTest.java
##########
@@ -607,6 +719,10 @@ protected AddressControl createManagementControl(final 
SimpleString address) thr
 
    // Private -------------------------------------------------------
 
+   private ActiveMQServerControl createManagementControl() throws Exception {

Review comment:
       this does not belong on AdressControlTest...
   
   
   ActiveMQServerControlTest is the one testing the ServerControl
   
   
   on AddressControlTest you should be testing the AddressControl instead.




-- 
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]


Reply via email to