git.net

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[GitHub] activemq-artemis pull request #2116: ARTEMIS-1895 - Add duplicate metadata f...


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

    https://github.com/apache/activemq-artemis/pull/2116#discussion_r191787441
  
    --- Diff: tests/integration-tests/src/test/java/org/apache/activemq/artemis/tests/integration/jms/client/SessionMetadataAddExceptionTest.java ---
    @@ -75,6 +88,24 @@ public void testInvalidClientIdSetFactory() throws Exception {
           cf.createConnection();
        }
     
    +   @Test(timeout = 5000, expected = JMSException.class)
    +   public void testDuplicateClientIdSet() throws Exception {
    +      ActiveMQConnectionFactory activeMQConnectionFactory = (ActiveMQConnectionFactory) cf;
    +      Connection con = cf.createConnection();
    +      Connection con2 = cf.createConnection();
    +      try {
    +         con.setClientID("valid");
    +         con2.setClientID("valid");
    +         fail("Should have failed for duplicate clientId");
    +      } catch (Exception e) {
    --- End diff --
    
    I switched it to catch the InvalidClientIdException that gets thrown


---



( ! ) Warning: include(msgfooter.php): failed to open stream: No such file or directory in /var/www/git/apache-activemq-developers/msg08722.html on line 94
Call Stack
#TimeMemoryFunctionLocation
10.0006363704{main}( ).../msg08722.html:0

( ! ) Warning: include(): Failed opening 'msgfooter.php' for inclusion (include_path='.:/var/www/git') in /var/www/git/apache-activemq-developers/msg08722.html on line 94
Call Stack
#TimeMemoryFunctionLocation
10.0006363704{main}( ).../msg08722.html:0