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 <clebertsuco...@apache.org> Authored: Tue Oct 31 08:54:23 2017 -0400 Committer: Justin Bertram <jbert...@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();