Uploaded image for project: 'Camel'
  1. Camel
  2. CAMEL-11771

Remove commons-logging import

    XMLWordPrintableJSON

    Details

    • Type: Test
    • Status: Resolved
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 2.20.0
    • Component/s: None
    • Labels:
      None
    • Estimated Complexity:
      Novice

      Description

      We should use slf4j-api instead as there is a few unit test that uses commons-logging api directly.

      davsclaus:/Users/davsclaus/workspace/camel (boot2)/$ git grep "commons\.logging"
      components/camel-cxf/src/test/java/org/apache/camel/component/cxf/mtom/MtomTestHelper.java:    static boolean isAwtHeadless(org.apache.commons.logging.Log log, org.slf4j.Logger logger) {
      components/camel-jetty9/src/test/java/org/apache/camel/component/jetty/SpringHttpsRouteTest.java:import org.apache.commons.logging.Log;
      components/camel-jetty9/src/test/java/org/apache/camel/component/jetty/SpringHttpsRouteTest.java:import org.apache.commons.logging.LogFactory;
      components/camel-zookeeper-master/src/test/java/org/apache/camel/component/zookeepermaster/CuratorFactoryBean.java:import org.apache.commons.logging.Log;
      components/camel-zookeeper-master/src/test/java/org/apache/camel/component/zookeepermaster/CuratorFactoryBean.java:import org.apache.commons.logging.LogFactory;
      components/camel-zookeeper/src/test/java/org/apache/camel/component/zookeeper/policy/FailoverCuratorLeaderRoutePolicyTest.java:import org.apache.commons.logging.LogFactory;
      components/camel-zookeeper/src/test/java/org/apache/camel/component/zookeeper/policy/FailoverRoutePolicyTest.java:import org.apache.commons.logging.LogFactory;
      components/camel-zookeeper/src/test/java/org/apache/camel/component/zookeeper/policy/MultiMasterCuratorLeaderRoutePolicyTest.java:import org.apache.commons.logging.LogFactory;
      parent/pom.xml:      org.apache.commons.logging.*;version="[1.1,2)",
      tests/camel-itest-karaf/src/test/resources/commons-logging.properties:#org.apache.commons.logging.Log=org.apache.commons.logging.impl.Jdk14Logger
      tests/camel-itest-karaf/src/test/resources/commons-logging.properties:org.apache.commons.logging.Log=org.apache.commons.logging.impl.Log4JLogger
      tests/camel-itest/src/test/resources/commons-logging.properties:#org.apache.commons.logging.Log=org.apache.commons.logging.impl.Jdk14Logger
      tests/camel-itest/src/test/resources/commons-logging.properties:org.apache.commons.logging.Log=org.apache.commons.logging.impl.Log4JLogger
      

        Attachments

          Issue Links

            Activity

              People

              • Assignee:
                vrlgohel Viral Gohel
                Reporter:
                davsclaus Claus Ibsen
              • Votes:
                0 Vote for this issue
                Watchers:
                4 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: