[GitHub] activemq-artemis pull request #2448: ARTEMIS-2189 allow deleting temp dest w...

[jbertram]

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

    https://github.com/apache/activemq-artemis/pull/2448#discussion_r238422596
  
    --- Diff: 
tests/jms-tests/src/test/java/org/apache/activemq/artemis/jms/tests/TemporaryDestinationTest.java
 ---
    @@ -265,6 +266,126 @@ public void testTemporaryQueueDeleted() throws 
Exception {
           }
        }
     
    +   @Test
    +   public void testTemporaryQueueDeletedAfterSessionClosed() throws 
Exception {
    --- End diff --
    
    The issue here is not with the broker but with the core client. Therefore, 
I'm not sure it makes sense to test other clients on this particular use-case.


---