This is an automated email from the ASF dual-hosted git repository. robbie pushed a commit to branch new-logging in repository https://gitbox.apache.org/repos/asf/activemq-artemis.git
commit 6c6d8b238f65050ec2279e9c3c6afb7f2aadb5ae Author: Robbie Gemmell <[email protected]> AuthorDate: Mon Aug 15 16:10:22 2022 +0100 remove rather than reinstate a couple of unnecesarry level manipulations --- .../activemq/artemis/tests/integration/mqtt5/MQTT5TestSupport.java | 2 -- .../activemq/artemis/tests/integration/stomp/v12/StompV12Test.java | 2 -- 2 files changed, 4 deletions(-) diff --git a/tests/integration-tests/src/test/java/org/apache/activemq/artemis/tests/integration/mqtt5/MQTT5TestSupport.java b/tests/integration-tests/src/test/java/org/apache/activemq/artemis/tests/integration/mqtt5/MQTT5TestSupport.java index bc3cb6a3b2..f17d025c4e 100644 --- a/tests/integration-tests/src/test/java/org/apache/activemq/artemis/tests/integration/mqtt5/MQTT5TestSupport.java +++ b/tests/integration-tests/src/test/java/org/apache/activemq/artemis/tests/integration/mqtt5/MQTT5TestSupport.java @@ -166,8 +166,6 @@ public class MQTT5TestSupport extends ActiveMQTestBase { exceptions.clear(); startBroker(); createJMSConnection(); - //TODO: shouldnt really be doing this as it never resets the level - //TODO: org.jboss.logmanager.Logger.getLogger(MQTTUtil.class.getName()).setLevel(org.jboss.logmanager.Level.TRACE); } @Override diff --git a/tests/integration-tests/src/test/java/org/apache/activemq/artemis/tests/integration/stomp/v12/StompV12Test.java b/tests/integration-tests/src/test/java/org/apache/activemq/artemis/tests/integration/stomp/v12/StompV12Test.java index a0ab038e8c..0c8c2c3a08 100644 --- a/tests/integration-tests/src/test/java/org/apache/activemq/artemis/tests/integration/stomp/v12/StompV12Test.java +++ b/tests/integration-tests/src/test/java/org/apache/activemq/artemis/tests/integration/stomp/v12/StompV12Test.java @@ -2492,9 +2492,7 @@ public class StompV12Test extends StompTestBase { } @Test - @Ignore("Needs updated to account for logging impl changes") //TODO: reinstate public void testSubscribeWithNonZeroConsumerWindowSizeAndClientAck() throws Exception { - //TODO: Note it is also never reset: org.jboss.logmanager.Logger.getLogger(StompConnection.class.getName()).setLevel(org.jboss.logmanager.Level.DEBUG); // the size of each message was determined from the DEBUG logging from org.apache.activemq.artemis.core.protocol.stomp.StompConnection final int MESSAGE_SIZE = 270; final int TIMEOUT = 1000;
