You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@activemq.apache.org by jb...@apache.org on 2016/07/15 14:31:09 UTC

[1/2] activemq-artemis git commit: This closes #636

Repository: activemq-artemis
Updated Branches:
  refs/heads/master 55fe2b3b2 -> 770127438


This closes #636


Project: http://git-wip-us.apache.org/repos/asf/activemq-artemis/repo
Commit: http://git-wip-us.apache.org/repos/asf/activemq-artemis/commit/77012743
Tree: http://git-wip-us.apache.org/repos/asf/activemq-artemis/tree/77012743
Diff: http://git-wip-us.apache.org/repos/asf/activemq-artemis/diff/77012743

Branch: refs/heads/master
Commit: 770127438e9b21b85434cae148e37fca2d7c9ef4
Parents: 55fe2b3 aebf87a
Author: jbertram <jb...@apache.org>
Authored: Fri Jul 15 09:30:53 2016 -0500
Committer: jbertram <jb...@apache.org>
Committed: Fri Jul 15 09:30:53 2016 -0500

----------------------------------------------------------------------
 .../jms/server/management/JMSServerControlTest.java            | 6 ++++++
 1 file changed, 6 insertions(+)
----------------------------------------------------------------------



[2/2] activemq-artemis git commit: ARTEMIS-632 JMSServerControlUsingJMSTest fails

Posted by jb...@apache.org.
ARTEMIS-632 JMSServerControlUsingJMSTest fails

The testRemoteClientIDConnection fails. The failure occurs when
connection objects are destroyed by garbage collector before time.


Project: http://git-wip-us.apache.org/repos/asf/activemq-artemis/repo
Commit: http://git-wip-us.apache.org/repos/asf/activemq-artemis/commit/aebf87ae
Tree: http://git-wip-us.apache.org/repos/asf/activemq-artemis/tree/aebf87ae
Diff: http://git-wip-us.apache.org/repos/asf/activemq-artemis/diff/aebf87ae

Branch: refs/heads/master
Commit: aebf87aeeeb6f03eb7c30c069d87f78e4a2eb8b6
Parents: 55fe2b3
Author: Erich Duda <ed...@redhat.com>
Authored: Fri Jul 15 09:01:36 2016 +0200
Committer: jbertram <jb...@apache.org>
Committed: Fri Jul 15 09:30:53 2016 -0500

----------------------------------------------------------------------
 .../jms/server/management/JMSServerControlTest.java            | 6 ++++++
 1 file changed, 6 insertions(+)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/activemq-artemis/blob/aebf87ae/tests/integration-tests/src/test/java/org/apache/activemq/artemis/tests/integration/jms/server/management/JMSServerControlTest.java
----------------------------------------------------------------------
diff --git a/tests/integration-tests/src/test/java/org/apache/activemq/artemis/tests/integration/jms/server/management/JMSServerControlTest.java b/tests/integration-tests/src/test/java/org/apache/activemq/artemis/tests/integration/jms/server/management/JMSServerControlTest.java
index b42bba9..11546fc 100644
--- a/tests/integration-tests/src/test/java/org/apache/activemq/artemis/tests/integration/jms/server/management/JMSServerControlTest.java
+++ b/tests/integration-tests/src/test/java/org/apache/activemq/artemis/tests/integration/jms/server/management/JMSServerControlTest.java
@@ -947,6 +947,12 @@ public class JMSServerControlTest extends ManagementTestBase {
       }
 
       Assert.assertTrue(failed);
+
+      // If these objects are not referenced at the end of the test, they can be destroyed by garbage collector
+      // during the test, what can lead to test failure.
+      connection.close();
+      connection2.close();
+      connection3.close();
    }
 
    // Package protected ---------------------------------------------