ARTEMIS-4247 Fixing BrokerInSyncTest::testExpiryNoReaper

Once the context is set to disable mirror, it should stay that way even if reset is called.
apache · Apr 26, 2023 · c2ad54f · c2ad54f
1 parent 74fa4ca
commit c2ad54f
Showing 1 changed file with 6 additions and 1 deletion.
diff --git a/...server/src/main/java/org/apache/activemq/artemis/core/server/impl/RoutingContextImpl.java b/...server/src/main/java/org/apache/activemq/artemis/core/server/impl/RoutingContextImpl.java
@@ -167,7 +167,12 @@ public RoutingContextImpl clear() {
 
       this.internalOnly = null;
 
-      mirrorOption = MirrorOption.enabled;
+      // once we set to disabled, we keep it always disabled.
+      // This is because the routing object used to route commands will disable this
+      // and it should stay that way no matter what
+      if (mirrorOption != MirrorOption.disabled) {
+         mirrorOption = MirrorOption.enabled;
+      }
 
       return this;
    }