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

    https://github.com/apache/activemq-artemis/pull/1750#discussion_r160195533
  
    --- 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 problematic use-case wasn't necessarily with a configuration file, but 
simply using a URL (which can be done programmatically).  I changed the test a 
bit before I merged it.  It's a bit simpler this way.


---

Reply via email to