diff --git a/features/src/main/feature/camel-features.xml b/features/src/main/feature/camel-features.xml index 4796fedf5..74a15bb07 100644 --- a/features/src/main/feature/camel-features.xml +++ b/features/src/main/feature/camel-features.xml @@ -3472,6 +3472,10 @@ Chain 2: mvn:org.apache.tika/tika-parser-html-module/${tika-version} mvn:org.apache.tika/tika-parser-text-module/${tika-version} mvn:commons-io/commons-io/${commons-io-version} + mvn:org.jsoup/jsoup/${jsoup-version} + mvn:commons-codec/commons-codec/${commons-codec-version} + mvn:org.apache.commons/commons-csv/${commons-csv-version} + wrap:mvn:com.github.albfernandez/juniversalchardet/${juniversalchardet-version} mvn:org.apache.camel.karaf/camel-tika/${project.version} diff --git a/pom.xml b/pom.xml index d608d2404..a64ecbc78 100644 --- a/pom.xml +++ b/pom.xml @@ -374,7 +374,7 @@ 5.13.4 6.0.1 2.3.0 - 1.0.3 + 2.5.0 1.1.0 2.7.4 1.1.3 diff --git a/tests/features/camel-tika/pom.xml b/tests/features/camel-tika/pom.xml new file mode 100644 index 000000000..69a7a7443 --- /dev/null +++ b/tests/features/camel-tika/pom.xml @@ -0,0 +1,48 @@ + + + + 4.0.0 + + org.apache.camel.karaf + camel-karaf-features-test + 4.18.3-SNAPSHOT + + + camel-tika-test + Apache Camel :: Karaf :: Tests :: Features :: Tika + + + + org.apache.camel + camel-core + ${camel-version} + provided + + + + + + + ../../../src/main/resources + false + + + + diff --git a/tests/features/camel-tika/src/main/java/org/apache/karaf/camel/test/CamelTikaRouteSupplier.java b/tests/features/camel-tika/src/main/java/org/apache/karaf/camel/test/CamelTikaRouteSupplier.java new file mode 100644 index 000000000..b311c5ce2 --- /dev/null +++ b/tests/features/camel-tika/src/main/java/org/apache/karaf/camel/test/CamelTikaRouteSupplier.java @@ -0,0 +1,47 @@ +/* + * 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) { + // Parse the plain-text body as text. This exercises the tika-parser-text-module and + // its transitive juniversalchardet dependency (charset detection), which are the + // bundles that were missing from the camel-tika feature (issue #713). + producerRoute.log("Will parse: ${body}") + .to("tika:parse?tikaParseOutputFormat=text") + .convertBodyTo(String.class) + .log("Parsed: ${body}") + .toF("mock:%s", getResultMockName()); + } +} diff --git a/tests/features/camel-tika/src/test/java/org/apache/karaf/camel/itest/CamelTikaITest.java b/tests/features/camel-tika/src/test/java/org/apache/karaf/camel/itest/CamelTikaITest.java new file mode 100644 index 000000000..112ab39b3 --- /dev/null +++ b/tests/features/camel-tika/src/test/java/org/apache/karaf/camel/itest/CamelTikaITest.java @@ -0,0 +1,62 @@ +/* + * 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 static org.junit.Assert.assertNotNull; +import static org.junit.Assert.assertTrue; + +import java.util.List; + +import org.apache.camel.Exchange; +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; + +@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) { + mock.expectedMessageCount(1); + } + + @Test + public void testResultMock() throws Exception { + MockEndpoint endpoint = getMockEndpoint(); + List exchanges = endpoint.getExchanges(); + assertNotNull(exchanges); + + String parsed = exchanges.get(0).getIn().getBody(String.class); + assertNotNull(parsed); + // The text parser (backed by juniversalchardet for charset detection) must extract + // the original text content. Without the missing feature bundles this route would + // fail to resolve/parse at all. + assertTrue("Parsed content should contain the original text but was: " + parsed, + parsed.contains("quick brown fox")); + + assertMockEndpointsSatisfied(); + } +} diff --git a/tests/features/pom.xml b/tests/features/pom.xml index 03bce3137..9edef3078 100644 --- a/tests/features/pom.xml +++ b/tests/features/pom.xml @@ -110,6 +110,7 @@ camel-quartz camel-saxon camel-spring-rabbitmq + camel-tika camel-velocity camel-weather camel-xslt-saxon