Author: davsclaus
Date: Sun Jul 26 11:35:56 2009
New Revision: 797911
URL: http://svn.apache.org/viewvc?rev=797911&view=rev
Log:
CAMEL-1849: added unit test demonstrating the issue that the temp files is not
deleted in case of an http exception.
Added:
camel/branches/camel-1.x/components/camel-jetty/src/test/java/org/apache/camel/component/jetty/HttpStreamCacheFileTest.java
(with props)
Added:
camel/branches/camel-1.x/components/camel-jetty/src/test/java/org/apache/camel/component/jetty/HttpStreamCacheFileTest.java
URL:
http://svn.apache.org/viewvc/camel/branches/camel-1.x/components/camel-jetty/src/test/java/org/apache/camel/component/jetty/HttpStreamCacheFileTest.java?rev=797911&view=auto
==============================================================================
---
camel/branches/camel-1.x/components/camel-jetty/src/test/java/org/apache/camel/component/jetty/HttpStreamCacheFileTest.java
(added)
+++
camel/branches/camel-1.x/components/camel-jetty/src/test/java/org/apache/camel/component/jetty/HttpStreamCacheFileTest.java
Sun Jul 26 11:35:56 2009
@@ -0,0 +1,102 @@
+/**
+ * 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.jetty;
+
+import java.io.File;
+
+import org.apache.camel.ContextTestSupport;
+import org.apache.camel.Exchange;
+import org.apache.camel.Processor;
+import org.apache.camel.RuntimeCamelException;
+import org.apache.camel.builder.RouteBuilder;
+import org.apache.camel.component.http.HttpOperationFailedException;
+import org.apache.camel.component.http.HttpProducer;
+import org.apache.camel.converter.stream.CachedOutputStream;
+
+/**
+ * @version $Revision$
+ */
+public class HttpStreamCacheFileTest extends ContextTestSupport {
+
+ private String body = "12345678901234567890123456789012345678901234567890";
+
+ @Override
+ public void setUp() throws Exception {
+ deleteDirectory("./target/cachedir");
+ createDirectory("./target/cachedir");
+ super.setUp();
+ }
+
+ public void testStreamCacheToFileShouldBeDeletedInCaseOfResponse() throws
Exception {
+ Object response = template.requestBody("direct:start", "Hello World");
+ String out = context.getTypeConverter().convertTo(String.class,
response);
+ assertEquals("Bye World", out);
+
+ // the temporary files should have been deleted
+ File file = new File("./target/cachedir");
+ String[] files = file.list();
+ assertEquals("There should be no files", files.length, 0);
+ }
+
+ public void testStreamCacheToFileShouldBeDeletedInCaseOfException() throws
Exception {
+ try {
+ template.requestBody("direct:start", null);
+ fail("Should have thrown an exception");
+ } catch (RuntimeCamelException e) {
+ HttpOperationFailedException hofe =
assertIsInstanceOf(HttpOperationFailedException.class, e.getCause());
+ String s = context.getTypeConverter().convertTo(String.class,
hofe.getResponseBody());
+ assertEquals("Response body", body, s);
+ }
+
+ // the temporary files should have been deleted
+ File file = new File("./target/cachedir");
+ String[] files = file.list();
+ // TODO: CAMEL-1849 when fixed this one passes
+ //assertEquals("There should be no files", files.length, 0);
+ }
+
+ @Override
+ protected RouteBuilder createRouteBuilder() throws Exception {
+ return new RouteBuilder() {
+ @Override
+ public void configure() throws Exception {
+ // disable redeliveries
+ errorHandler(deadLetterChannel().maximumRedeliveries(0));
+
+ // enable stream caching and use a low threshold so its forced
to write to file
+ context.getProperties().put(CachedOutputStream.TEMP_DIR,
"./target/cachedir");
+ context.getProperties().put(CachedOutputStream.THRESHOLD,
"16");
+
+ // use a route so we got an unit of work
+ from("direct:start").to("http://localhost:8123/myserver";);
+
+ from("jetty://http://localhost:8123/myserver";)
+ .process(new Processor() {
+ public void process(Exchange exchange) throws
Exception {
+ if (exchange.getIn().getBody() == null) {
+
exchange.getOut().setBody("12345678901234567890123456789012345678901234567890");
+
exchange.getOut().setHeader(HttpProducer.HTTP_RESPONSE_CODE, 500);
+ } else {
+ exchange.getOut().setBody("Bye World");
+ }
+ }
+ });
+ }
+ };
+ }
+
+}
Propchange:
camel/branches/camel-1.x/components/camel-jetty/src/test/java/org/apache/camel/component/jetty/HttpStreamCacheFileTest.java
------------------------------------------------------------------------------
svn:eol-style = native
Propchange:
camel/branches/camel-1.x/components/camel-jetty/src/test/java/org/apache/camel/component/jetty/HttpStreamCacheFileTest.java
------------------------------------------------------------------------------
svn:keywords = Rev Date
