This is an automated email from the ASF dual-hosted git repository. davsclaus pushed a commit to branch main in repository https://gitbox.apache.org/repos/asf/camel.git
The following commit(s) were added to refs/heads/main by this push: new c929e96e0a3 CAMEL-22120: Added unit test c929e96e0a3 is described below commit c929e96e0a3077c968b183e3d68b3084113124de Author: Claus Ibsen <claus.ib...@gmail.com> AuthorDate: Wed Jun 18 11:47:53 2025 +0200 CAMEL-22120: Added unit test --- .../apache/camel/component/http/BaseHttpTest.java | 6 +- .../component/http/HttpGetContentTypeTest.java | 76 ++++++++++++++++++++++ 2 files changed, 81 insertions(+), 1 deletion(-) diff --git a/components/camel-http/src/test/java/org/apache/camel/component/http/BaseHttpTest.java b/components/camel-http/src/test/java/org/apache/camel/component/http/BaseHttpTest.java index 0a7e3238ab8..d84c1cc6e31 100644 --- a/components/camel-http/src/test/java/org/apache/camel/component/http/BaseHttpTest.java +++ b/components/camel-http/src/test/java/org/apache/camel/component/http/BaseHttpTest.java @@ -41,10 +41,14 @@ public abstract class BaseHttpTest extends HttpServerTestSupport { protected void assertHeaders(Map<String, Object> headers) { assertEquals(HttpStatus.SC_OK, headers.get(Exchange.HTTP_RESPONSE_CODE)); - assertEquals("12", headers.get(CONTENT_LENGTH)); + assertEquals(expectedContentLength(), headers.get(CONTENT_LENGTH)); assertNotNull(headers.get(CONTENT_TYPE), "Should have Content-Type header"); } + protected String expectedContentLength() { + return "12"; + } + protected void assertBody(String body) { assertEquals(getExpectedContent(), body); } diff --git a/components/camel-http/src/test/java/org/apache/camel/component/http/HttpGetContentTypeTest.java b/components/camel-http/src/test/java/org/apache/camel/component/http/HttpGetContentTypeTest.java new file mode 100644 index 00000000000..eee4a345e2c --- /dev/null +++ b/components/camel-http/src/test/java/org/apache/camel/component/http/HttpGetContentTypeTest.java @@ -0,0 +1,76 @@ +/* + * 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.camel.component.http; + +import org.apache.camel.Exchange; +import org.apache.hc.core5.http.Header; +import org.apache.hc.core5.http.HttpStatus; +import org.apache.hc.core5.http.impl.bootstrap.HttpServer; +import org.apache.hc.core5.http.impl.bootstrap.ServerBootstrap; +import org.apache.hc.core5.http.io.entity.StringEntity; +import org.junit.jupiter.api.Test; + +import static org.junit.jupiter.api.Assertions.assertEquals; + +public class HttpGetContentTypeTest extends BaseHttpTest { + + private HttpServer localServer; + + @Override + public void setupResources() throws Exception { + localServer = ServerBootstrap.bootstrap() + .setCanonicalHostName("localhost").setHttpProcessor(getBasicHttpProcessor()) + .setConnectionReuseStrategy(getConnectionReuseStrategy()).setResponseFactory(getHttpResponseFactory()) + .setSslContext(getSSLContext()) + .register("/myget", (request, response, context) -> { + Header ctHeader = request.getFirstHeader(Exchange.CONTENT_TYPE); + String ct = ctHeader != null ? ctHeader.getValue() : ""; + assertEquals("application/json", ct); + response.setEntity(new StringEntity(getExpectedContent())); + response.setCode(HttpStatus.SC_OK); + }).create(); + localServer.start(); + } + + @Override + public void cleanupResources() throws Exception { + if (localServer != null) { + localServer.stop(); + } + } + + @Test + public void sendGetContentType() { + Exchange exchange = template.request( + "http://localhost:" + localServer.getLocalPort() + "/myget?getWithBody=true", exchange1 -> { + exchange1.getIn().setHeader(Exchange.CONTENT_TYPE, "application/json"); + exchange1.getIn().setBody(""); + }); + + assertExchange(exchange); + } + + @Override + protected String expectedContentLength() { + return "19"; + } + + @Override + protected String getExpectedContent() { + return "CT=application/json"; + } +}