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 2017/10/31 13:33:58 UTC

[3/4] activemq-artemis git commit: ARTEMIS-1489 Adjusting timeouts on test

ARTEMIS-1489 Adjusting timeouts on test

The Critical analyzer is supposed to catch Deadlocks, and such
can only be resolved by killing the VM.

This test is using shutdown as it would be a bit more complex to
handle Byteman to then halt the VM. This will validate the
CriticalAnalyzer is capturing the event, and in such it would halt the VM.


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

Branch: refs/heads/master
Commit: f8a0eec2cfb997e3ecde3f46128a888e7370f968
Parents: 2bf690e
Author: Clebert Suconic <cl...@apache.org>
Authored: Tue Oct 31 08:54:23 2017 -0400
Committer: Justin Bertram <jb...@apache.org>
Committed: Tue Oct 31 08:33:44 2017 -0500

----------------------------------------------------------------------
 .../extras/byteman/CriticalAnalyzerFaultInjectionTest.java     | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/activemq-artemis/blob/f8a0eec2/tests/extra-tests/src/test/java/org/apache/activemq/artemis/tests/extras/byteman/CriticalAnalyzerFaultInjectionTest.java
----------------------------------------------------------------------
diff --git a/tests/extra-tests/src/test/java/org/apache/activemq/artemis/tests/extras/byteman/CriticalAnalyzerFaultInjectionTest.java b/tests/extra-tests/src/test/java/org/apache/activemq/artemis/tests/extras/byteman/CriticalAnalyzerFaultInjectionTest.java
index a22702b..785b9eb 100644
--- a/tests/extra-tests/src/test/java/org/apache/activemq/artemis/tests/extras/byteman/CriticalAnalyzerFaultInjectionTest.java
+++ b/tests/extra-tests/src/test/java/org/apache/activemq/artemis/tests/extras/byteman/CriticalAnalyzerFaultInjectionTest.java
@@ -17,6 +17,7 @@
 package org.apache.activemq.artemis.tests.extras.byteman;
 
 import javax.jms.DeliveryMode;
+import javax.jms.JMSException;
 import javax.jms.MessageProducer;
 import javax.jms.Queue;
 import javax.jms.Session;
@@ -41,8 +42,8 @@ public class CriticalAnalyzerFaultInjectionTest extends JMSTestBase {
 
    // Critical Analyzer Settings
    private static long CHECK_PERIOD = 100;
-   private static long TIMEOUT = 3000;
-   public static long TEST_TIMEOUT = 60000;
+   private static long TIMEOUT = 1000;
+   public static long TEST_TIMEOUT = 5000;
 
    private SimpleString address = SimpleString.toSimpleString("faultInjectionTestAddress");
 
@@ -95,6 +96,7 @@ public class CriticalAnalyzerFaultInjectionTest extends JMSTestBase {
          p.setDeliveryMode(DeliveryMode.PERSISTENT);
          conn.start();
          p.send(s.createTextMessage("payload"));
+      } catch (JMSException expected) {
       } finally {
          if (conn != null) {
             conn.close();