activemq-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From rob...@apache.org
Subject [activemq-artemis] branch main updated: ARTEMIS-3341 update checkstyle to latest + update code
Date Fri, 11 Jun 2021 11:16:54 GMT
This is an automated email from the ASF dual-hosted git repository.

robbie pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/activemq-artemis.git


The following commit(s) were added to refs/heads/main by this push:
     new a4c1155  ARTEMIS-3341 update checkstyle to latest + update code
a4c1155 is described below

commit a4c1155a013b015f53faca9f3558a081a1707491
Author: Justin Bertram <jbertram@apache.org>
AuthorDate: Thu Jun 10 22:56:40 2021 -0500

    ARTEMIS-3341 update checkstyle to latest + update code
    
    As a follow-up to #3618/dc7de893747b90b627d729f9f18a758bb4dad9d5 update
    checkstyle to the latest version, restoring the originally intended
    "RightCurly" style, and updating all the code to properly adhere to the
    style as enforced by the new checkstyle version.
    
    The version of checkstyle we used before the aforementioned commit had
    a bug which didn't properly enforced our intended "RightCurly" style
    (see https://github.com/checkstyle/checkstyle/issues/6345). That commit
    changed the style to accommodate the handful of unintended style
    violations. This commit reverts that change for 2 main reasons:
     - The style was always intended to use `alone` for both `METHOD_DEF`
    and `CTOR_DEF`.
     - There are over 1,000 existing uses of the intended style and around
    30 violations of this style which were unintentionally allowed.
    
    Reverting the style back to the original and cleaning up the unintented
    violations makes the code more consistent and prevents further style
    inconsistencies in the future.
    
    There were a handful of other changes related to checkstyle bugs which
    allowed unintended style violations. These were related to indentation
    levels.
    
    This closes #3619
    (with some minor changes from Robbie to fix remaining violations)
---
 .../activemq/artemis/utils/TypedProperties.java    |  3 +-
 .../jdbc/store/drivers/AbstractJDBCDriver.java     |  3 +-
 .../protocol/amqp/proton/handler/EventHandler.java | 87 ++++++++++++++-------
 .../core/server/impl/PostOfficeJournalLoader.java  |  2 +-
 .../core/server/transformer/Transformer.java       |  3 +-
 .../core/server/impl/jdbc/TestJDBCDriver.java      |  3 +-
 .../artemis/tests/util/NoProcessFilesBehind.java   |  2 +-
 etc/checkstyle.xml                                 |  6 +-
 pom.xml                                            |  2 +-
 .../extras/byteman/ActiveMQMessageHandlerTest.java | 26 ++++---
 .../ActiveMQServerControlMultiThreadTest.java      | 22 +++---
 .../tests/extras/byteman/BMFailoverTest.java       | 52 +++++++------
 .../byteman/ClusteredBridgeReconnectTest.java      | 37 ++++-----
 .../extras/byteman/ClusteredGroupingTest.java      | 90 ++++++++++++----------
 .../byteman/ConcurrentDeliveryCancelTest.java      | 13 ++--
 .../CriticalAnalyzerFaultInjectionTest.java        | 13 ++--
 .../byteman/DisconnectOnCriticalFailureTest.java   | 39 +++++-----
 .../tests/extras/byteman/FailureXATest.java        | 28 +++----
 .../tests/extras/byteman/FileLockMonitorTest.java  |  6 +-
 .../extras/byteman/FileLockNodeManagerTest.java    |  3 +-
 .../artemis/tests/extras/byteman/GroupingTest.java | 13 ++--
 .../byteman/HierarchicalObjectRepositoryTest.java  | 10 ++-
 .../byteman/InterruptedMessageHandlerTest.java     | 24 +++---
 .../extras/byteman/JMSBridgeReconnectionTest.java  | 24 +++---
 .../extras/byteman/JournalImplConcurrencyTest.java | 16 ++--
 .../extras/byteman/LargeMessageOnShutdownTest.java |  6 +-
 .../byteman/LargeMessageOverReplicationTest.java   | 26 ++++---
 .../byteman/LargeMessageReplicationTest.java       | 37 +++++----
 .../artemis/tests/extras/byteman/LatencyTest.java  | 24 +++---
 .../LoggingActiveMQServerPluginAMQPTest.java       | 34 ++++----
 .../LoggingActiveMQServerPluginOpenWireTest.java   | 34 ++++----
 .../byteman/LoggingActiveMQServerPluginTest.java   | 88 +++++++++++----------
 .../byteman/ManagementExceptionHandlingTest.java   | 28 +++----
 .../tests/extras/byteman/OrphanedConsumerTest.java | 56 +++++++-------
 .../tests/extras/byteman/PagingLeakTest.java       | 24 +++---
 .../tests/extras/byteman/PagingOMETest.java        | 13 ++--
 .../extras/byteman/QueueDeploymentFailedTest.java  |  1 +
 .../RaceOnClosingConsumerWhileReconnecting.java    | 39 +++++-----
 .../extras/byteman/RaceOnCursorIteratorTest.java   | 26 ++++---
 ...RaceOnSyncLargeMessageOverReplication2Test.java | 24 +++---
 .../RaceOnSyncLargeMessageOverReplicationTest.java | 13 ++--
 .../extras/byteman/ReplicationBackupTest.java      | 13 ++--
 .../extras/byteman/StartStopDeadlockTest.java      | 37 +++++----
 .../extras/byteman/StompInternalStateTest.java     | 15 ++--
 .../tests/extras/byteman/TimeoutXATest.java        | 28 +++----
 .../analyzer/FileSystemSyncBlockedTest.java        | 20 +++--
 .../integration/amqp/SaslKrb5LDAPSecurityTest.java |  4 +-
 .../integration/cluster/bridge/BridgeTest.java     |  2 +-
 48 files changed, 614 insertions(+), 505 deletions(-)

diff --git a/artemis-commons/src/main/java/org/apache/activemq/artemis/utils/TypedProperties.java b/artemis-commons/src/main/java/org/apache/activemq/artemis/utils/TypedProperties.java
index 05b401b..5948d5a 100644
--- a/artemis-commons/src/main/java/org/apache/activemq/artemis/utils/TypedProperties.java
+++ b/artemis-commons/src/main/java/org/apache/activemq/artemis/utils/TypedProperties.java
@@ -23,7 +23,8 @@ package org.apache.activemq.artemis.utils;
 @Deprecated
 public class TypedProperties extends org.apache.activemq.artemis.utils.collections.TypedProperties {
 
-   public TypedProperties() { }
+   public TypedProperties() {
+   }
 
    public TypedProperties(final org.apache.activemq.artemis.utils.collections.TypedProperties other) {
       super(other);
diff --git a/artemis-jdbc-store/src/main/java/org/apache/activemq/artemis/jdbc/store/drivers/AbstractJDBCDriver.java b/artemis-jdbc-store/src/main/java/org/apache/activemq/artemis/jdbc/store/drivers/AbstractJDBCDriver.java
index 77f9ee5..24b622f 100644
--- a/artemis-jdbc-store/src/main/java/org/apache/activemq/artemis/jdbc/store/drivers/AbstractJDBCDriver.java
+++ b/artemis-jdbc-store/src/main/java/org/apache/activemq/artemis/jdbc/store/drivers/AbstractJDBCDriver.java
@@ -39,7 +39,8 @@ public abstract class AbstractJDBCDriver {
 
    protected JDBCConnectionProvider connectionProvider;
 
-   public AbstractJDBCDriver() { }
+   public AbstractJDBCDriver() {
+   }
 
    public AbstractJDBCDriver(JDBCConnectionProvider connectionProvider, SQLProvider provider) {
       this.connectionProvider = connectionProvider;
diff --git a/artemis-protocols/artemis-amqp-protocol/src/main/java/org/apache/activemq/artemis/protocol/amqp/proton/handler/EventHandler.java b/artemis-protocols/artemis-amqp-protocol/src/main/java/org/apache/activemq/artemis/protocol/amqp/proton/handler/EventHandler.java
index 717c6cd..1f2877a 100644
--- a/artemis-protocols/artemis-amqp-protocol/src/main/java/org/apache/activemq/artemis/protocol/amqp/proton/handler/EventHandler.java
+++ b/artemis-protocols/artemis-amqp-protocol/src/main/java/org/apache/activemq/artemis/protocol/amqp/proton/handler/EventHandler.java
@@ -29,63 +29,92 @@ import org.apache.qpid.proton.engine.Transport;
  */
 public interface EventHandler {
 
-   default void onAuthInit(ProtonHandler handler, Connection connection, boolean sasl) { }
+   default void onAuthInit(ProtonHandler handler, Connection connection, boolean sasl) {
+   }
 
-   default void onSaslRemoteMechanismChosen(ProtonHandler handler, String mech) { }
+   default void onSaslRemoteMechanismChosen(ProtonHandler handler, String mech) {
+   }
 
-   default void onAuthFailed(ProtonHandler protonHandler, Connection connection) { }
+   default void onAuthFailed(ProtonHandler protonHandler, Connection connection) {
+   }
 
-   default void onAuthSuccess(ProtonHandler protonHandler, Connection connection) { }
+   default void onAuthSuccess(ProtonHandler protonHandler, Connection connection) {
+   }
 
-   default void onSaslMechanismsOffered(ProtonHandler handler, String[] mechanisms) { }
+   default void onSaslMechanismsOffered(ProtonHandler handler, String[] mechanisms) {
+   }
 
-   default void onInit(Connection connection) throws Exception { }
+   default void onInit(Connection connection) throws Exception {
+   }
 
-   default void onLocalOpen(Connection connection) throws Exception { }
+   default void onLocalOpen(Connection connection) throws Exception {
+   }
 
-   default void onRemoteOpen(Connection connection) throws Exception { }
+   default void onRemoteOpen(Connection connection) throws Exception {
+   }
 
-   default void onLocalClose(Connection connection) throws Exception { }
+   default void onLocalClose(Connection connection) throws Exception {
+   }
 
-   default void onRemoteClose(Connection connection) throws Exception { }
+   default void onRemoteClose(Connection connection) throws Exception {
+   }
 
-   default void onFinal(Connection connection) throws Exception { }
+   default void onFinal(Connection connection) throws Exception {
+   }
 
-   default void onInit(Session session) throws Exception { }
+   default void onInit(Session session) throws Exception {
+   }
 
-   default void onLocalOpen(Session session) throws Exception { }
+   default void onLocalOpen(Session session) throws Exception {
+   }
 
-   default void onRemoteOpen(Session session) throws Exception { }
+   default void onRemoteOpen(Session session) throws Exception {
+   }
 
-   default void onLocalClose(Session session) throws Exception { }
+   default void onLocalClose(Session session) throws Exception {
+   }
 
-   default void onRemoteClose(Session session) throws Exception { }
+   default void onRemoteClose(Session session) throws Exception {
+   }
 
-   default void onFinal(Session session) throws Exception { }
+   default void onFinal(Session session) throws Exception {
+   }
 
-   default void onInit(Link link) throws Exception { }
+   default void onInit(Link link) throws Exception {
+   }
 
-   default void onLocalOpen(Link link) throws Exception { }
+   default void onLocalOpen(Link link) throws Exception {
+   }
 
-   default void onRemoteOpen(Link link) throws Exception { }
+   default void onRemoteOpen(Link link) throws Exception {
+   }
 
-   default void onLocalClose(Link link) throws Exception { }
+   default void onLocalClose(Link link) throws Exception {
+   }
 
-   default void onRemoteClose(Link link) throws Exception { }
+   default void onRemoteClose(Link link) throws Exception {
+   }
 
-   default void onFlow(Link link) throws Exception { }
+   default void onFlow(Link link) throws Exception {
+   }
 
-   default void onFinal(Link link) throws Exception { }
+   default void onFinal(Link link) throws Exception {
+   }
 
-   default void onRemoteDetach(Link link) throws Exception { }
+   default void onRemoteDetach(Link link) throws Exception {
+   }
 
-   default void onLocalDetach(Link link) throws Exception { }
+   default void onLocalDetach(Link link) throws Exception {
+   }
 
-   default void onDelivery(Delivery delivery) throws Exception { }
+   default void onDelivery(Delivery delivery) throws Exception {
+   }
 
-   default void onTransport(Transport transport) throws Exception { }
+   default void onTransport(Transport transport) throws Exception {
+   }
 
-   default void pushBytes(ByteBuf bytes) { }
+   default void pushBytes(ByteBuf bytes) {
+   }
 
    default boolean flowControl(ReadyListener readyListener) {
       return true;
diff --git a/artemis-server/src/main/java/org/apache/activemq/artemis/core/server/impl/PostOfficeJournalLoader.java b/artemis-server/src/main/java/org/apache/activemq/artemis/core/server/impl/PostOfficeJournalLoader.java
index 4c5865d..7701bbb 100644
--- a/artemis-server/src/main/java/org/apache/activemq/artemis/core/server/impl/PostOfficeJournalLoader.java
+++ b/artemis-server/src/main/java/org/apache/activemq/artemis/core/server/impl/PostOfficeJournalLoader.java
@@ -167,7 +167,7 @@ public class PostOfficeJournalLoader implements JournalLoader {
          if (queueBindingInfo.getQueueStatusEncodings() != null) {
             for (QueueStatusEncoding encoding : queueBindingInfo.getQueueStatusEncodings()) {
                if (encoding.getStatus() == AddressQueueStatus.PAUSED)
-               queue.reloadPause(encoding.getId());
+                  queue.reloadPause(encoding.getId());
             }
          }
 
diff --git a/artemis-server/src/main/java/org/apache/activemq/artemis/core/server/transformer/Transformer.java b/artemis-server/src/main/java/org/apache/activemq/artemis/core/server/transformer/Transformer.java
index 4301740..d275b3c 100644
--- a/artemis-server/src/main/java/org/apache/activemq/artemis/core/server/transformer/Transformer.java
+++ b/artemis-server/src/main/java/org/apache/activemq/artemis/core/server/transformer/Transformer.java
@@ -22,7 +22,8 @@ import org.apache.activemq.artemis.api.core.Message;
 
 public interface Transformer {
 
-   default void init(Map<String, String> properties) { }
+   default void init(Map<String, String> properties) {
+   }
 
    Message transform(Message message);
 }
diff --git a/artemis-server/src/test/java/org/apache/activemq/artemis/core/server/impl/jdbc/TestJDBCDriver.java b/artemis-server/src/test/java/org/apache/activemq/artemis/core/server/impl/jdbc/TestJDBCDriver.java
index 061a5d5..f660bea 100644
--- a/artemis-server/src/test/java/org/apache/activemq/artemis/core/server/impl/jdbc/TestJDBCDriver.java
+++ b/artemis-server/src/test/java/org/apache/activemq/artemis/core/server/impl/jdbc/TestJDBCDriver.java
@@ -48,7 +48,8 @@ public class TestJDBCDriver extends AbstractJDBCDriver {
    }
 
    @Override
-   protected void prepareStatements() { }
+   protected void prepareStatements() {
+   }
 
    @Override
    protected void createSchema() {
diff --git a/artemis-server/src/test/java/org/apache/activemq/artemis/tests/util/NoProcessFilesBehind.java b/artemis-server/src/test/java/org/apache/activemq/artemis/tests/util/NoProcessFilesBehind.java
index 527ffb2..f9e0ddb 100644
--- a/artemis-server/src/test/java/org/apache/activemq/artemis/tests/util/NoProcessFilesBehind.java
+++ b/artemis-server/src/test/java/org/apache/activemq/artemis/tests/util/NoProcessFilesBehind.java
@@ -93,7 +93,7 @@ public class NoProcessFilesBehind extends TestWatcher {
             processInput.readLine();
             while ((outputLine = processInput.readLine()) != null) {
                if (!filtered || (!outputLine.endsWith(".jar") && !outputLine.endsWith(".so") && !outputLine.contains("type=STREAM")))
-               openFiles.add(outputLine);
+                  openFiles.add(outputLine);
             }
          }
       } catch (Exception ignore) {
diff --git a/etc/checkstyle.xml b/etc/checkstyle.xml
index ede3c5a..c1c6aec 100644
--- a/etc/checkstyle.xml
+++ b/etc/checkstyle.xml
@@ -69,11 +69,7 @@ under the License.
       <module name="RightCurly"/>
       <module name="RightCurly">
          <property name="option" value="alone"/>
-         <property name="tokens" value="CLASS_DEF, LITERAL_FOR, LITERAL_WHILE, LITERAL_DO, STATIC_INIT, INSTANCE_INIT"/>
-      </module>
-      <module name="RightCurly">
-         <property name="option" value="alone_or_singleline"/>
-         <property name="tokens" value="METHOD_DEF, CTOR_DEF"/>
+         <property name="tokens" value="CLASS_DEF, METHOD_DEF, CTOR_DEF, LITERAL_FOR, LITERAL_WHILE, LITERAL_DO, STATIC_INIT, INSTANCE_INIT"/>
       </module>
       <!-- Checks that there is no whitespace after certain tokens; e.g. "." and "!". -->
       <module name="NoWhitespaceAfter"/>
diff --git a/pom.xml b/pom.xml
index b460f96..9b6847d 100644
--- a/pom.xml
+++ b/pom.xml
@@ -101,7 +101,7 @@
       <maven.bundle.plugin.version>5.1.2</maven.bundle.plugin.version>
       <maven.checkstyle.plugin.version>3.1.1</maven.checkstyle.plugin.version>
       <sevntu.checks.version>1.39.0</sevntu.checks.version>
-      <checkstyle.version>8.29</checkstyle.version>
+      <checkstyle.version>8.43</checkstyle.version>
       <mockito.version>3.9.0</mockito.version>
       <jctools.version>2.1.2</jctools.version>
       <netty.version>4.1.65.Final</netty.version>
diff --git a/tests/extra-tests/src/test/java/org/apache/activemq/artemis/tests/extras/byteman/ActiveMQMessageHandlerTest.java b/tests/extra-tests/src/test/java/org/apache/activemq/artemis/tests/extras/byteman/ActiveMQMessageHandlerTest.java
index af6ed08..a080f8b 100644
--- a/tests/extra-tests/src/test/java/org/apache/activemq/artemis/tests/extras/byteman/ActiveMQMessageHandlerTest.java
+++ b/tests/extra-tests/src/test/java/org/apache/activemq/artemis/tests/extras/byteman/ActiveMQMessageHandlerTest.java
@@ -65,12 +65,13 @@ public class ActiveMQMessageHandlerTest extends ActiveMQRATestBase {
 
    @Test
    @BMRules(
-      rules = {@BMRule(
-         name = "interrupt",
-         targetClass = "org.apache.activemq.artemis.core.protocol.core.impl.ActiveMQSessionContext",
-         targetMethod = "xaEnd",
-         targetLocation = "ENTRY",
-         action = "org.apache.activemq.artemis.tests.extras.byteman.ActiveMQMessageHandlerTest.interrupt();")})
+      rules = {
+         @BMRule(
+            name = "interrupt",
+            targetClass = "org.apache.activemq.artemis.core.protocol.core.impl.ActiveMQSessionContext",
+            targetMethod = "xaEnd",
+            targetLocation = "ENTRY",
+            action = "org.apache.activemq.artemis.tests.extras.byteman.ActiveMQMessageHandlerTest.interrupt();")})
    public void testSimpleMessageReceivedOnQueue() throws Exception {
       ActiveMQResourceAdapter qResourceAdapter = newResourceAdapter();
       resourceAdapter = qResourceAdapter;
@@ -134,12 +135,13 @@ public class ActiveMQMessageHandlerTest extends ActiveMQRATestBase {
 
    @Test
    @BMRules(
-      rules = {@BMRule(
-         name = "interrupt",
-         targetClass = "org.apache.activemq.artemis.core.protocol.core.impl.ActiveMQSessionContext",
-         targetMethod = "xaEnd",
-         targetLocation = "ENTRY",
-         action = "org.apache.activemq.artemis.tests.extras.byteman.ActiveMQMessageHandlerTest.interrupt();")})
+      rules = {
+         @BMRule(
+            name = "interrupt",
+            targetClass = "org.apache.activemq.artemis.core.protocol.core.impl.ActiveMQSessionContext",
+            targetMethod = "xaEnd",
+            targetLocation = "ENTRY",
+            action = "org.apache.activemq.artemis.tests.extras.byteman.ActiveMQMessageHandlerTest.interrupt();")})
    public void testSimpleMessageReceivedOnQueueTwoPhase() throws Exception {
       ActiveMQResourceAdapter qResourceAdapter = newResourceAdapter();
       resourceAdapter = qResourceAdapter;
diff --git a/tests/extra-tests/src/test/java/org/apache/activemq/artemis/tests/extras/byteman/ActiveMQServerControlMultiThreadTest.java b/tests/extra-tests/src/test/java/org/apache/activemq/artemis/tests/extras/byteman/ActiveMQServerControlMultiThreadTest.java
index 7ac0949..081ab88 100644
--- a/tests/extra-tests/src/test/java/org/apache/activemq/artemis/tests/extras/byteman/ActiveMQServerControlMultiThreadTest.java
+++ b/tests/extra-tests/src/test/java/org/apache/activemq/artemis/tests/extras/byteman/ActiveMQServerControlMultiThreadTest.java
@@ -64,11 +64,12 @@ public class ActiveMQServerControlMultiThreadTest extends ManagementTestBase {
     */
 
    @Test
-   @BMRules(rules = {@BMRule(name = "Delay listAddress() by 2 secs ",
-      targetClass = "org.apache.activemq.artemis.core.postoffice.impl.PostOfficeImpl",
-      targetMethod = "getAddressInfo(org.apache.activemq.artemis.api.core.SimpleString)",
-      targetLocation = "ENTRY",
-      action = "org.apache.activemq.artemis.tests.extras.byteman.ActiveMQServerControlMultiThreadTest.delay(2)")})
+   @BMRules(rules = {
+      @BMRule(name = "Delay listAddress() by 2 secs ",
+         targetClass = "org.apache.activemq.artemis.core.postoffice.impl.PostOfficeImpl",
+         targetMethod = "getAddressInfo(org.apache.activemq.artemis.api.core.SimpleString)",
+         targetLocation = "ENTRY",
+         action = "org.apache.activemq.artemis.tests.extras.byteman.ActiveMQServerControlMultiThreadTest.delay(2)")})
 
    public void listAddressDuringDeleteAddress() throws Exception {
 
@@ -125,11 +126,12 @@ public class ActiveMQServerControlMultiThreadTest extends ManagementTestBase {
     */
 
    @Test
-   @BMRules(rules = {@BMRule(name = "Delay listConsumers() by 2 secs ",
-      targetClass = "org.apache.activemq.artemis.core.management.impl.view.ConsumerView",
-      targetMethod = "toJson(org.apache.activemq.artemis.core.server.ServerConsumer)",
-      targetLocation = "ENTRY",
-      action = "org.apache.activemq.artemis.tests.extras.byteman.ActiveMQServerControlMultiThreadTest.delay(2)")})
+   @BMRules(rules = {
+      @BMRule(name = "Delay listConsumers() by 2 secs ",
+         targetClass = "org.apache.activemq.artemis.core.management.impl.view.ConsumerView",
+         targetMethod = "toJson(org.apache.activemq.artemis.core.server.ServerConsumer)",
+         targetLocation = "ENTRY",
+         action = "org.apache.activemq.artemis.tests.extras.byteman.ActiveMQServerControlMultiThreadTest.delay(2)")})
 
    public void listConsumersDuringSessionClose() throws Exception {
 
diff --git a/tests/extra-tests/src/test/java/org/apache/activemq/artemis/tests/extras/byteman/BMFailoverTest.java b/tests/extra-tests/src/test/java/org/apache/activemq/artemis/tests/extras/byteman/BMFailoverTest.java
index 3e97dfc..7739df7 100644
--- a/tests/extra-tests/src/test/java/org/apache/activemq/artemis/tests/extras/byteman/BMFailoverTest.java
+++ b/tests/extra-tests/src/test/java/org/apache/activemq/artemis/tests/extras/byteman/BMFailoverTest.java
@@ -90,12 +90,13 @@ public class BMFailoverTest extends FailoverTestBase {
 
    @Test
    @BMRules(
-      rules = {@BMRule(
-         name = "trace ActiveMQSessionContext xaEnd",
-         targetClass = "org.apache.activemq.artemis.core.protocol.core.impl.ActiveMQSessionContext",
-         targetMethod = "xaEnd",
-         targetLocation = "AT EXIT",
-         action = "org.apache.activemq.artemis.tests.extras.byteman.BMFailoverTest.stopAndThrow()")})
+      rules = {
+         @BMRule(
+            name = "trace ActiveMQSessionContext xaEnd",
+            targetClass = "org.apache.activemq.artemis.core.protocol.core.impl.ActiveMQSessionContext",
+            targetMethod = "xaEnd",
+            targetLocation = "AT EXIT",
+            action = "org.apache.activemq.artemis.tests.extras.byteman.BMFailoverTest.stopAndThrow()")})
    //https://bugzilla.redhat.com/show_bug.cgi?id=1152410
    public void testFailOnEndAndRetry() throws Exception {
       serverToStop = liveServer;
@@ -161,12 +162,13 @@ public class BMFailoverTest extends FailoverTestBase {
 
    @Test
    @BMRules(
-      rules = {@BMRule(
-         name = "trace clientsessionimpl commit",
-         targetClass = "org.apache.activemq.artemis.core.client.impl.ClientSessionImpl",
-         targetMethod = "start(javax.transaction.xa.Xid, int)",
-         targetLocation = "AT EXIT",
-         action = "org.apache.activemq.artemis.tests.extras.byteman.BMFailoverTest.serverToStop.getServer().stop(true)")})
+      rules = {
+         @BMRule(
+            name = "trace clientsessionimpl commit",
+            targetClass = "org.apache.activemq.artemis.core.client.impl.ClientSessionImpl",
+            targetMethod = "start(javax.transaction.xa.Xid, int)",
+            targetLocation = "AT EXIT",
+            action = "org.apache.activemq.artemis.tests.extras.byteman.BMFailoverTest.serverToStop.getServer().stop(true)")})
    public void testFailoverOnCommit2() throws Exception {
       serverToStop = liveServer;
       locator = getServerLocator();
@@ -248,12 +250,13 @@ public class BMFailoverTest extends FailoverTestBase {
 
    @Test
    @BMRules(
-      rules = {@BMRule(
-         name = "trace clientsessionimpl commit",
-         targetClass = "org.apache.activemq.artemis.core.client.impl.ClientSessionImpl",
-         targetMethod = "commit",
-         targetLocation = "ENTRY",
-         action = "org.apache.activemq.artemis.tests.extras.byteman.BMFailoverTest.serverToStop.getServer().stop(true)")})
+      rules = {
+         @BMRule(
+            name = "trace clientsessionimpl commit",
+            targetClass = "org.apache.activemq.artemis.core.client.impl.ClientSessionImpl",
+            targetMethod = "commit",
+            targetLocation = "ENTRY",
+            action = "org.apache.activemq.artemis.tests.extras.byteman.BMFailoverTest.serverToStop.getServer().stop(true)")})
    public void testFailoverOnCommit() throws Exception {
       serverToStop = liveServer;
       locator = getServerLocator();
@@ -277,12 +280,13 @@ public class BMFailoverTest extends FailoverTestBase {
 
    @Test
    @BMRules(
-      rules = {@BMRule(
-         name = "trace clientsessionimpl commit",
-         targetClass = "org.apache.activemq.artemis.core.client.impl.ClientSessionImpl",
-         targetMethod = "commit",
-         targetLocation = "ENTRY",
-         action = "org.apache.activemq.artemis.tests.extras.byteman.BMFailoverTest.serverToStop.getServer().stop(true)")})
+      rules = {
+         @BMRule(
+            name = "trace clientsessionimpl commit",
+            targetClass = "org.apache.activemq.artemis.core.client.impl.ClientSessionImpl",
+            targetMethod = "commit",
+            targetLocation = "ENTRY",
+            action = "org.apache.activemq.artemis.tests.extras.byteman.BMFailoverTest.serverToStop.getServer().stop(true)")})
    public void testFailoverOnReceiveCommit() throws Exception {
       serverToStop = liveServer;
       locator = getServerLocator();
diff --git a/tests/extra-tests/src/test/java/org/apache/activemq/artemis/tests/extras/byteman/ClusteredBridgeReconnectTest.java b/tests/extra-tests/src/test/java/org/apache/activemq/artemis/tests/extras/byteman/ClusteredBridgeReconnectTest.java
index 6ec30fc..e2111e9 100644
--- a/tests/extra-tests/src/test/java/org/apache/activemq/artemis/tests/extras/byteman/ClusteredBridgeReconnectTest.java
+++ b/tests/extra-tests/src/test/java/org/apache/activemq/artemis/tests/extras/byteman/ClusteredBridgeReconnectTest.java
@@ -71,24 +71,25 @@ public class ClusteredBridgeReconnectTest extends ClusterTestBase {
 
    @Test
    @BMRules(
-      rules = {@BMRule(
-         name = "enter",
-         targetClass = "org.apache.activemq.artemis.core.server.cluster.impl.BridgeImpl",
-         targetMethod = "connect",
-         targetLocation = "ENTRY",
-         action = "org.apache.activemq.artemis.tests.extras.byteman.ClusteredBridgeReconnectTest.enterConnect();"), @BMRule(
-         name = "exit",
-         targetClass = "org.apache.activemq.artemis.core.server.cluster.impl.BridgeImpl",
-         targetMethod = "connect",
-         targetLocation = "EXIT",
-         action = "org.apache.activemq.artemis.tests.extras.byteman.ClusteredBridgeReconnectTest.exitConnect();"), @BMRule(
-         name = "send",
-         targetClass = "org.apache.activemq.artemis.core.protocol.core.impl.ChannelImpl",
-         targetMethod = "send(org.apache.activemq.artemis.core.protocol.core.Packet)",
-         targetLocation = "EXIT",
-         action = "org.apache.activemq.artemis.tests.extras.byteman.ClusteredBridgeReconnectTest.send();")
-
-      })
+      rules = {
+         @BMRule(
+            name = "enter",
+            targetClass = "org.apache.activemq.artemis.core.server.cluster.impl.BridgeImpl",
+            targetMethod = "connect",
+            targetLocation = "ENTRY",
+            action = "org.apache.activemq.artemis.tests.extras.byteman.ClusteredBridgeReconnectTest.enterConnect();"),
+         @BMRule(
+            name = "exit",
+            targetClass = "org.apache.activemq.artemis.core.server.cluster.impl.BridgeImpl",
+            targetMethod = "connect",
+            targetLocation = "EXIT",
+            action = "org.apache.activemq.artemis.tests.extras.byteman.ClusteredBridgeReconnectTest.exitConnect();"),
+         @BMRule(
+            name = "send",
+            targetClass = "org.apache.activemq.artemis.core.protocol.core.impl.ChannelImpl",
+            targetMethod = "send(org.apache.activemq.artemis.core.protocol.core.Packet)",
+            targetLocation = "EXIT",
+            action = "org.apache.activemq.artemis.tests.extras.byteman.ClusteredBridgeReconnectTest.send();")})
    public void testReconnectBridge() throws Exception {
       setupServer(0, isFileStorage(), isNetty());
       setupServer(1, isFileStorage(), isNetty());
diff --git a/tests/extra-tests/src/test/java/org/apache/activemq/artemis/tests/extras/byteman/ClusteredGroupingTest.java b/tests/extra-tests/src/test/java/org/apache/activemq/artemis/tests/extras/byteman/ClusteredGroupingTest.java
index 773f291..2ac7b92 100644
--- a/tests/extra-tests/src/test/java/org/apache/activemq/artemis/tests/extras/byteman/ClusteredGroupingTest.java
+++ b/tests/extra-tests/src/test/java/org/apache/activemq/artemis/tests/extras/byteman/ClusteredGroupingTest.java
@@ -41,17 +41,19 @@ public class ClusteredGroupingTest extends ClusterTestBase {
 
    @Test
    @BMRules(
-      rules = {@BMRule(
-         name = "blow-up",
-         targetClass = "org.apache.activemq.artemis.core.server.group.impl.LocalGroupingHandler",
-         targetMethod = "removeGrouping",
-         targetLocation = "ENTRY",
-         action = "org.apache.activemq.artemis.tests.extras.byteman.ClusteredGroupingTest.pause($1);"), @BMRule(
-         name = "blow-up2",
-         targetClass = "org.apache.activemq.artemis.core.server.group.impl.GroupHandlingAbstract",
-         targetMethod = "forceRemove",
-         targetLocation = "ENTRY",
-         action = "org.apache.activemq.artemis.tests.extras.byteman.ClusteredGroupingTest.restart2();")})
+      rules = {
+         @BMRule(
+            name = "blow-up",
+            targetClass = "org.apache.activemq.artemis.core.server.group.impl.LocalGroupingHandler",
+            targetMethod = "removeGrouping",
+            targetLocation = "ENTRY",
+            action = "org.apache.activemq.artemis.tests.extras.byteman.ClusteredGroupingTest.pause($1);"),
+         @BMRule(
+            name = "blow-up2",
+            targetClass = "org.apache.activemq.artemis.core.server.group.impl.GroupHandlingAbstract",
+            targetMethod = "forceRemove",
+            targetLocation = "ENTRY",
+            action = "org.apache.activemq.artemis.tests.extras.byteman.ClusteredGroupingTest.restart2();")})
    public void test2serversLocalGoesDown() throws Exception {
       setupServer(0, isFileStorage(), isNetty());
       setupServer(1, isFileStorage(), isNetty());
@@ -101,16 +103,18 @@ public class ClusteredGroupingTest extends ClusterTestBase {
 
    @Test
    @BMRules(
-      rules = {@BMRule(
-         name = "blow-up",
-         targetClass = "org.apache.activemq.artemis.core.server.group.impl.RemoteGroupingHandler",
-         targetMethod = "onNotification",
-         targetLocation = "ENTRY",
-         action = "org.apache.activemq.artemis.tests.extras.byteman.ClusteredGroupingTest.pause2($1);"), @BMRule(name = "blow-up2",
-         targetClass = "org.apache.activemq.artemis.core.server.group.impl.RemoteGroupingHandler",
-         targetMethod = "remove",
-         targetLocation = "ENTRY",
-         action = "org.apache.activemq.artemis.tests.extras.byteman.ClusteredGroupingTest.restart2();")})
+      rules = {
+         @BMRule(
+            name = "blow-up",
+            targetClass = "org.apache.activemq.artemis.core.server.group.impl.RemoteGroupingHandler",
+            targetMethod = "onNotification",
+            targetLocation = "ENTRY",
+            action = "org.apache.activemq.artemis.tests.extras.byteman.ClusteredGroupingTest.pause2($1);"),
+         @BMRule(name = "blow-up2",
+            targetClass = "org.apache.activemq.artemis.core.server.group.impl.RemoteGroupingHandler",
+            targetMethod = "remove",
+            targetLocation = "ENTRY",
+            action = "org.apache.activemq.artemis.tests.extras.byteman.ClusteredGroupingTest.restart2();")})
    public void test3serversLocalGoesDown() throws Exception {
       setupServer(0, isFileStorage(), isNetty());
       setupServer(1, isFileStorage(), isNetty());
@@ -171,16 +175,18 @@ public class ClusteredGroupingTest extends ClusterTestBase {
 
    @Test
    @BMRules(
-      rules = {@BMRule(
-         name = "blow-up",
-         targetClass = "org.apache.activemq.artemis.core.server.group.impl.LocalGroupingHandler",
-         targetMethod = "onNotification",
-         targetLocation = "ENTRY",
-         action = "org.apache.activemq.artemis.tests.extras.byteman.ClusteredGroupingTest.pause2($1);"), @BMRule(name = "blow-up2",
-         targetClass = "org.apache.activemq.artemis.core.server.group.impl.LocalGroupingHandler",
-         targetMethod = "remove",
-         targetLocation = "ENTRY",
-         action = "org.apache.activemq.artemis.tests.extras.byteman.ClusteredGroupingTest.restart2();")})
+      rules = {
+         @BMRule(
+            name = "blow-up",
+            targetClass = "org.apache.activemq.artemis.core.server.group.impl.LocalGroupingHandler",
+            targetMethod = "onNotification",
+            targetLocation = "ENTRY",
+            action = "org.apache.activemq.artemis.tests.extras.byteman.ClusteredGroupingTest.pause2($1);"),
+         @BMRule(name = "blow-up2",
+            targetClass = "org.apache.activemq.artemis.core.server.group.impl.LocalGroupingHandler",
+            targetMethod = "remove",
+            targetLocation = "ENTRY",
+            action = "org.apache.activemq.artemis.tests.extras.byteman.ClusteredGroupingTest.restart2();")})
    public void testLocal3serversLocalGoesDown() throws Exception {
       setupServer(0, isFileStorage(), isNetty());
       setupServer(1, isFileStorage(), isNetty());
@@ -241,16 +247,18 @@ public class ClusteredGroupingTest extends ClusterTestBase {
 
    @Test
    @BMRules(
-      rules = {@BMRule(
-         name = "blow-up",
-         targetClass = "org.apache.activemq.artemis.core.server.group.impl.LocalGroupingHandler",
-         targetMethod = "onNotification",
-         targetLocation = "ENTRY",
-         action = "org.apache.activemq.artemis.tests.extras.byteman.ClusteredGroupingTest.pause2($1);"), @BMRule(name = "blow-up2",
-         targetClass = "org.apache.activemq.artemis.core.server.group.impl.LocalGroupingHandler",
-         targetMethod = "remove",
-         targetLocation = "ENTRY",
-         action = "org.apache.activemq.artemis.tests.extras.byteman.ClusteredGroupingTest.restart2();")})
+      rules = {
+         @BMRule(
+            name = "blow-up",
+            targetClass = "org.apache.activemq.artemis.core.server.group.impl.LocalGroupingHandler",
+            targetMethod = "onNotification",
+            targetLocation = "ENTRY",
+            action = "org.apache.activemq.artemis.tests.extras.byteman.ClusteredGroupingTest.pause2($1);"),
+         @BMRule(name = "blow-up2",
+            targetClass = "org.apache.activemq.artemis.core.server.group.impl.LocalGroupingHandler",
+            targetMethod = "remove",
+            targetLocation = "ENTRY",
+            action = "org.apache.activemq.artemis.tests.extras.byteman.ClusteredGroupingTest.restart2();")})
    public void testLocal4serversLocalGoesDown() throws Exception {
       setupServer(0, isFileStorage(), isNetty());
       setupServer(1, isFileStorage(), isNetty());
diff --git a/tests/extra-tests/src/test/java/org/apache/activemq/artemis/tests/extras/byteman/ConcurrentDeliveryCancelTest.java b/tests/extra-tests/src/test/java/org/apache/activemq/artemis/tests/extras/byteman/ConcurrentDeliveryCancelTest.java
index 548e0a6..1ae3db5 100644
--- a/tests/extra-tests/src/test/java/org/apache/activemq/artemis/tests/extras/byteman/ConcurrentDeliveryCancelTest.java
+++ b/tests/extra-tests/src/test/java/org/apache/activemq/artemis/tests/extras/byteman/ConcurrentDeliveryCancelTest.java
@@ -83,12 +83,13 @@ public class ConcurrentDeliveryCancelTest extends JMSTestBase {
 
    @Test
    @BMRules(
-      rules = {@BMRule(
-         name = "enterCancel-holdThere",
-         targetClass = "org.apache.activemq.artemis.core.server.impl.ServerConsumerImpl",
-         targetMethod = "close",
-         targetLocation = "ENTRY",
-         action = "org.apache.activemq.artemis.tests.extras.byteman.ConcurrentDeliveryCancelTest.enterCancel();")})
+      rules = {
+         @BMRule(
+            name = "enterCancel-holdThere",
+            targetClass = "org.apache.activemq.artemis.core.server.impl.ServerConsumerImpl",
+            targetMethod = "close",
+            targetLocation = "ENTRY",
+            action = "org.apache.activemq.artemis.tests.extras.byteman.ConcurrentDeliveryCancelTest.enterCancel();")})
    public void testConcurrentCancels() throws Exception {
 
       log.debug(server.getConfiguration().getJournalLocation().toString());
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 77bfe02..1dd3608 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
@@ -76,12 +76,13 @@ public class CriticalAnalyzerFaultInjectionTest extends JMSTestBase {
    }
 
    @BMRules(
-      rules = {@BMRule(
-         name = "Sync file data hangs",
-         targetClass = "org.apache.activemq.artemis.core.io.nio.NIOSequentialFile",
-         targetMethod = "sync",
-         targetLocation = "ENTRY",
-         action = "org.apache.activemq.artemis.tests.extras.byteman.CriticalAnalyzerFaultInjectionTest.methodHang();")})
+      rules = {
+         @BMRule(
+            name = "Sync file data hangs",
+            targetClass = "org.apache.activemq.artemis.core.io.nio.NIOSequentialFile",
+            targetMethod = "sync",
+            targetLocation = "ENTRY",
+            action = "org.apache.activemq.artemis.tests.extras.byteman.CriticalAnalyzerFaultInjectionTest.methodHang();")})
    @Test(timeout = 60000)
    public void testSlowDiskSync() throws Exception {
       sendConsumeDurableMessage();
diff --git a/tests/extra-tests/src/test/java/org/apache/activemq/artemis/tests/extras/byteman/DisconnectOnCriticalFailureTest.java b/tests/extra-tests/src/test/java/org/apache/activemq/artemis/tests/extras/byteman/DisconnectOnCriticalFailureTest.java
index 2be7d67..f1bbb21 100644
--- a/tests/extra-tests/src/test/java/org/apache/activemq/artemis/tests/extras/byteman/DisconnectOnCriticalFailureTest.java
+++ b/tests/extra-tests/src/test/java/org/apache/activemq/artemis/tests/extras/byteman/DisconnectOnCriticalFailureTest.java
@@ -56,12 +56,13 @@ public class DisconnectOnCriticalFailureTest extends JMSTestBase {
 
    @Test
    @BMRules(
-      rules = {@BMRule(
-         name = "Corrupt Decoding",
-         targetClass = "org.apache.activemq.artemis.core.protocol.core.impl.PacketDecoder",
-         targetMethod = "decode(byte)",
-         targetLocation = "ENTRY",
-         action = "org.apache.activemq.artemis.tests.extras.byteman.DisconnectOnCriticalFailureTest.doThrow();")})
+      rules = {
+         @BMRule(
+            name = "Corrupt Decoding",
+            targetClass = "org.apache.activemq.artemis.core.protocol.core.impl.PacketDecoder",
+            targetMethod = "decode(byte)",
+            targetLocation = "ENTRY",
+            action = "org.apache.activemq.artemis.tests.extras.byteman.DisconnectOnCriticalFailureTest.doThrow();")})
    public void testSendDisconnect() throws Exception {
       createQueue("queue1");
       final Connection producerConnection = nettyCf.createConnection();
@@ -90,12 +91,13 @@ public class DisconnectOnCriticalFailureTest extends JMSTestBase {
 
    @Test
    @BMRules(
-      rules = {@BMRule(
-         name = "Corrupt Decoding",
-         targetClass = "org.apache.activemq.artemis.core.protocol.ClientPacketDecoder",
-         targetMethod = "decode(org.apache.activemq.artemis.api.core.ActiveMQBuffer)",
-         targetLocation = "ENTRY",
-         action = "org.apache.activemq.artemis.tests.extras.byteman.DisconnectOnCriticalFailureTest.doThrow($1);")})
+      rules = {
+         @BMRule(
+            name = "Corrupt Decoding",
+            targetClass = "org.apache.activemq.artemis.core.protocol.ClientPacketDecoder",
+            targetMethod = "decode(org.apache.activemq.artemis.api.core.ActiveMQBuffer)",
+            targetLocation = "ENTRY",
+            action = "org.apache.activemq.artemis.tests.extras.byteman.DisconnectOnCriticalFailureTest.doThrow($1);")})
    public void testClientDisconnect() throws Exception {
       Queue q1 = createQueue("queue1");
       final Connection connection = nettyCf.createConnection();
@@ -132,12 +134,13 @@ public class DisconnectOnCriticalFailureTest extends JMSTestBase {
 
    @Test(timeout = 60000)
    @BMRules(
-      rules = {@BMRule(
-         name = "Corrupt Decoding",
-         targetClass = "org.apache.activemq.artemis.core.protocol.ClientPacketDecoder",
-         targetMethod = "decode(org.apache.activemq.artemis.api.core.ActiveMQBuffer)",
-         targetLocation = "ENTRY",
-         action = "org.apache.activemq.artemis.tests.extras.byteman.DisconnectOnCriticalFailureTest.doThrow($1);")})
+      rules = {
+         @BMRule(
+            name = "Corrupt Decoding",
+            targetClass = "org.apache.activemq.artemis.core.protocol.ClientPacketDecoder",
+            targetMethod = "decode(org.apache.activemq.artemis.api.core.ActiveMQBuffer)",
+            targetLocation = "ENTRY",
+            action = "org.apache.activemq.artemis.tests.extras.byteman.DisconnectOnCriticalFailureTest.doThrow($1);")})
    public void testClientDisconnectLarge() throws Exception {
       Queue q1 = createQueue("queue1");
       final Connection connection = nettyCf.createConnection();
diff --git a/tests/extra-tests/src/test/java/org/apache/activemq/artemis/tests/extras/byteman/FailureXATest.java b/tests/extra-tests/src/test/java/org/apache/activemq/artemis/tests/extras/byteman/FailureXATest.java
index 762cee9..97138be 100644
--- a/tests/extra-tests/src/test/java/org/apache/activemq/artemis/tests/extras/byteman/FailureXATest.java
+++ b/tests/extra-tests/src/test/java/org/apache/activemq/artemis/tests/extras/byteman/FailureXATest.java
@@ -63,25 +63,27 @@ public class FailureXATest extends ActiveMQTestBase {
 
    @Test
    @BMRules(
-      rules = {@BMRule(
-         name = "Crash after onephase committed",
-         targetClass = "org.apache.activemq.artemis.core.server.impl.ServerSessionImpl",
-         targetMethod = "xaCommit(javax.transaction.xa.Xid, boolean)",
-         targetLocation = "EXIT",
-         action = "throw new RuntimeException()")})
+      rules = {
+         @BMRule(
+            name = "Crash after onephase committed",
+            targetClass = "org.apache.activemq.artemis.core.server.impl.ServerSessionImpl",
+            targetMethod = "xaCommit(javax.transaction.xa.Xid, boolean)",
+            targetLocation = "EXIT",
+            action = "throw new RuntimeException()")})
    public void testCrashServerAfterOnePhaseCommit() throws Exception {
       doTestCrashServerAfterXACommit(true);
    }
 
    @Test
    @BMRules(
-      rules = {@BMRule(
-         name = "Crash after onephase committed",
-         targetClass = "org.apache.activemq.artemis.core.server.impl.ServerSessionImpl",
-         targetMethod = "xaCommit(javax.transaction.xa.Xid, boolean)",
-         targetLocation = "EXIT",
-         //helper = "org.apache.activemq.artemis.tests.extras.byteman.FailureXATest",
-         action = "throw new RuntimeException()")})
+      rules = {
+         @BMRule(
+            name = "Crash after onephase committed",
+            targetClass = "org.apache.activemq.artemis.core.server.impl.ServerSessionImpl",
+            targetMethod = "xaCommit(javax.transaction.xa.Xid, boolean)",
+            targetLocation = "EXIT",
+            //helper = "org.apache.activemq.artemis.tests.extras.byteman.FailureXATest",
+            action = "throw new RuntimeException()")})
    public void testCrashServerAfterTwoPhaseCommit() throws Exception {
       doTestCrashServerAfterXACommit(false);
    }
diff --git a/tests/extra-tests/src/test/java/org/apache/activemq/artemis/tests/extras/byteman/FileLockMonitorTest.java b/tests/extra-tests/src/test/java/org/apache/activemq/artemis/tests/extras/byteman/FileLockMonitorTest.java
index 2aad98c..32260f8 100644
--- a/tests/extra-tests/src/test/java/org/apache/activemq/artemis/tests/extras/byteman/FileLockMonitorTest.java
+++ b/tests/extra-tests/src/test/java/org/apache/activemq/artemis/tests/extras/byteman/FileLockMonitorTest.java
@@ -50,7 +50,7 @@ public class FileLockMonitorTest {
 
    @Test
    @BMRules(rules = {
-         @BMRule(name = "lock is invalid", targetClass = "org.apache.activemq.artemis.core.server.impl.FileLockNodeManager", targetMethod = "isLiveLockLost", action = "return true;") })
+      @BMRule(name = "lock is invalid", targetClass = "org.apache.activemq.artemis.core.server.impl.FileLockNodeManager", targetMethod = "isLiveLockLost", action = "return true;") })
    public void testLockMonitorInvalid() throws Exception {
       lostLock = false;
       startServer();
@@ -66,7 +66,7 @@ public class FileLockMonitorTest {
 
    @Test
    @BMRules(rules = {
-         @BMRule(name = "lock is invalid", targetClass = "org.apache.activemq.artemis.core.server.impl.FileLockNodeManager",
+      @BMRule(name = "lock is invalid", targetClass = "org.apache.activemq.artemis.core.server.impl.FileLockNodeManager",
             targetMethod = "getState",
             action = "org.apache.activemq.artemis.tests.extras.byteman.FileLockMonitorTest.throwNodeManagerException(\"EFS is disconnected\");") })
    public void testLockMonitorIOException() throws Exception {
@@ -88,7 +88,7 @@ public class FileLockMonitorTest {
 
    @Test
    @BMRules(rules = {
-         @BMRule(name = "lock is invalid", targetClass = "org.apache.activemq.artemis.core.server.impl.FileLockNodeManager", targetMethod = "getState", action = "return 70;") })
+      @BMRule(name = "lock is invalid", targetClass = "org.apache.activemq.artemis.core.server.impl.FileLockNodeManager", targetMethod = "getState", action = "return 70;") })
    public void testLockMonitorHasLockWrongState() throws Exception {
       lostLock = false;
       startServer();
diff --git a/tests/extra-tests/src/test/java/org/apache/activemq/artemis/tests/extras/byteman/FileLockNodeManagerTest.java b/tests/extra-tests/src/test/java/org/apache/activemq/artemis/tests/extras/byteman/FileLockNodeManagerTest.java
index 148755f..6f8d439 100644
--- a/tests/extra-tests/src/test/java/org/apache/activemq/artemis/tests/extras/byteman/FileLockNodeManagerTest.java
+++ b/tests/extra-tests/src/test/java/org/apache/activemq/artemis/tests/extras/byteman/FileLockNodeManagerTest.java
@@ -45,7 +45,8 @@ public class FileLockNodeManagerTest {
 
    @Test
    @BMRules(
-         rules = {@BMRule(
+         rules = {
+            @BMRule(
                name = "throw IOException during activation",
                targetClass = "org.apache.activemq.artemis.core.server.impl.FileLockNodeManager",
                targetMethod = "tryLock",
diff --git a/tests/extra-tests/src/test/java/org/apache/activemq/artemis/tests/extras/byteman/GroupingTest.java b/tests/extra-tests/src/test/java/org/apache/activemq/artemis/tests/extras/byteman/GroupingTest.java
index 477d5da..40a7aa8 100644
--- a/tests/extra-tests/src/test/java/org/apache/activemq/artemis/tests/extras/byteman/GroupingTest.java
+++ b/tests/extra-tests/src/test/java/org/apache/activemq/artemis/tests/extras/byteman/GroupingTest.java
@@ -61,12 +61,13 @@ public class GroupingTest extends JMSTestBase {
 
    @Test
    @BMRules(
-      rules = {@BMRule(
-         name = "trace clientsessionimpl commit",
-         targetClass = "org.apache.activemq.artemis.core.server.impl.ServerSessionImpl",
-         targetMethod = "rollback",
-         targetLocation = "EXIT",
-         action = "org.apache.activemq.artemis.tests.extras.byteman.GroupingTest.pause();")})
+      rules = {
+         @BMRule(
+            name = "trace clientsessionimpl commit",
+            targetClass = "org.apache.activemq.artemis.core.server.impl.ServerSessionImpl",
+            targetMethod = "rollback",
+            targetLocation = "EXIT",
+            action = "org.apache.activemq.artemis.tests.extras.byteman.GroupingTest.pause();")})
    public void testGroupingRollbackOnClose() throws Exception {
       Connection sendConnection = null;
       Connection connection = null;
diff --git a/tests/extra-tests/src/test/java/org/apache/activemq/artemis/tests/extras/byteman/HierarchicalObjectRepositoryTest.java b/tests/extra-tests/src/test/java/org/apache/activemq/artemis/tests/extras/byteman/HierarchicalObjectRepositoryTest.java
index 749bc49..cb3e458 100644
--- a/tests/extra-tests/src/test/java/org/apache/activemq/artemis/tests/extras/byteman/HierarchicalObjectRepositoryTest.java
+++ b/tests/extra-tests/src/test/java/org/apache/activemq/artemis/tests/extras/byteman/HierarchicalObjectRepositoryTest.java
@@ -33,10 +33,12 @@ import org.junit.Test;
 import org.junit.runner.RunWith;
 
 @RunWith(BMUnitRunner.class)
-@BMRules(rules = {@BMRule(name = "modify map during iteration",
-   targetClass = "org.apache.activemq.artemis.core.settings.impl.HierarchicalObjectRepository",
-   targetMethod = "getPossibleMatches(String)", targetLocation = "AT INVOKE java.util.HashMap.put",
-   action = "org.apache.activemq.artemis.tests.extras.byteman.HierarchicalObjectRepositoryTest.bum()"),})
+@BMRules(rules = {
+   @BMRule(name = "modify map during iteration",
+      targetClass = "org.apache.activemq.artemis.core.settings.impl.HierarchicalObjectRepository",
+      targetMethod = "getPossibleMatches(String)",
+      targetLocation = "AT INVOKE java.util.HashMap.put",
+      action = "org.apache.activemq.artemis.tests.extras.byteman.HierarchicalObjectRepositoryTest.bum()"),})
 public class HierarchicalObjectRepositoryTest {
 
    private static final String A = "a.";
diff --git a/tests/extra-tests/src/test/java/org/apache/activemq/artemis/tests/extras/byteman/InterruptedMessageHandlerTest.java b/tests/extra-tests/src/test/java/org/apache/activemq/artemis/tests/extras/byteman/InterruptedMessageHandlerTest.java
index 6ebbb50..6a28b94 100644
--- a/tests/extra-tests/src/test/java/org/apache/activemq/artemis/tests/extras/byteman/InterruptedMessageHandlerTest.java
+++ b/tests/extra-tests/src/test/java/org/apache/activemq/artemis/tests/extras/byteman/InterruptedMessageHandlerTest.java
@@ -63,17 +63,19 @@ public class InterruptedMessageHandlerTest extends ActiveMQRATestBase {
 
    @Test
    @BMRules(
-      rules = {@BMRule(
-         name = "throw ActiveMQException(CONNETION_TIMEOUT) during rollback",
-         targetClass = "org.apache.activemq.artemis.core.client.impl.ClientSessionImpl",
-         targetMethod = "flushAcks",
-         targetLocation = "AFTER INVOKE flushAcks",
-         action = "org.apache.activemq.artemis.tests.extras.byteman.InterruptedMessageHandlerTest.throwActiveMQQExceptionConnectionTimeout();"), @BMRule(
-         name = "check that outcome of XA transaction is TwoPhaseOutcome.FINISH_ERROR=8",
-         targetClass = "com.arjuna.ats.internal.jta.resources.arjunacore.XAResourceRecord",
-         targetMethod = "topLevelAbort",
-         targetLocation = "AT EXIT",
-         action = "org.apache.activemq.artemis.tests.extras.byteman.InterruptedMessageHandlerTest.assertTxOutComeIsOfStatusFinishedError($!);")})
+      rules = {
+         @BMRule(
+            name = "throw ActiveMQException(CONNETION_TIMEOUT) during rollback",
+            targetClass = "org.apache.activemq.artemis.core.client.impl.ClientSessionImpl",
+            targetMethod = "flushAcks",
+            targetLocation = "AFTER INVOKE flushAcks",
+            action = "org.apache.activemq.artemis.tests.extras.byteman.InterruptedMessageHandlerTest.throwActiveMQQExceptionConnectionTimeout();"),
+         @BMRule(
+            name = "check that outcome of XA transaction is TwoPhaseOutcome.FINISH_ERROR=8",
+            targetClass = "com.arjuna.ats.internal.jta.resources.arjunacore.XAResourceRecord",
+            targetMethod = "topLevelAbort",
+            targetLocation = "AT EXIT",
+            action = "org.apache.activemq.artemis.tests.extras.byteman.InterruptedMessageHandlerTest.assertTxOutComeIsOfStatusFinishedError($!);")})
    public void testSimpleMessageReceivedOnQueueTwoPhaseFailPrepareByConnectionTimout() throws Exception {
       ActiveMQResourceAdapter qResourceAdapter = newResourceAdapter();
       resourceAdapter = qResourceAdapter;
diff --git a/tests/extra-tests/src/test/java/org/apache/activemq/artemis/tests/extras/byteman/JMSBridgeReconnectionTest.java b/tests/extra-tests/src/test/java/org/apache/activemq/artemis/tests/extras/byteman/JMSBridgeReconnectionTest.java
index 1733431..829d68a 100644
--- a/tests/extra-tests/src/test/java/org/apache/activemq/artemis/tests/extras/byteman/JMSBridgeReconnectionTest.java
+++ b/tests/extra-tests/src/test/java/org/apache/activemq/artemis/tests/extras/byteman/JMSBridgeReconnectionTest.java
@@ -40,17 +40,19 @@ public class JMSBridgeReconnectionTest extends BridgeTestBase {
 
    @Test
    @BMRules(
-      rules = {@BMRule(
-         name = "trace clientsessionimpl send",
-         targetClass = "org.apache.activemq.artemis.core.protocol.core.impl.ChannelImpl",
-         targetMethod = "send",
-         targetLocation = "ENTRY",
-         action = "org.apache.activemq.artemis.tests.extras.byteman.JMSBridgeReconnectionTest.pause($1);"), @BMRule(
-         name = "trace sendRegularMessage",
-         targetClass = "org.apache.activemq.artemis.core.client.impl.ClientProducerImpl",
-         targetMethod = "sendRegularMessage",
-         targetLocation = "ENTRY",
-         action = "org.apache.activemq.artemis.tests.extras.byteman.JMSBridgeReconnectionTest.pause2($2,$3,$4);")})
+      rules = {
+         @BMRule(
+            name = "trace clientsessionimpl send",
+            targetClass = "org.apache.activemq.artemis.core.protocol.core.impl.ChannelImpl",
+            targetMethod = "send",
+            targetLocation = "ENTRY",
+            action = "org.apache.activemq.artemis.tests.extras.byteman.JMSBridgeReconnectionTest.pause($1);"),
+         @BMRule(
+            name = "trace sendRegularMessage",
+            targetClass = "org.apache.activemq.artemis.core.client.impl.ClientProducerImpl",
+            targetMethod = "sendRegularMessage",
+            targetLocation = "ENTRY",
+            action = "org.apache.activemq.artemis.tests.extras.byteman.JMSBridgeReconnectionTest.pause2($2,$3,$4);")})
    public void performCrashDestinationStopBridge() throws Exception {
       activeMQServer = server1;
       ConnectionFactoryFactory factInUse0 = cff0;
diff --git a/tests/extra-tests/src/test/java/org/apache/activemq/artemis/tests/extras/byteman/JournalImplConcurrencyTest.java b/tests/extra-tests/src/test/java/org/apache/activemq/artemis/tests/extras/byteman/JournalImplConcurrencyTest.java
index 5df1afa..64d25e5 100644
--- a/tests/extra-tests/src/test/java/org/apache/activemq/artemis/tests/extras/byteman/JournalImplConcurrencyTest.java
+++ b/tests/extra-tests/src/test/java/org/apache/activemq/artemis/tests/extras/byteman/JournalImplConcurrencyTest.java
@@ -43,15 +43,13 @@ public class JournalImplConcurrencyTest extends JournalImplTestBase {
     */
    @Test(timeout = 2000)
    @BMRules(rules = {
-         @BMRule(
-               name = "BlockOnFinalLargeMessagePacket",
-               targetClass = "java.util.concurrent.locks.ReentrantReadWriteLock",
-               targetMethod = "readLock()",
-               targetLocation = "EXIT",
-               condition = "Thread.currentThread().getName().contains(\"ArtemisIOThread\")",
-               action = "throw RuntimeException(\"Injected exception\");"
-         )
-   })
+      @BMRule(
+         name = "BlockOnFinalLargeMessagePacket",
+         targetClass = "java.util.concurrent.locks.ReentrantReadWriteLock",
+         targetMethod = "readLock()",
+         targetLocation = "EXIT",
+         condition = "Thread.currentThread().getName().contains(\"ArtemisIOThread\")",
+         action = "throw RuntimeException(\"Injected exception\");")})
    public void testTryDelete() throws Exception {
       setup(10, 10 * 1024, true);
       createJournal();
diff --git a/tests/extra-tests/src/test/java/org/apache/activemq/artemis/tests/extras/byteman/LargeMessageOnShutdownTest.java b/tests/extra-tests/src/test/java/org/apache/activemq/artemis/tests/extras/byteman/LargeMessageOnShutdownTest.java
index d21e070..ee98a73 100644
--- a/tests/extra-tests/src/test/java/org/apache/activemq/artemis/tests/extras/byteman/LargeMessageOnShutdownTest.java
+++ b/tests/extra-tests/src/test/java/org/apache/activemq/artemis/tests/extras/byteman/LargeMessageOnShutdownTest.java
@@ -74,15 +74,13 @@ public class LargeMessageOnShutdownTest extends ActiveMQTestBase {
             action =
                "org.apache.activemq.artemis.tests.extras.byteman.LargeMessageOnShutdownTest.stopServer();" +
                "waitFor(\"testLargeMessageOnShutdown\", 5000);" +
-               "flag(\"testLargeMessageOnShutdown\")"
-         ),
+               "flag(\"testLargeMessageOnShutdown\")"),
          @BMRule(
             name = "ReleaseBlockOnSessionCleanup",
             targetClass = "org.apache.activemq.artemis.core.protocol.core.ServerSessionPacketHandler",
             targetMethod = "clearLargeMessage()",
             targetLocation = "EXIT",
-            action = "signalWake(\"testLargeMessageOnShutdown\")"
-         )
+            action = "signalWake(\"testLargeMessageOnShutdown\")")
       }
    )
    public void testLargeMessageOnShutdown() throws Exception {
diff --git a/tests/extra-tests/src/test/java/org/apache/activemq/artemis/tests/extras/byteman/LargeMessageOverReplicationTest.java b/tests/extra-tests/src/test/java/org/apache/activemq/artemis/tests/extras/byteman/LargeMessageOverReplicationTest.java
index ddedeaa..7a28812 100644
--- a/tests/extra-tests/src/test/java/org/apache/activemq/artemis/tests/extras/byteman/LargeMessageOverReplicationTest.java
+++ b/tests/extra-tests/src/test/java/org/apache/activemq/artemis/tests/extras/byteman/LargeMessageOverReplicationTest.java
@@ -133,12 +133,13 @@ public class LargeMessageOverReplicationTest extends ActiveMQTestBase {
   */
    @Test
    @BMRules(
-      rules = {@BMRule(
-         name = "InterruptSending",
-         targetClass = "org.apache.activemq.artemis.core.protocol.core.impl.ActiveMQSessionContext",
-         targetMethod = "sendLargeMessageChunk",
-         targetLocation = "ENTRY",
-         action = "org.apache.activemq.artemis.tests.extras.byteman.LargeMessageOverReplicationTest.messageChunkSent();")})
+      rules = {
+         @BMRule(
+            name = "InterruptSending",
+            targetClass = "org.apache.activemq.artemis.core.protocol.core.impl.ActiveMQSessionContext",
+            targetMethod = "sendLargeMessageChunk",
+            targetLocation = "ENTRY",
+            action = "org.apache.activemq.artemis.tests.extras.byteman.LargeMessageOverReplicationTest.messageChunkSent();")})
    public void testSendLargeMessage() throws Exception {
 
       MapMessage message = createLargeMessage();
@@ -168,12 +169,13 @@ public class LargeMessageOverReplicationTest extends ActiveMQTestBase {
 
    @Test
    @BMRules(
-      rules = {@BMRule(
-         name = "InterruptReceive",
-         targetClass = "org.apache.activemq.artemis.core.protocol.core.impl.CoreSessionCallback",
-         targetMethod = "sendLargeMessageContinuation",
-         targetLocation = "ENTRY",
-         action = "org.apache.activemq.artemis.tests.extras.byteman.LargeMessageOverReplicationTest.messageChunkReceived();")})
+      rules = {
+         @BMRule(
+            name = "InterruptReceive",
+            targetClass = "org.apache.activemq.artemis.core.protocol.core.impl.CoreSessionCallback",
+            targetMethod = "sendLargeMessageContinuation",
+            targetLocation = "ENTRY",
+            action = "org.apache.activemq.artemis.tests.extras.byteman.LargeMessageOverReplicationTest.messageChunkReceived();")})
    public void testReceiveLargeMessage() throws Exception {
 
       MapMessage message = createLargeMessage();
diff --git a/tests/extra-tests/src/test/java/org/apache/activemq/artemis/tests/extras/byteman/LargeMessageReplicationTest.java b/tests/extra-tests/src/test/java/org/apache/activemq/artemis/tests/extras/byteman/LargeMessageReplicationTest.java
index 7c75971..d3e174c 100644
--- a/tests/extra-tests/src/test/java/org/apache/activemq/artemis/tests/extras/byteman/LargeMessageReplicationTest.java
+++ b/tests/extra-tests/src/test/java/org/apache/activemq/artemis/tests/extras/byteman/LargeMessageReplicationTest.java
@@ -111,23 +111,26 @@ public class LargeMessageReplicationTest extends FailoverTestBase {
 
    @Test
    @BMRules(
-      rules = {@BMRule(
-         name = "record large message copy thread",
-         targetClass = "org.apache.activemq.artemis.core.persistence.impl.journal.LargeServerMessageImpl",
-         targetMethod = "copy(long)",
-         targetLocation = "ENTRY",
-         action = "org.apache.activemq.artemis.tests.extras.byteman.LargeMessageReplicationTest.copyThread()"), @BMRule(
-         name = "record byte array in addBytes",
-         targetClass = "org.apache.activemq.artemis.core.persistence.impl.journal.LargeServerMessageImpl",
-         targetMethod = "addBytes(byte[])",
-         targetLocation = "ENTRY",
-         action = "org.apache.activemq.artemis.tests.extras.byteman.LargeMessageReplicationTest.addBytesIn($1)"), @BMRule(
-         name = "record byte array used for reading large message",
-         targetClass = "^org.apache.activemq.artemis.core.io.SequentialFile",
-         isInterface = true,
-         targetMethod = "read(java.nio.ByteBuffer)",
-         targetLocation = "ENTRY",
-         action = "org.apache.activemq.artemis.tests.extras.byteman.LargeMessageReplicationTest.originBuff($1)")})
+      rules = {
+         @BMRule(
+            name = "record large message copy thread",
+            targetClass = "org.apache.activemq.artemis.core.persistence.impl.journal.LargeServerMessageImpl",
+            targetMethod = "copy(long)",
+            targetLocation = "ENTRY",
+            action = "org.apache.activemq.artemis.tests.extras.byteman.LargeMessageReplicationTest.copyThread()"),
+         @BMRule(
+            name = "record byte array in addBytes",
+            targetClass = "org.apache.activemq.artemis.core.persistence.impl.journal.LargeServerMessageImpl",
+            targetMethod = "addBytes(byte[])",
+            targetLocation = "ENTRY",
+            action = "org.apache.activemq.artemis.tests.extras.byteman.LargeMessageReplicationTest.addBytesIn($1)"),
+         @BMRule(
+            name = "record byte array used for reading large message",
+            targetClass = "^org.apache.activemq.artemis.core.io.SequentialFile",
+            isInterface = true,
+            targetMethod = "read(java.nio.ByteBuffer)",
+            targetLocation = "ENTRY",
+            action = "org.apache.activemq.artemis.tests.extras.byteman.LargeMessageReplicationTest.originBuff($1)")})
    //https://issues.apache.org/jira/browse/ARTEMIS-1220
    public void testDivertCopyMessageBuffer() throws Exception {
       Map<String, Object> params = new HashMap<>();
diff --git a/tests/extra-tests/src/test/java/org/apache/activemq/artemis/tests/extras/byteman/LatencyTest.java b/tests/extra-tests/src/test/java/org/apache/activemq/artemis/tests/extras/byteman/LatencyTest.java
index 5709071..9c78974 100644
--- a/tests/extra-tests/src/test/java/org/apache/activemq/artemis/tests/extras/byteman/LatencyTest.java
+++ b/tests/extra-tests/src/test/java/org/apache/activemq/artemis/tests/extras/byteman/LatencyTest.java
@@ -41,17 +41,19 @@ public class LatencyTest extends ActiveMQTestBase {
    * */
    @Test
    @BMRules(
-      rules = {@BMRule(
-         name = "trace ClientBootstrap.connect",
-         targetClass = "org.jboss.netty.bootstrap.ClientBootstrap",
-         targetMethod = "connect",
-         targetLocation = "ENTRY",
-         action = "org.apache.activemq.artemis.tests.extras.byteman.LatencyTest.debugLog(\"netty connecting\")"), @BMRule(
-         name = "sleep OioWorker.run",
-         targetClass = "org.jboss.netty.channel.socket.oio.OioWorker",
-         targetMethod = "run",
-         targetLocation = "ENTRY",
-         action = "Thread.sleep(500)")})
+      rules = {
+         @BMRule(
+            name = "trace ClientBootstrap.connect",
+            targetClass = "org.jboss.netty.bootstrap.ClientBootstrap",
+            targetMethod = "connect",
+            targetLocation = "ENTRY",
+            action = "org.apache.activemq.artemis.tests.extras.byteman.LatencyTest.debugLog(\"netty connecting\")"),
+         @BMRule(
+            name = "sleep OioWorker.run",
+            targetClass = "org.jboss.netty.channel.socket.oio.OioWorker",
+            targetMethod = "run",
+            targetLocation = "ENTRY",
+            action = "Thread.sleep(500)")})
    public void testLatency() throws Exception {
       ActiveMQServer server = createServer(createDefaultNettyConfig());
       server.start();
diff --git a/tests/extra-tests/src/test/java/org/apache/activemq/artemis/tests/extras/byteman/LoggingActiveMQServerPluginAMQPTest.java b/tests/extra-tests/src/test/java/org/apache/activemq/artemis/tests/extras/byteman/LoggingActiveMQServerPluginAMQPTest.java
index 9d5b077..6a0b511 100644
--- a/tests/extra-tests/src/test/java/org/apache/activemq/artemis/tests/extras/byteman/LoggingActiveMQServerPluginAMQPTest.java
+++ b/tests/extra-tests/src/test/java/org/apache/activemq/artemis/tests/extras/byteman/LoggingActiveMQServerPluginAMQPTest.java
@@ -48,11 +48,12 @@ public class LoggingActiveMQServerPluginAMQPTest extends LoggingActiveMQServerPl
     */
    @Override
    @Test
-   @BMRules(rules = {@BMRule(name = "test logAll EVENT",
-      targetClass = "org.jboss.logging.Logger",
-      targetMethod = "logv",
-      targetLocation = "ENTRY",
-      action = "org.apache.activemq.artemis.tests.extras.byteman.LoggingActiveMQServerPluginTest.infoLog($2, $4, $0)")})
+   @BMRules(rules = {
+      @BMRule(name = "test logAll EVENT",
+         targetClass = "org.jboss.logging.Logger",
+         targetMethod = "logv",
+         targetLocation = "ENTRY",
+         action = "org.apache.activemq.artemis.tests.extras.byteman.LoggingActiveMQServerPluginTest.infoLog($2, $4, $0)")})
    public void testLogAll() throws Exception {
 
       //initial plugin
@@ -94,11 +95,12 @@ public class LoggingActiveMQServerPluginAMQPTest extends LoggingActiveMQServerPl
     */
    @Override
    @Test
-   @BMRules(rules = {@BMRule(name = "test LOG_CONSUMER_EVENTS",
-      targetClass = "org.jboss.logging.Logger",
-      targetMethod = "logv",
-      targetLocation = "ENTRY",
-      action = "org.apache.activemq.artemis.tests.extras.byteman.LoggingActiveMQServerPluginTest.infoLog($2, $4, $0)")})
+   @BMRules(rules = {
+      @BMRule(name = "test LOG_CONSUMER_EVENTS",
+         targetClass = "org.jboss.logging.Logger",
+         targetMethod = "logv",
+         targetLocation = "ENTRY",
+         action = "org.apache.activemq.artemis.tests.extras.byteman.LoggingActiveMQServerPluginTest.infoLog($2, $4, $0)")})
    public void testLogConsumerEvents() throws Exception {
 
       ActiveMQServer activeMQServer = createServerWithLoggingPlugin(LoggingActiveMQServerPlugin.LOG_CONSUMER_EVENTS);
@@ -140,12 +142,12 @@ public class LoggingActiveMQServerPluginAMQPTest extends LoggingActiveMQServerPl
     */
    @Override
    @Test
-   @BMRules(rules = {@BMRule(name = "test LOG_SESSION_EVENTS",
-      targetClass = "org.jboss.logging.Logger",
-      targetMethod = "logv",
-      targetLocation = "ENTRY",
-      action = "org.apache.activemq.artemis.tests.extras.byteman.LoggingActiveMQServerPluginTest.infoLog($2, $4, $0)")})
-
+   @BMRules(rules = {
+      @BMRule(name = "test LOG_SESSION_EVENTS",
+         targetClass = "org.jboss.logging.Logger",
+         targetMethod = "logv",
+         targetLocation = "ENTRY",
+         action = "org.apache.activemq.artemis.tests.extras.byteman.LoggingActiveMQServerPluginTest.infoLog($2, $4, $0)")})
    public void testLogSessionEvents() throws Exception {
 
       ActiveMQServer activeMQServer  = createServerWithLoggingPlugin(LoggingActiveMQServerPlugin.LOG_SESSION_EVENTS);
diff --git a/tests/extra-tests/src/test/java/org/apache/activemq/artemis/tests/extras/byteman/LoggingActiveMQServerPluginOpenWireTest.java b/tests/extra-tests/src/test/java/org/apache/activemq/artemis/tests/extras/byteman/LoggingActiveMQServerPluginOpenWireTest.java
index a7ffc4c..7e9c359 100644
--- a/tests/extra-tests/src/test/java/org/apache/activemq/artemis/tests/extras/byteman/LoggingActiveMQServerPluginOpenWireTest.java
+++ b/tests/extra-tests/src/test/java/org/apache/activemq/artemis/tests/extras/byteman/LoggingActiveMQServerPluginOpenWireTest.java
@@ -52,11 +52,12 @@ public class LoggingActiveMQServerPluginOpenWireTest extends LoggingActiveMQServ
     */
    @Override
    @Test
-   @BMRules(rules = {@BMRule(name = "test queue creation log",
-      targetClass = "org.jboss.logging.Logger",
-      targetMethod = "logv",
-      targetLocation = "ENTRY",
-      action = "org.apache.activemq.artemis.tests.extras.byteman.LoggingActiveMQServerPluginTest.infoLog($2, $4, $0)")})
+   @BMRules(rules = {
+      @BMRule(name = "test queue creation log",
+         targetClass = "org.jboss.logging.Logger",
+         targetMethod = "logv",
+         targetLocation = "ENTRY",
+         action = "org.apache.activemq.artemis.tests.extras.byteman.LoggingActiveMQServerPluginTest.infoLog($2, $4, $0)")})
    public void testQueueCreationLog() throws Exception {
 
       //initial plugin
@@ -100,12 +101,12 @@ public class LoggingActiveMQServerPluginOpenWireTest extends LoggingActiveMQServ
     */
    @Override
    @Test
-   @BMRules(rules = {@BMRule(name = "test LOG_SESSION_EVENTS",
-      targetClass = "org.jboss.logging.Logger",
-      targetMethod = "logv",
-      targetLocation = "ENTRY",
-      action = "org.apache.activemq.artemis.tests.extras.byteman.LoggingActiveMQServerPluginTest.infoLog($2, $4, $0)")})
-
+   @BMRules(rules = {
+      @BMRule(name = "test LOG_SESSION_EVENTS",
+         targetClass = "org.jboss.logging.Logger",
+         targetMethod = "logv",
+         targetLocation = "ENTRY",
+         action = "org.apache.activemq.artemis.tests.extras.byteman.LoggingActiveMQServerPluginTest.infoLog($2, $4, $0)")})
    public void testLogSessionEvents() throws Exception {
 
       ActiveMQServer activeMQServer  = createServerWithLoggingPlugin(LoggingActiveMQServerPlugin.LOG_SESSION_EVENTS);
@@ -141,11 +142,12 @@ public class LoggingActiveMQServerPluginOpenWireTest extends LoggingActiveMQServ
     */
    @Override
    @Test
-   @BMRules(rules = {@BMRule(name = "test logAll EVENT",
-      targetClass = "org.jboss.logging.Logger",
-      targetMethod = "logv",
-      targetLocation = "ENTRY",
-      action = "org.apache.activemq.artemis.tests.extras.byteman.LoggingActiveMQServerPluginTest.infoLog($2, $4, $0)")})
+   @BMRules(rules = {
+      @BMRule(name = "test logAll EVENT",
+         targetClass = "org.jboss.logging.Logger",
+         targetMethod = "logv",
+         targetLocation = "ENTRY",
+         action = "org.apache.activemq.artemis.tests.extras.byteman.LoggingActiveMQServerPluginTest.infoLog($2, $4, $0)")})
    public void testLogAll() throws Exception {
 
       //initial plugin
diff --git a/tests/extra-tests/src/test/java/org/apache/activemq/artemis/tests/extras/byteman/LoggingActiveMQServerPluginTest.java b/tests/extra-tests/src/test/java/org/apache/activemq/artemis/tests/extras/byteman/LoggingActiveMQServerPluginTest.java
index be64e14..4bb6d3b 100644
--- a/tests/extra-tests/src/test/java/org/apache/activemq/artemis/tests/extras/byteman/LoggingActiveMQServerPluginTest.java
+++ b/tests/extra-tests/src/test/java/org/apache/activemq/artemis/tests/extras/byteman/LoggingActiveMQServerPluginTest.java
@@ -78,11 +78,12 @@ public class LoggingActiveMQServerPluginTest extends ActiveMQTestBase {
     */
 
    @Test
-   @BMRules(rules = {@BMRule(name = "test LOG_CONNECTION_EVENTS",
-      targetClass = "org.jboss.logging.Logger",
-      targetMethod = "logv",
-      targetLocation = "ENTRY",
-      action = "org.apache.activemq.artemis.tests.extras.byteman.LoggingActiveMQServerPluginTest.infoLog($2, $4, $0)")})
+   @BMRules(rules = {
+      @BMRule(name = "test LOG_CONNECTION_EVENTS",
+         targetClass = "org.jboss.logging.Logger",
+         targetMethod = "logv",
+         targetLocation = "ENTRY",
+         action = "org.apache.activemq.artemis.tests.extras.byteman.LoggingActiveMQServerPluginTest.infoLog($2, $4, $0)")})
    public void testLogConnectEvents() throws Exception {
 
       ActiveMQServer activeMQServer = createServerWithLoggingPlugin(LoggingActiveMQServerPlugin.LOG_CONNECTION_EVENTS);
@@ -129,11 +130,12 @@ public class LoggingActiveMQServerPluginTest extends ActiveMQTestBase {
     * @throws Exception
     */
    @Test
-   @BMRules(rules = {@BMRule(name = "test LOG_SESSION_EVENTS",
-      targetClass = "org.jboss.logging.Logger",
-      targetMethod = "logv",
-      targetLocation = "ENTRY",
-      action = "org.apache.activemq.artemis.tests.extras.byteman.LoggingActiveMQServerPluginTest.infoLog($2, $4, $0)")})
+   @BMRules(rules = {
+      @BMRule(name = "test LOG_SESSION_EVENTS",
+         targetClass = "org.jboss.logging.Logger",
+         targetMethod = "logv",
+         targetLocation = "ENTRY",
+         action = "org.apache.activemq.artemis.tests.extras.byteman.LoggingActiveMQServerPluginTest.infoLog($2, $4, $0)")})
    public void testLogSessionEvents() throws Exception {
 
       ActiveMQServer activeMQServer = createServerWithLoggingPlugin(LoggingActiveMQServerPlugin.LOG_SESSION_EVENTS);
@@ -165,11 +167,12 @@ public class LoggingActiveMQServerPluginTest extends ActiveMQTestBase {
     */
 
    @Test
-   @BMRules(rules = {@BMRule(name = "test LOG_CONSUMER_EVENTS",
-      targetClass = "org.jboss.logging.Logger",
-      targetMethod = "logv",
-      targetLocation = "ENTRY",
-      action = "org.apache.activemq.artemis.tests.extras.byteman.LoggingActiveMQServerPluginTest.infoLog($2, $4, $0)")})
+   @BMRules(rules = {
+      @BMRule(name = "test LOG_CONSUMER_EVENTS",
+         targetClass = "org.jboss.logging.Logger",
+         targetMethod = "logv",
+         targetLocation = "ENTRY",
+         action = "org.apache.activemq.artemis.tests.extras.byteman.LoggingActiveMQServerPluginTest.infoLog($2, $4, $0)")})
    public void testLogConsumerEvents() throws Exception {
 
       ActiveMQServer activeMQServer = createServerWithLoggingPlugin(LoggingActiveMQServerPlugin.LOG_CONSUMER_EVENTS);
@@ -207,11 +210,12 @@ public class LoggingActiveMQServerPluginTest extends ActiveMQTestBase {
     * @throws Exception
     */
    @Test
-   @BMRules(rules = {@BMRule(name = "test LOG_DELIVERING_EVENTS",
-      targetClass = "org.jboss.logging.Logger",
-      targetMethod = "logv",
-      targetLocation = "ENTRY",
-      action = "org.apache.activemq.artemis.tests.extras.byteman.LoggingActiveMQServerPluginTest.infoLog($2, $4, $0)")})
+   @BMRules(rules = {
+      @BMRule(name = "test LOG_DELIVERING_EVENTS",
+         targetClass = "org.jboss.logging.Logger",
+         targetMethod = "logv",
+         targetLocation = "ENTRY",
+         action = "org.apache.activemq.artemis.tests.extras.byteman.LoggingActiveMQServerPluginTest.infoLog($2, $4, $0)")})
    public void testLogDeliveringEvents() throws Exception {
 
       ActiveMQServer activeMQServer = createServerWithLoggingPlugin(LoggingActiveMQServerPlugin.LOG_DELIVERING_EVENTS);
@@ -249,11 +253,12 @@ public class LoggingActiveMQServerPluginTest extends ActiveMQTestBase {
     * @throws Exception
     */
    @Test
-   @BMRules(rules = {@BMRule(name = "test LOG_SENDING_EVENTS",
-      targetClass = "org.jboss.logging.Logger",
-      targetMethod = "logv",
-      targetLocation = "ENTRY",
-      action = "org.apache.activemq.artemis.tests.extras.byteman.LoggingActiveMQServerPluginTest.infoLog($2, $4, $0)")})
+   @BMRules(rules = {
+      @BMRule(name = "test LOG_SENDING_EVENTS",
+         targetClass = "org.jboss.logging.Logger",
+         targetMethod = "logv",
+         targetLocation = "ENTRY",
+         action = "org.apache.activemq.artemis.tests.extras.byteman.LoggingActiveMQServerPluginTest.infoLog($2, $4, $0)")})
    public void testLogSendingEvents() throws Exception {
 
       //initial plugin
@@ -293,11 +298,12 @@ public class LoggingActiveMQServerPluginTest extends ActiveMQTestBase {
     * @throws Exception
     */
    @Test
-   @BMRules(rules = {@BMRule(name = "test queue creation log",
-      targetClass = "org.jboss.logging.Logger",
-      targetMethod = "logv",
-      targetLocation = "ENTRY",
-      action = "org.apache.activemq.artemis.tests.extras.byteman.LoggingActiveMQServerPluginTest.infoLog($2, $4, $0)")})
+   @BMRules(rules = {
+      @BMRule(name = "test queue creation log",
+         targetClass = "org.jboss.logging.Logger",
+         targetMethod = "logv",
+         targetLocation = "ENTRY",
+         action = "org.apache.activemq.artemis.tests.extras.byteman.LoggingActiveMQServerPluginTest.infoLog($2, $4, $0)")})
    public void testQueueCreationLog() throws Exception {
 
       //initial plugin
@@ -336,11 +342,12 @@ public class LoggingActiveMQServerPluginTest extends ActiveMQTestBase {
     * @throws Exception
     */
    @Test
-   @BMRules(rules = {@BMRule(name = "test queue creation log",
-      targetClass = "org.jboss.logging.Logger",
-      targetMethod = "logv",
-      targetLocation = "ENTRY",
-      action = "org.apache.activemq.artemis.tests.extras.byteman.LoggingActiveMQServerPluginTest.infoLog($2, $4, $0)")})
+   @BMRules(rules = {
+      @BMRule(name = "test queue creation log",
+         targetClass = "org.jboss.logging.Logger",
+         targetMethod = "logv",
+         targetLocation = "ENTRY",
+         action = "org.apache.activemq.artemis.tests.extras.byteman.LoggingActiveMQServerPluginTest.infoLog($2, $4, $0)")})
    public void testExpireMessageLog() throws Exception {
 
       //initial plugin
@@ -381,11 +388,12 @@ public class LoggingActiveMQServerPluginTest extends ActiveMQTestBase {
     * @throws Exception
     */
    @Test
-   @BMRules(rules = {@BMRule(name = "test logAll EVENT",
-      targetClass = "org.jboss.logging.Logger",
-      targetMethod = "logv",
-      targetLocation = "ENTRY",
-      action = "org.apache.activemq.artemis.tests.extras.byteman.LoggingActiveMQServerPluginTest.infoLog($2, $4, $0)")})
+   @BMRules(rules = {
+      @BMRule(name = "test logAll EVENT",
+         targetClass = "org.jboss.logging.Logger",
+         targetMethod = "logv",
+         targetLocation = "ENTRY",
+         action = "org.apache.activemq.artemis.tests.extras.byteman.LoggingActiveMQServerPluginTest.infoLog($2, $4, $0)")})
    public void testLogAll() throws Exception {
 
       //initial plugin
diff --git a/tests/extra-tests/src/test/java/org/apache/activemq/artemis/tests/extras/byteman/ManagementExceptionHandlingTest.java b/tests/extra-tests/src/test/java/org/apache/activemq/artemis/tests/extras/byteman/ManagementExceptionHandlingTest.java
index 3712f45..a210b1d 100644
--- a/tests/extra-tests/src/test/java/org/apache/activemq/artemis/tests/extras/byteman/ManagementExceptionHandlingTest.java
+++ b/tests/extra-tests/src/test/java/org/apache/activemq/artemis/tests/extras/byteman/ManagementExceptionHandlingTest.java
@@ -62,13 +62,13 @@ public class ManagementExceptionHandlingTest extends ActiveMQTestBase {
    }
 
    @Test
-   @BMRules(
-           rules = {@BMRule(
-                   name = "checking ActiveMQServerControl methods",
-                   targetClass = "org.apache.activemq.artemis.core.server.impl.ActiveMQServerImpl",
-                   targetMethod = "createQueue(org.apache.activemq.artemis.api.core.SimpleString, org.apache.activemq.artemis.api.core.RoutingType, org.apache.activemq.artemis.api.core.SimpleString, org.apache.activemq.artemis.api.core.SimpleString, boolean, boolean, int, boolean, boolean)",
-                   targetLocation = "EXIT",
-                   action = "throw new org.apache.activemq.artemis.api.core.ActiveMQException(\"gotcha\")")})
+   @BMRules(rules = {
+      @BMRule(
+         name = "checking ActiveMQServerControl methods",
+         targetClass = "org.apache.activemq.artemis.core.server.impl.ActiveMQServerImpl",
+         targetMethod = "createQueue(org.apache.activemq.artemis.api.core.SimpleString, org.apache.activemq.artemis.api.core.RoutingType, org.apache.activemq.artemis.api.core.SimpleString, org.apache.activemq.artemis.api.core.SimpleString, boolean, boolean, int, boolean, boolean)",
+         targetLocation = "EXIT",
+         action = "throw new org.apache.activemq.artemis.api.core.ActiveMQException(\"gotcha\")")})
    public void testActiveMQServerControl() throws Exception {
       try {
          server.getActiveMQServerControl().createQueue(new QueueConfiguration("some.queue").setAddress("some.address").setRoutingType(RoutingType.ANYCAST).toJSON());
@@ -81,13 +81,13 @@ public class ManagementExceptionHandlingTest extends ActiveMQTestBase {
    }
 
    @Test
-   @BMRules(
-           rules = {@BMRule(
-                   name = "checking ActiveMQServerControl methods",
-                   targetClass = "org.apache.activemq.artemis.core.postoffice.impl.PostOfficeImpl",
-                   targetMethod = "route(org.apache.activemq.artemis.api.core.Message, boolean)",
-                   targetLocation = "ENTRY",
-                   action = "throw new org.apache.activemq.artemis.api.core.ActiveMQException(\"gotcha\")")})
+   @BMRules(rules = {
+      @BMRule(
+         name = "checking ActiveMQServerControl methods",
+         targetClass = "org.apache.activemq.artemis.core.postoffice.impl.PostOfficeImpl",
+         targetMethod = "route(org.apache.activemq.artemis.api.core.Message, boolean)",
+         targetLocation = "ENTRY",
+         action = "throw new org.apache.activemq.artemis.api.core.ActiveMQException(\"gotcha\")")})
    public void testAddressControl() throws Exception {
       server.getActiveMQServerControl().createAddress("test.address", "ANYCAST");
       MBeanServer mbs = ManagementFactory.getPlatformMBeanServer();
diff --git a/tests/extra-tests/src/test/java/org/apache/activemq/artemis/tests/extras/byteman/OrphanedConsumerTest.java b/tests/extra-tests/src/test/java/org/apache/activemq/artemis/tests/extras/byteman/OrphanedConsumerTest.java
index 733a862..39bf3de 100644
--- a/tests/extra-tests/src/test/java/org/apache/activemq/artemis/tests/extras/byteman/OrphanedConsumerTest.java
+++ b/tests/extra-tests/src/test/java/org/apache/activemq/artemis/tests/extras/byteman/OrphanedConsumerTest.java
@@ -109,19 +109,21 @@ public class OrphanedConsumerTest extends ActiveMQTestBase {
     */
    @Test
    @BMRules(
-      rules = {@BMRule(
-         name = "closeExit",
-         targetClass = "org.apache.activemq.artemis.core.server.impl.ServerConsumerImpl",
-         targetMethod = "close",
-         targetLocation = "AT EXIT",
-         condition = "org.apache.activemq.artemis.tests.extras.byteman.OrphanedConsumerTest.isConditionActive()",
-         action = "org.apache.activemq.artemis.tests.extras.byteman.OrphanedConsumerTest.debugLog(\"throwing stuff\");throw new InterruptedException()"), @BMRule(
-         name = "closeEnter",
-         targetClass = "org.apache.activemq.artemis.core.server.impl.ServerConsumerImpl",
-         targetMethod = "close",
-         targetLocation = "ENTRY",
-         condition = "org.apache.activemq.artemis.tests.extras.byteman.OrphanedConsumerTest.isConditionActive()",
-         action = "org.apache.activemq.artemis.tests.extras.byteman.OrphanedConsumerTest.leavingCloseOnTestCountersWhileClosing()")})
+      rules = {
+         @BMRule(
+            name = "closeExit",
+            targetClass = "org.apache.activemq.artemis.core.server.impl.ServerConsumerImpl",
+            targetMethod = "close",
+            targetLocation = "AT EXIT",
+            condition = "org.apache.activemq.artemis.tests.extras.byteman.OrphanedConsumerTest.isConditionActive()",
+            action = "org.apache.activemq.artemis.tests.extras.byteman.OrphanedConsumerTest.debugLog(\"throwing stuff\");throw new InterruptedException()"),
+         @BMRule(
+            name = "closeEnter",
+            targetClass = "org.apache.activemq.artemis.core.server.impl.ServerConsumerImpl",
+            targetMethod = "close",
+            targetLocation = "ENTRY",
+            condition = "org.apache.activemq.artemis.tests.extras.byteman.OrphanedConsumerTest.isConditionActive()",
+            action = "org.apache.activemq.artemis.tests.extras.byteman.OrphanedConsumerTest.leavingCloseOnTestCountersWhileClosing()")})
    public void testOrphanedConsumers() throws Exception {
       internalTestOrphanedConsumers(false);
    }
@@ -136,19 +138,21 @@ public class OrphanedConsumerTest extends ActiveMQTestBase {
     */
    @Test
    @BMRules(
-      rules = {@BMRule(
-         name = "closeExit",
-         targetClass = "org.apache.activemq.artemis.core.server.impl.ServerConsumerImpl",
-         targetMethod = "close",
-         targetLocation = "AT EXIT",
-         condition = "org.apache.activemq.artemis.tests.extras.byteman.OrphanedConsumerTest.isConditionActive()",
-         action = "org.apache.activemq.artemis.tests.extras.byteman.OrphanedConsumerTest.debugLog(\"throwing stuff\");throw new InterruptedException()"), @BMRule(
-         name = "closeEnter",
-         targetClass = "org.apache.activemq.artemis.core.server.impl.ServerConsumerImpl",
-         targetMethod = "close",
-         targetLocation = "ENTRY",
-         condition = "org.apache.activemq.artemis.tests.extras.byteman.OrphanedConsumerTest.isConditionActive()",
-         action = "org.apache.activemq.artemis.tests.extras.byteman.OrphanedConsumerTest.leavingCloseOnTestCountersWhileClosing()")})
+      rules = {
+         @BMRule(
+            name = "closeExit",
+            targetClass = "org.apache.activemq.artemis.core.server.impl.ServerConsumerImpl",
+            targetMethod = "close",
+            targetLocation = "AT EXIT",
+            condition = "org.apache.activemq.artemis.tests.extras.byteman.OrphanedConsumerTest.isConditionActive()",
+            action = "org.apache.activemq.artemis.tests.extras.byteman.OrphanedConsumerTest.debugLog(\"throwing stuff\");throw new InterruptedException()"),
+         @BMRule(
+            name = "closeEnter",
+            targetClass = "org.apache.activemq.artemis.core.server.impl.ServerConsumerImpl",
+            targetMethod = "close",
+            targetLocation = "ENTRY",
+            condition = "org.apache.activemq.artemis.tests.extras.byteman.OrphanedConsumerTest.isConditionActive()",
+            action = "org.apache.activemq.artemis.tests.extras.byteman.OrphanedConsumerTest.leavingCloseOnTestCountersWhileClosing()")})
    public void testOrphanedConsumersByManagement() throws Exception {
       internalTestOrphanedConsumers(true);
    }
diff --git a/tests/extra-tests/src/test/java/org/apache/activemq/artemis/tests/extras/byteman/PagingLeakTest.java b/tests/extra-tests/src/test/java/org/apache/activemq/artemis/tests/extras/byteman/PagingLeakTest.java
index da3294d..06bf378 100644
--- a/tests/extra-tests/src/test/java/org/apache/activemq/artemis/tests/extras/byteman/PagingLeakTest.java
+++ b/tests/extra-tests/src/test/java/org/apache/activemq/artemis/tests/extras/byteman/PagingLeakTest.java
@@ -66,17 +66,19 @@ public class PagingLeakTest extends ActiveMQTestBase {
 
    @Test
    @BMRules(
-      rules = {@BMRule(
-         name = "newPosition",
-         targetClass = "org.apache.activemq.artemis.core.paging.cursor.impl.PagePositionImpl",
-         targetMethod = "<init>()",
-         targetLocation = "ENTRY",
-         action = "org.apache.activemq.artemis.tests.extras.byteman.PagingLeakTest.newPosition()"), @BMRule(
-         name = "finalPosition",
-         targetClass = "org.apache.activemq.artemis.core.paging.cursor.impl.PagePositionImpl",
-         targetMethod = "finalize",
-         targetLocation = "ENTRY",
-         action = "org.apache.activemq.artemis.tests.extras.byteman.PagingLeakTest.deletePosition()")})
+      rules = {
+         @BMRule(
+            name = "newPosition",
+            targetClass = "org.apache.activemq.artemis.core.paging.cursor.impl.PagePositionImpl",
+            targetMethod = "<init>()",
+            targetLocation = "ENTRY",
+            action = "org.apache.activemq.artemis.tests.extras.byteman.PagingLeakTest.newPosition()"),
+         @BMRule(
+            name = "finalPosition",
+            targetClass = "org.apache.activemq.artemis.core.paging.cursor.impl.PagePositionImpl",
+            targetMethod = "finalize",
+            targetLocation = "ENTRY",
+            action = "org.apache.activemq.artemis.tests.extras.byteman.PagingLeakTest.deletePosition()")})
    public void testValidateLeak() throws Throwable {
       log.debug("location::" + getBindingsDir());
 
diff --git a/tests/extra-tests/src/test/java/org/apache/activemq/artemis/tests/extras/byteman/PagingOMETest.java b/tests/extra-tests/src/test/java/org/apache/activemq/artemis/tests/extras/byteman/PagingOMETest.java
index bc94f85..63b876c 100644
--- a/tests/extra-tests/src/test/java/org/apache/activemq/artemis/tests/extras/byteman/PagingOMETest.java
+++ b/tests/extra-tests/src/test/java/org/apache/activemq/artemis/tests/extras/byteman/PagingOMETest.java
@@ -77,12 +77,13 @@ public class PagingOMETest extends ActiveMQTestBase {
 
    @Test
    @BMRules(
-      rules = {@BMRule(
-         name = "fakeOME",
-         targetClass = "org.apache.activemq.artemis.core.paging.cursor.PagedReferenceImpl",
-         targetMethod = "getPagedMessage",
-         targetLocation = "ENTRY",
-         action = "org.apache.activemq.artemis.tests.extras.byteman.PagingOMETest.refCheck()")})
+      rules = {
+         @BMRule(
+            name = "fakeOME",
+            targetClass = "org.apache.activemq.artemis.core.paging.cursor.PagedReferenceImpl",
+            targetMethod = "getPagedMessage",
+            targetLocation = "ENTRY",
+            action = "org.apache.activemq.artemis.tests.extras.byteman.PagingOMETest.refCheck()")})
    public void testPageCleanup() throws Exception {
       clearDataRecreateServerDirs();
 
diff --git a/tests/extra-tests/src/test/java/org/apache/activemq/artemis/tests/extras/byteman/QueueDeploymentFailedTest.java b/tests/extra-tests/src/test/java/org/apache/activemq/artemis/tests/extras/byteman/QueueDeploymentFailedTest.java
index d932c41..aec7903 100644
--- a/tests/extra-tests/src/test/java/org/apache/activemq/artemis/tests/extras/byteman/QueueDeploymentFailedTest.java
+++ b/tests/extra-tests/src/test/java/org/apache/activemq/artemis/tests/extras/byteman/QueueDeploymentFailedTest.java
@@ -32,6 +32,7 @@ import org.junit.runner.RunWith;
 public class QueueDeploymentFailedTest extends ActiveMQTestBase {
 
    @Test
+
    @BMRule(name = "blow up queue deployment",
       targetClass = "org.apache.activemq.artemis.core.server.impl.ActiveMQServerImpl",
       targetMethod = "createQueue(SimpleString,RoutingType,SimpleString,SimpleString,SimpleString,boolean,boolean,boolean,boolean,boolean,int,boolean,boolean,boolean,int,long,boolean",
diff --git a/tests/extra-tests/src/test/java/org/apache/activemq/artemis/tests/extras/byteman/RaceOnClosingConsumerWhileReconnecting.java b/tests/extra-tests/src/test/java/org/apache/activemq/artemis/tests/extras/byteman/RaceOnClosingConsumerWhileReconnecting.java
index a7a6e39..306f04e 100644
--- a/tests/extra-tests/src/test/java/org/apache/activemq/artemis/tests/extras/byteman/RaceOnClosingConsumerWhileReconnecting.java
+++ b/tests/extra-tests/src/test/java/org/apache/activemq/artemis/tests/extras/byteman/RaceOnClosingConsumerWhileReconnecting.java
@@ -97,12 +97,13 @@ public class RaceOnClosingConsumerWhileReconnecting extends ActiveMQTestBase {
 
    @Test
    @BMRules(
-      rules = {@BMRule(
-         name = "session.removeConsumer wait",
-         targetClass = "org.apache.activemq.artemis.core.client.impl.ClientSessionImpl",
-         targetMethod = "removeConsumer(org.apache.activemq.artemis.core.client.impl.ClientConsumerInternal)",
-         targetLocation = "ENTRY",
-         action = "org.apache.activemq.artemis.tests.extras.byteman.RaceOnClosingConsumerWhileReconnecting.waitForReconnection();")})
+      rules = {
+         @BMRule(
+            name = "session.removeConsumer wait",
+            targetClass = "org.apache.activemq.artemis.core.client.impl.ClientSessionImpl",
+            targetMethod = "removeConsumer(org.apache.activemq.artemis.core.client.impl.ClientConsumerInternal)",
+            targetLocation = "ENTRY",
+            action = "org.apache.activemq.artemis.tests.extras.byteman.RaceOnClosingConsumerWhileReconnecting.waitForReconnection();")})
    public void testClosingConsumerBeforeReconnecting() throws Exception {
       conn = session.getConnection();
 
@@ -119,12 +120,13 @@ public class RaceOnClosingConsumerWhileReconnecting extends ActiveMQTestBase {
 
    @Test
    @BMRules(
-      rules = {@BMRule(
-         name = "session.closeConsumer before recreating consumer",
-         targetClass = "org.apache.activemq.artemis.core.client.impl.ClientSessionImpl",
-         targetMethod = "handleFailover",
-         targetLocation = "AFTER WRITE $consumerInternal 1",
-         action = "org.apache.activemq.artemis.tests.extras.byteman.RaceOnClosingConsumerWhileReconnecting.closeConsumer();")})
+      rules = {
+         @BMRule(
+            name = "session.closeConsumer before recreating consumer",
+            targetClass = "org.apache.activemq.artemis.core.client.impl.ClientSessionImpl",
+            targetMethod = "handleFailover",
+            targetLocation = "AFTER WRITE $consumerInternal 1",
+            action = "org.apache.activemq.artemis.tests.extras.byteman.RaceOnClosingConsumerWhileReconnecting.closeConsumer();")})
    public void testClosingConsumerBeforeRecreatingOneConsumer() throws Exception {
       RemotingConnection conn = session.getConnection();
 
@@ -139,12 +141,13 @@ public class RaceOnClosingConsumerWhileReconnecting extends ActiveMQTestBase {
 
    @Test
    @BMRules(
-      rules = {@BMRule(
-         name = "session.closeConsumer before recreating consumer",
-         targetClass = "org.apache.activemq.artemis.core.client.impl.ClientSessionImpl",
-         targetMethod = "handleFailover",
-         targetLocation = "AFTER WRITE $consumerInternal 1",
-         action = "org.apache.activemq.artemis.tests.extras.byteman.RaceOnClosingConsumerWhileReconnecting.closeConsumer();")})
+      rules = {
+         @BMRule(
+            name = "session.closeConsumer before recreating consumer",
+            targetClass = "org.apache.activemq.artemis.core.client.impl.ClientSessionImpl",
+            targetMethod = "handleFailover",
+            targetLocation = "AFTER WRITE $consumerInternal 1",
+            action = "org.apache.activemq.artemis.tests.extras.byteman.RaceOnClosingConsumerWhileReconnecting.closeConsumer();")})
    public void testClosingConsumerBeforeRecreatingTwoConsumers() throws Exception {
       RemotingConnection conn = session.getConnection();
 
diff --git a/tests/extra-tests/src/test/java/org/apache/activemq/artemis/tests/extras/byteman/RaceOnCursorIteratorTest.java b/tests/extra-tests/src/test/java/org/apache/activemq/artemis/tests/extras/byteman/RaceOnCursorIteratorTest.java
index 7196923..0c2cba8 100644
--- a/tests/extra-tests/src/test/java/org/apache/activemq/artemis/tests/extras/byteman/RaceOnCursorIteratorTest.java
+++ b/tests/extra-tests/src/test/java/org/apache/activemq/artemis/tests/extras/byteman/RaceOnCursorIteratorTest.java
@@ -143,12 +143,13 @@ public class RaceOnCursorIteratorTest extends ActiveMQTestBase {
 
    @Test
    @BMRules(
-      rules = {@BMRule(
-         name = "raceLiveCache",
-         targetClass = "org.apache.activemq.artemis.core.paging.cursor.impl.PageSubscriptionImpl",
-         targetMethod = "moveNextPage",
-         targetLocation = "EXIT",
-         action = "org.apache.activemq.artemis.tests.extras.byteman.RaceOnCursorIteratorTest.raceAddLivePageCache()")})
+      rules = {
+         @BMRule(
+            name = "raceLiveCache",
+            targetClass = "org.apache.activemq.artemis.core.paging.cursor.impl.PageSubscriptionImpl",
+            targetMethod = "moveNextPage",
+            targetLocation = "EXIT",
+            action = "org.apache.activemq.artemis.tests.extras.byteman.RaceOnCursorIteratorTest.raceAddLivePageCache()")})
    public void testSkipLivePageCache() {
       skipLivePageCache = true;
       // Simulate scenario #1 depicted in https://issues.apache.org/jira/browse/ARTEMIS-2418
@@ -159,12 +160,13 @@ public class RaceOnCursorIteratorTest extends ActiveMQTestBase {
 
    @Test
    @BMRules(
-      rules = {@BMRule(
-         name = "raceLiveCache",
-         targetClass = "org.apache.activemq.artemis.core.paging.cursor.impl.PageSubscriptionImpl",
-         targetMethod = "moveNextPage",
-         targetLocation = "EXIT",
-         action = "org.apache.activemq.artemis.tests.extras.byteman.RaceOnCursorIteratorTest.raceAddLivePageCache()"),
+      rules = {
+         @BMRule(
+            name = "raceLiveCache",
+            targetClass = "org.apache.activemq.artemis.core.paging.cursor.impl.PageSubscriptionImpl",
+            targetMethod = "moveNextPage",
+            targetLocation = "EXIT",
+            action = "org.apache.activemq.artemis.tests.extras.byteman.RaceOnCursorIteratorTest.raceAddLivePageCache()"),
          @BMRule(
             name = "raceNullCache",
             targetClass = "org.apache.activemq.artemis.core.paging.cursor.impl.PageCursorProviderImpl",
diff --git a/tests/extra-tests/src/test/java/org/apache/activemq/artemis/tests/extras/byteman/RaceOnSyncLargeMessageOverReplication2Test.java b/tests/extra-tests/src/test/java/org/apache/activemq/artemis/tests/extras/byteman/RaceOnSyncLargeMessageOverReplication2Test.java
index 3d3dc71..ea8d628 100644
--- a/tests/extra-tests/src/test/java/org/apache/activemq/artemis/tests/extras/byteman/RaceOnSyncLargeMessageOverReplication2Test.java
+++ b/tests/extra-tests/src/test/java/org/apache/activemq/artemis/tests/extras/byteman/RaceOnSyncLargeMessageOverReplication2Test.java
@@ -160,17 +160,19 @@ public class RaceOnSyncLargeMessageOverReplication2Test extends ActiveMQTestBase
 
    @Test
    @BMRules(
-      rules = {@BMRule(
-         name = "InterruptSending",
-         targetClass = "org.apache.activemq.artemis.core.protocol.core.impl.ActiveMQSessionContext",
-         targetMethod = "sendLargeMessageChunk",
-         targetLocation = "ENTRY",
-         action = "org.apache.activemq.artemis.tests.extras.byteman.RaceOnSyncLargeMessageOverReplication2Test.messageChunkSent();"), @BMRule(
-         name = "InterruptSync",
-         targetClass = "org.apache.activemq.artemis.core.persistence.impl.journal.JournalStorageManager",
-         targetMethod = "sendLargeMessageFiles",
-         targetLocation = "ENTRY",
-         action = "org.apache.activemq.artemis.tests.extras.byteman.RaceOnSyncLargeMessageOverReplication2Test.syncLargeMessage();")})
+      rules = {
+         @BMRule(
+            name = "InterruptSending",
+            targetClass = "org.apache.activemq.artemis.core.protocol.core.impl.ActiveMQSessionContext",
+            targetMethod = "sendLargeMessageChunk",
+            targetLocation = "ENTRY",
+            action = "org.apache.activemq.artemis.tests.extras.byteman.RaceOnSyncLargeMessageOverReplication2Test.messageChunkSent();"),
+         @BMRule(
+            name = "InterruptSync",
+            targetClass = "org.apache.activemq.artemis.core.persistence.impl.journal.JournalStorageManager",
+            targetMethod = "sendLargeMessageFiles",
+            targetLocation = "ENTRY",
+            action = "org.apache.activemq.artemis.tests.extras.byteman.RaceOnSyncLargeMessageOverReplication2Test.syncLargeMessage();")})
    public void testSendLargeMessage() throws Exception {
 
       final CountDownLatch failedOver = new CountDownLatch(1);
diff --git a/tests/extra-tests/src/test/java/org/apache/activemq/artemis/tests/extras/byteman/RaceOnSyncLargeMessageOverReplicationTest.java b/tests/extra-tests/src/test/java/org/apache/activemq/artemis/tests/extras/byteman/RaceOnSyncLargeMessageOverReplicationTest.java
index e5d60b3..e804232 100644
--- a/tests/extra-tests/src/test/java/org/apache/activemq/artemis/tests/extras/byteman/RaceOnSyncLargeMessageOverReplicationTest.java
+++ b/tests/extra-tests/src/test/java/org/apache/activemq/artemis/tests/extras/byteman/RaceOnSyncLargeMessageOverReplicationTest.java
@@ -150,12 +150,13 @@ public class RaceOnSyncLargeMessageOverReplicationTest extends ActiveMQTestBase
   */
    @Test
    @BMRules(
-      rules = {@BMRule(
-         name = "InterruptSync",
-         targetClass = "org.apache.activemq.artemis.core.persistence.impl.journal.JournalStorageManager",
-         targetMethod = "createLargeMessage(long,org.apache.activemq.artemis.core.message.impl.MessageInternal)",
-         targetLocation = "EXIT",
-         action = "org.apache.activemq.artemis.tests.extras.byteman.RaceOnSyncLargeMessageOverReplicationTest.syncLargeMessage();")})
+      rules = {
+         @BMRule(
+            name = "InterruptSync",
+            targetClass = "org.apache.activemq.artemis.core.persistence.impl.journal.JournalStorageManager",
+            targetMethod = "createLargeMessage(long,org.apache.activemq.artemis.core.message.impl.MessageInternal)",
+            targetLocation = "EXIT",
+            action = "org.apache.activemq.artemis.tests.extras.byteman.RaceOnSyncLargeMessageOverReplicationTest.syncLargeMessage();")})
    public void testSendLargeMessage() throws Exception {
 
       final CountDownLatch failedOver = new CountDownLatch(1);
diff --git a/tests/extra-tests/src/test/java/org/apache/activemq/artemis/tests/extras/byteman/ReplicationBackupTest.java b/tests/extra-tests/src/test/java/org/apache/activemq/artemis/tests/extras/byteman/ReplicationBackupTest.java
index b1ff797..631206c 100644
--- a/tests/extra-tests/src/test/java/org/apache/activemq/artemis/tests/extras/byteman/ReplicationBackupTest.java
+++ b/tests/extra-tests/src/test/java/org/apache/activemq/artemis/tests/extras/byteman/ReplicationBackupTest.java
@@ -43,12 +43,13 @@ public class ReplicationBackupTest extends ActiveMQTestBase {
    */
    @Test
    @BMRules(
-      rules = {@BMRule(
-         name = "prevent backup annoucement",
-         targetClass = "org.apache.activemq.artemis.core.server.impl.SharedNothingLiveActivation",
-         targetMethod = "run",
-         targetLocation = "AT EXIT",
-         action = "org.apache.activemq.artemis.tests.extras.byteman.ReplicationBackupTest.breakIt();")})
+      rules = {
+         @BMRule(
+            name = "prevent backup annoucement",
+            targetClass = "org.apache.activemq.artemis.core.server.impl.SharedNothingLiveActivation",
+            targetMethod = "run",
+            targetLocation = "AT EXIT",
+            action = "org.apache.activemq.artemis.tests.extras.byteman.ReplicationBackupTest.breakIt();")})
    public void testReplicatedBackupAnnouncement() throws Exception {
       TransportConfiguration liveConnector = TransportConfigurationUtils.getNettyConnector(true, 0);
       TransportConfiguration liveAcceptor = TransportConfigurationUtils.getNettyAcceptor(true, 0);
diff --git a/tests/extra-tests/src/test/java/org/apache/activemq/artemis/tests/extras/byteman/StartStopDeadlockTest.java b/tests/extra-tests/src/test/java/org/apache/activemq/artemis/tests/extras/byteman/StartStopDeadlockTest.java
index 8bb83f6..2c7bdb3 100644
--- a/tests/extra-tests/src/test/java/org/apache/activemq/artemis/tests/extras/byteman/StartStopDeadlockTest.java
+++ b/tests/extra-tests/src/test/java/org/apache/activemq/artemis/tests/extras/byteman/StartStopDeadlockTest.java
@@ -52,23 +52,26 @@ public class StartStopDeadlockTest extends ActiveMQTestBase {
    @Test
    @BMRules(
 
-      rules = {@BMRule(
-         name = "Server.start wait-init",
-         targetClass = "org.apache.activemq.artemis.core.server.impl.ActiveMQServerImpl",
-         targetMethod = "initialisePart2",
-         targetLocation = "ENTRY",
-         condition = "incrementCounter(\"server-Init\") == 2",
-         action = "org.apache.activemq.artemis.tests.extras.byteman.StartStopDeadlockTest.debugLog(\"server backup init\"), waitFor(\"start-init\")"), @BMRule(
-         name = "JMSServer.stop wait-init",
-         targetClass = "org.apache.activemq.artemis.jms.server.impl.JMSServerManagerImpl",
-         targetMethod = "stop",
-         targetLocation = "ENTRY",
-         action = "signalWake(\"start-init\", true)"), @BMRule(
-         name = "StartStopDeadlockTest tearDown",
-         targetClass = "org.apache.activemq.artemis.tests.extras.byteman.StartStopDeadlockTest",
-         targetMethod = "tearDown",
-         targetLocation = "ENTRY",
-         action = "deleteCounter(\"server-Init\")")})
+      rules = {
+         @BMRule(
+            name = "Server.start wait-init",
+            targetClass = "org.apache.activemq.artemis.core.server.impl.ActiveMQServerImpl",
+            targetMethod = "initialisePart2",
+            targetLocation = "ENTRY",
+            condition = "incrementCounter(\"server-Init\") == 2",
+            action = "org.apache.activemq.artemis.tests.extras.byteman.StartStopDeadlockTest.debugLog(\"server backup init\"), waitFor(\"start-init\")"),
+         @BMRule(
+            name = "JMSServer.stop wait-init",
+            targetClass = "org.apache.activemq.artemis.jms.server.impl.JMSServerManagerImpl",
+            targetMethod = "stop",
+            targetLocation = "ENTRY",
+            action = "signalWake(\"start-init\", true)"),
+         @BMRule(
+            name = "StartStopDeadlockTest tearDown",
+            targetClass = "org.apache.activemq.artemis.tests.extras.byteman.StartStopDeadlockTest",
+            targetMethod = "tearDown",
+            targetLocation = "ENTRY",
+            action = "deleteCounter(\"server-Init\")")})
    public void testDeadlock() throws Exception {
 
       // A live server that will always be crashed
diff --git a/tests/extra-tests/src/test/java/org/apache/activemq/artemis/tests/extras/byteman/StompInternalStateTest.java b/tests/extra-tests/src/test/java/org/apache/activemq/artemis/tests/extras/byteman/StompInternalStateTest.java
index a336cd8..44db34c 100644
--- a/tests/extra-tests/src/test/java/org/apache/activemq/artemis/tests/extras/byteman/StompInternalStateTest.java
+++ b/tests/extra-tests/src/test/java/org/apache/activemq/artemis/tests/extras/byteman/StompInternalStateTest.java
@@ -51,13 +51,14 @@ public class StompInternalStateTest extends ActiveMQTestBase {
 
    @Test
    @BMRules(
-      rules = {@BMRule(
-         name = "StompProtocolManager Leak Server Rule",
-         targetClass = "org.apache.activemq.artemis.core.protocol.stomp.StompProtocolManager",
-         targetMethod = "onNotification(org.apache.activemq.artemis.core.server.management.Notification)",
-         targetLocation = "EXIT",
-         helper = "org.apache.activemq.artemis.tests.extras.byteman.StompInternalStateTest",
-         action = "verifyBindingAddRemove($1, $0.destinations)")})
+      rules = {
+         @BMRule(
+            name = "StompProtocolManager Leak Server Rule",
+            targetClass = "org.apache.activemq.artemis.core.protocol.stomp.StompProtocolManager",
+            targetMethod = "onNotification(org.apache.activemq.artemis.core.server.management.Notification)",
+            targetLocation = "EXIT",
+            helper = "org.apache.activemq.artemis.tests.extras.byteman.StompInternalStateTest",
+            action = "verifyBindingAddRemove($1, $0.destinations)")})
    public void testStompProtocolManagerLeak() throws Exception {
       ClientSession session = null;
       try {
diff --git a/tests/extra-tests/src/test/java/org/apache/activemq/artemis/tests/extras/byteman/TimeoutXATest.java b/tests/extra-tests/src/test/java/org/apache/activemq/artemis/tests/extras/byteman/TimeoutXATest.java
index 6afccd8..db5730b 100644
--- a/tests/extra-tests/src/test/java/org/apache/activemq/artemis/tests/extras/byteman/TimeoutXATest.java
+++ b/tests/extra-tests/src/test/java/org/apache/activemq/artemis/tests/extras/byteman/TimeoutXATest.java
@@ -82,19 +82,21 @@ public class TimeoutXATest extends ActiveMQTestBase {
 
    @Test
    @BMRules(
-      rules = {@BMRule(
-         name = "removing TX",
-         targetClass = "org.apache.activemq.artemis.core.transaction.impl.ResourceManagerImpl",
-         targetMethod = "removeTransaction(javax.transaction.xa.Xid)",
-         targetLocation = "ENTRY",
-         helper = "org.apache.activemq.artemis.tests.extras.byteman.TimeoutXATest",
-         action = "removingTX()"), @BMRule(
-         name = "afterRollback TX",
-         targetClass = "org.apache.activemq.artemis.core.transaction.impl.TransactionImpl",
-         targetMethod = "afterRollback",
-         targetLocation = "ENTRY",
-         helper = "org.apache.activemq.artemis.tests.extras.byteman.TimeoutXATest",
-         action = "afterRollback()")})
+      rules = {
+         @BMRule(
+            name = "removing TX",
+            targetClass = "org.apache.activemq.artemis.core.transaction.impl.ResourceManagerImpl",
+            targetMethod = "removeTransaction(javax.transaction.xa.Xid)",
+            targetLocation = "ENTRY",
+            helper = "org.apache.activemq.artemis.tests.extras.byteman.TimeoutXATest",
+            action = "removingTX()"),
+         @BMRule(
+            name = "afterRollback TX",
+            targetClass = "org.apache.activemq.artemis.core.transaction.impl.TransactionImpl",
+            targetMethod = "afterRollback",
+            targetLocation = "ENTRY",
+            helper = "org.apache.activemq.artemis.tests.extras.byteman.TimeoutXATest",
+            action = "afterRollback()")})
    public void testTimeoutOnTX2() throws Exception {
       ActiveMQConnectionFactory connectionFactory = new ActiveMQConnectionFactory();
       XAConnection connection = connectionFactory.createXAConnection();
diff --git a/tests/extra-tests/src/test/java/org/apache/activemq/artemis/tests/extras/byteman/critical/analyzer/FileSystemSyncBlockedTest.java b/tests/extra-tests/src/test/java/org/apache/activemq/artemis/tests/extras/byteman/critical/analyzer/FileSystemSyncBlockedTest.java
index c717f9f..0cd2b87 100644
--- a/tests/extra-tests/src/test/java/org/apache/activemq/artemis/tests/extras/byteman/critical/analyzer/FileSystemSyncBlockedTest.java
+++ b/tests/extra-tests/src/test/java/org/apache/activemq/artemis/tests/extras/byteman/critical/analyzer/FileSystemSyncBlockedTest.java
@@ -33,11 +33,21 @@ import org.junit.runner.RunWith;
 public class FileSystemSyncBlockedTest extends CriticalAnalyzerFaultInjectionTestBase {
    private static final Logger log = Logger.getLogger(FileSystemSyncBlockedTest.class);
 
-   @BMRules(rules = {@BMRule(name = "Simulate Slow Disk Sync", targetClass = "org.apache.activemq.artemis.core.io.nio.NIOSequentialFile", targetMethod = "sync", targetLocation = "ENTRY", condition = "!flagged(\"testSlowDiskSync\")",  // Once the server shutdowns we stop applying this rule.
-      action = "waitFor(\"testSlowDiskSync\")"), @BMRule(
-      // We ensure that no more
-      name = "Release Suspended Thread during Server Shutdown", // Releases wakes up suspended threads to allow shutdown to complete
-      targetClass = "org.apache.activemq.artemis.core.server.impl.ActiveMQServerImpl", targetMethod = "stop", targetLocation = "ENTRY", action = "flag(\"testSlowDiskSync\"); signalWake(\"testSlowDiskSync\")")})
+   @BMRules(rules = {
+      @BMRule(
+         name = "Simulate Slow Disk Sync",
+         targetClass = "org.apache.activemq.artemis.core.io.nio.NIOSequentialFile",
+         targetMethod = "sync",
+         targetLocation = "ENTRY",
+         condition = "!flagged(\"testSlowDiskSync\")",  // Once the server shutdowns we stop applying this rule.
+         action = "waitFor(\"testSlowDiskSync\")"),
+      @BMRule(
+         // We ensure that no more
+         name = "Release Suspended Thread during Server Shutdown", // Releases wakes up suspended threads to allow shutdown to complete
+         targetClass = "org.apache.activemq.artemis.core.server.impl.ActiveMQServerImpl",
+         targetMethod = "stop",
+         targetLocation = "ENTRY",
+         action = "flag(\"testSlowDiskSync\"); signalWake(\"testSlowDiskSync\")")})
    @Test(timeout = 60000)
    public void testSlowDiskSync() throws Exception {
       testSendDurableMessage();
diff --git a/tests/integration-tests/src/test/java/org/apache/activemq/artemis/tests/integration/amqp/SaslKrb5LDAPSecurityTest.java b/tests/integration-tests/src/test/java/org/apache/activemq/artemis/tests/integration/amqp/SaslKrb5LDAPSecurityTest.java
index 4f61016..c82a15a 100644
--- a/tests/integration-tests/src/test/java/org/apache/activemq/artemis/tests/integration/amqp/SaslKrb5LDAPSecurityTest.java
+++ b/tests/integration-tests/src/test/java/org/apache/activemq/artemis/tests/integration/amqp/SaslKrb5LDAPSecurityTest.java
@@ -107,8 +107,8 @@ import static org.apache.activemq.artemis.tests.util.ActiveMQTestBase.NETTY_ACCE
 @RunWith(FrameworkRunner.class)
 @CreateDS(name = "Example",
    partitions = {@CreatePartition(name = "example", suffix = "dc=example,dc=com",
-      contextEntry = @ContextEntry(entryLdif = "dn: dc=example,dc=com\n" + "dc: example\n" + "objectClass: top\n" + "objectClass: domain\n\n"),
-      indexes = {@CreateIndex(attribute = "objectClass"), @CreateIndex(attribute = "dc"), @CreateIndex(attribute = "ou")})},
+         contextEntry = @ContextEntry(entryLdif = "dn: dc=example,dc=com\n" + "dc: example\n" + "objectClass: top\n" + "objectClass: domain\n\n"),
+         indexes = {@CreateIndex(attribute = "objectClass"), @CreateIndex(attribute = "dc"), @CreateIndex(attribute = "ou")})},
       additionalInterceptors = { KeyDerivationInterceptor.class })
 
 @CreateLdapServer(transports = {@CreateTransport(protocol = "LDAP", port = 1024)},
diff --git a/tests/integration-tests/src/test/java/org/apache/activemq/artemis/tests/integration/cluster/bridge/BridgeTest.java b/tests/integration-tests/src/test/java/org/apache/activemq/artemis/tests/integration/cluster/bridge/BridgeTest.java
index de5ce7e..96b0446 100644
--- a/tests/integration-tests/src/test/java/org/apache/activemq/artemis/tests/integration/cluster/bridge/BridgeTest.java
+++ b/tests/integration-tests/src/test/java/org/apache/activemq/artemis/tests/integration/cluster/bridge/BridgeTest.java
@@ -1494,7 +1494,7 @@ public class BridgeTest extends ActiveMQTestBase {
             msgCount.incrementAndGet();
 
             if (i % 500 == 0)
-              instanceLog.debug("received " + i);
+               instanceLog.debug("received " + i);
          }
 
          boolean failed = false;

Mime
View raw message