activemq-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From clebertsuconic <...@git.apache.org>
Subject [GitHub] activemq-artemis pull request #1874: ARTEMIS-1687 reduce logging for fast-te...
Date Mon, 19 Feb 2018 16:10:20 GMT
Github user clebertsuconic commented on a diff in the pull request:

    https://github.com/apache/activemq-artemis/pull/1874#discussion_r169118349
  
    --- Diff: tests/unit-tests/src/test/java/org/apache/activemq/artemis/tests/unit/ra/ActiveMQResourceAdapterConfigTest.java
---
    @@ -457,10 +456,7 @@ public void testConfiguration() throws Exception {
                 newConfig.append("         \"         <config-property-value></config-property-value>\"
+ \n");
                 newConfig.append("         \"      </config-property>\" + \n");
              }
    -         System.out.println(newConfig);
              fail("methods not shown please see previous and add");
    --- End diff --
    
    this is only if failed, right? can't we keep it? it will tell us what is the exactly config
that needs to be added (when only there's a failure)


---

Mime
View raw message