Github user jbertram commented on a diff in the pull request:

    https://github.com/apache/activemq-artemis/pull/1750#discussion_r159895861
  
    --- Diff: 
tests/integration-tests/src/test/java/org/apache/activemq/artemis/tests/integration/jms/server/config/JMSConfigurationTest.java
 ---
    @@ -97,4 +111,56 @@ public void testSetupJMSConfiguration() throws 
Exception {
     
           server.stop();
        }
    +
    +   @Test
    +   public void testHandshakeTimeoutWithValueSet() throws Exception {
    +      JMSServerManager jmsServer;
    --- End diff --
    
    The JMSServerManager is deprecated so it's probably best not to use it in 
new tests.  Check out the "embedded-simple" example for how to programmatically 
instantiate a broker with a configuration file.


---

Reply via email to