[CAMEL-11771] Remove commons-logging import

• Type: Test
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 2.20.0
• Estimated Complexity: Novice
• Assignee: Viral Gohel
• Reporter: Claus Ibsen
• Created: 11/Sep/17 17:32
• Updated: 15/Sep/17 09:04
• Resolved: 14/Sep/17 17:51

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

Issue Links

• links to: GitHub Pull Request #1935