fix(#713): add missing transitive dependencies to camel-tika feature

features/src/main/feature/camel-features.xml

@@ -3473,6 +3473,10 @@ Chain 2:
         <bundle dependency='true'>mvn:org.apache.tika/tika-parser-html-module/${tika-version}</bundle>
         <bundle dependency='true'>mvn:org.apache.tika/tika-parser-text-module/${tika-version}</bundle>
         <bundle dependency='true'>mvn:commons-io/commons-io/${commons-io-version}</bundle>
+        <bundle dependency='true'>mvn:org.jsoup/jsoup/${jsoup-version}</bundle>
+        <bundle dependency='true'>mvn:commons-codec/commons-codec/${commons-codec-version}</bundle>
+        <bundle dependency='true'>mvn:org.apache.commons/commons-csv/${commons-csv-version}</bundle>
+        <bundle dependency='true'>wrap:mvn:com.github.albfernandez/juniversalchardet/${juniversalchardet-version}</bundle>
         <bundle>mvn:org.apache.camel.karaf/camel-tika/${project.version}</bundle>
     </feature>
     <feature name='camel-torchserve' version='${project.version}' start-level='50'>

pom.xml

@@ -373,7 +373,7 @@
         <junit-jupiter-version>5.13.4</junit-jupiter-version>
         <junit6-jupiter-version>6.0.1</junit6-jupiter-version>
         <junit-pioneer-version>2.3.0</junit-pioneer-version>
-        <juniversalchardet-version>1.0.3</juniversalchardet-version>
+        <juniversalchardet-version>2.5.0</juniversalchardet-version>
         <jxmpp-version>1.1.0</jxmpp-version>
         <jython-version>2.7.4</jython-version>
         <jzlib-version>1.1.3</jzlib-version>

tests/features/camel-tika/pom.xml

@@ -0,0 +1,48 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<!--
+
+    Licensed to the Apache Software Foundation (ASF) under one or more
+    contributor license agreements.  See the NOTICE file distributed with
+    this work for additional information regarding copyright ownership.
+    The ASF licenses this file to You under the Apache License, Version 2.0
+    (the "License"); you may not use this file except in compliance with
+    the License.  You may obtain a copy of the License at
+
+         http://www.apache.org/licenses/LICENSE-2.0
+
+    Unless required by applicable law or agreed to in writing, software
+    distributed under the License is distributed on an "AS IS" BASIS,
+    WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+    See the License for the specific language governing permissions and
+    limitations under the License.
+
+-->
+<project xmlns="http://maven.apache.org/POM/4.0.0" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/xsd/maven-4.0.0.xsd">
+    <modelVersion>4.0.0</modelVersion>
+    <parent>
+        <groupId>org.apache.camel.karaf</groupId>
+        <artifactId>camel-karaf-features-test</artifactId>
+        <version>4.18.2-SNAPSHOT</version>
+    </parent>
+
+    <artifactId>camel-tika-test</artifactId>
+    <name>Apache Camel :: Karaf :: Tests :: Features :: Tika</name>
+
+    <dependencies>
+        <dependency>
+            <groupId>org.apache.camel</groupId>
+            <artifactId>camel-core</artifactId>
+            <version>${camel-version}</version>
+            <scope>provided</scope>
+        </dependency>
+    </dependencies>
+
+    <build>
+        <resources>
+            <resource>
+                <directory>../../../src/main/resources</directory>
+                <filtering>false</filtering>
+            </resource>
+        </resources>
+    </build>
+</project>

tests/features/camel-tika/src/main/java/org/apache/karaf/camel/test/CamelTikaRouteSupplier.java

@@ -0,0 +1,48 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one or more
+ * contributor license agreements.  See the NOTICE file distributed with
+ * this work for additional information regarding copyright ownership.
+ * The ASF licenses this file to You under the Apache License, Version 2.0
+ * (the "License"); you may not use this file except in compliance with
+ * the License.  You may obtain a copy of the License at
+ *      http://www.apache.org/licenses/LICENSE-2.0
+ *
+ *  Unless required by applicable law or agreed to in writing, software
+ *  distributed under the License is distributed on an "AS IS" BASIS,
+ *  WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ *  See the License for the specific language governing permissions and
+ *  limitations under the License.
+ */
+package org.apache.karaf.camel.test;
+
+import org.apache.camel.builder.RouteBuilder;
+import org.apache.camel.model.RouteDefinition;
+import org.apache.karaf.camel.itests.AbstractCamelSingleFeatureResultMockBasedRouteSupplier;
+import org.apache.karaf.camel.itests.CamelRouteSupplier;
+import org.osgi.service.component.annotations.Component;
+
+@Component(
+        name = "karaf-camel-tika-test",
+        immediate = true,
+        service = CamelRouteSupplier.class
+)
+public class CamelTikaRouteSupplier extends AbstractCamelSingleFeatureResultMockBasedRouteSupplier {
+
+    @Override
+    protected boolean consumerEnabled() {
+        return false;
+    }
+
+    @Override
+    protected void configureProducer(RouteBuilder builder, RouteDefinition producerRoute) {
+        // Route the body through tika:parse. The value of this route is that it forces the
+        // camel-tika feature (including tika-parser-text-module and its juniversalchardet
+        // dependency, which were missing before issue #713) to resolve and run; a message
+        // reaching the mock proves the feature is now wired correctly.
+        producerRoute.log("Will parse: ${body}")
+                    .to("tika:parse?tikaParseOutputFormat=text")
+                    .convertBodyTo(String.class)
+                    .log("Parsed: ${body}")
+                    .toF("mock:%s", getResultMockName());
+    }
+}

tests/features/camel-tika/src/test/java/org/apache/karaf/camel/itest/CamelTikaITest.java

@@ -0,0 +1,57 @@
+/*
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ *      http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+package org.apache.karaf.camel.itest;
+
+import org.apache.camel.component.mock.MockEndpoint;
+import org.apache.karaf.camel.itests.AbstractCamelSingleFeatureResultMockBasedRouteITest;
+import org.junit.Test;
+import org.junit.runner.RunWith;
+import org.ops4j.pax.exam.junit.PaxExam;
+import org.ops4j.pax.exam.spi.reactors.ExamReactorStrategy;
+import org.ops4j.pax.exam.spi.reactors.PerClass;
+
+/**
+ * Verifies the {@code camel-tika} feature installs and a {@code tika:parse} route runs end-to-end
+ * (issue #713). The point of the test is that the feature now resolves: before the fix the
+ * {@code tika-parser-text-module} bundle failed to wire because its {@code juniversalchardet}
+ * dependency (package {@code org.mozilla.universalchardet}) was missing from the feature, so the
+ * route could never be created and no message would reach the mock.
+ * <p>
+ * The test deliberately does not assert on the extracted text: Tika's {@code AutoDetectParser}
+ * discovers parsers through the JDK {@link java.util.ServiceLoader}, which does not cross OSGi
+ * bundle boundaries, so {@code tika:parse} yields empty content in Karaf. Wiring Tika's parser SPI
+ * for OSGi is a separate concern beyond the scope of issue #713.
+ */
+@RunWith(PaxExam.class)
+@ExamReactorStrategy(PerClass.class)
+public class CamelTikaITest extends AbstractCamelSingleFeatureResultMockBasedRouteITest {
+
+    private static final String TEXT_SAMPLE = "The quick brown fox jumps over the lazy dog";
+
+    @Override
+    public String getBodyToSend() {
+        return TEXT_SAMPLE;
+    }
+
+    @Override
+    public void configureMock(MockEndpoint mock) {
+        // The feature resolves and the route processes exactly one exchange without error.
+        mock.expectedMessageCount(1);
+    }
+
+    @Test
+    public void testResultMock() throws Exception {
+        assertMockEndpointsSatisfied();
+    }
+}

tests/features/pom.xml

@@ -110,6 +110,7 @@
         <module>camel-quartz</module>
         <module>camel-saxon</module>
         <module>camel-spring-rabbitmq</module>
+        <module>camel-tika</module>
         <module>camel-velocity</module>
         <module>camel-weather</module>
         <module>camel-xslt-saxon</module>