svn commit: r959576 - in /camel/trunk/camel-core/src/test/java/org/apache/camel: component/file/ processor/aggregator/

[davsclaus]

Author: davsclaus
Date: Thu Jul  1 09:48:22 2010
New Revision: 959576

URL: http://svn.apache.org/viewvc?rev=959576&view=rev
Log:
Polished

Modified:
    
camel/trunk/camel-core/src/test/java/org/apache/camel/component/file/FileAbsoluteMoveDefaultIssueManual.java
    
camel/trunk/camel-core/src/test/java/org/apache/camel/component/file/FileBatchConsumerMemoryLeakTest.java
    
camel/trunk/camel-core/src/test/java/org/apache/camel/component/file/FileConsumerAbsoluteRootPathDefaultMoveTest.java
    
camel/trunk/camel-core/src/test/java/org/apache/camel/processor/aggregator/AggregatorExceptionInPredicateTest.java

Modified: 
camel/trunk/camel-core/src/test/java/org/apache/camel/component/file/FileAbsoluteMoveDefaultIssueManual.java
URL: 
http://svn.apache.org/viewvc/camel/trunk/camel-core/src/test/java/org/apache/camel/component/file/FileAbsoluteMoveDefaultIssueManual.java?rev=959576&r1=959575&r2=959576&view=diff
==============================================================================
--- 
camel/trunk/camel-core/src/test/java/org/apache/camel/component/file/FileAbsoluteMoveDefaultIssueManual.java
 (original)
+++ 
camel/trunk/camel-core/src/test/java/org/apache/camel/component/file/FileAbsoluteMoveDefaultIssueManual.java
 Thu Jul  1 09:48:22 2010
@@ -27,7 +27,7 @@ import org.apache.camel.language.simple.
  */
 public class FileAbsoluteMoveDefaultIssueManual extends ContextTestSupport {
 
-    // TODO run this test manually
+    // run this test manually
 
     @Override
     protected void setUp() throws Exception {

Modified: 
camel/trunk/camel-core/src/test/java/org/apache/camel/component/file/FileBatchConsumerMemoryLeakTest.java
URL: 
http://svn.apache.org/viewvc/camel/trunk/camel-core/src/test/java/org/apache/camel/component/file/FileBatchConsumerMemoryLeakTest.java?rev=959576&r1=959575&r2=959576&view=diff
==============================================================================
--- 
camel/trunk/camel-core/src/test/java/org/apache/camel/component/file/FileBatchConsumerMemoryLeakTest.java
 (original)
+++ 
camel/trunk/camel-core/src/test/java/org/apache/camel/component/file/FileBatchConsumerMemoryLeakTest.java
 Thu Jul  1 09:48:22 2010
@@ -57,7 +57,7 @@ public class FileBatchConsumerMemoryLeak
      * or succeed fast
      */
     public void xxxtestMemoryLeak() throws Exception {
-        // TODO: Run this manually and browse the memory usage, eg in IDEA 
there is a Statistics tab
+        // run this manually and browse the memory usage, eg in IDEA there is 
a Statistics tab
 
         deleteDirectory("target/filesorter/archiv");
         for (int c = 0; c < 100; c++) {

Modified: 
camel/trunk/camel-core/src/test/java/org/apache/camel/component/file/FileConsumerAbsoluteRootPathDefaultMoveTest.java
URL: 
http://svn.apache.org/viewvc/camel/trunk/camel-core/src/test/java/org/apache/camel/component/file/FileConsumerAbsoluteRootPathDefaultMoveTest.java?rev=959576&r1=959575&r2=959576&view=diff
==============================================================================
--- 
camel/trunk/camel-core/src/test/java/org/apache/camel/component/file/FileConsumerAbsoluteRootPathDefaultMoveTest.java
 (original)
+++ 
camel/trunk/camel-core/src/test/java/org/apache/camel/component/file/FileConsumerAbsoluteRootPathDefaultMoveTest.java
 Thu Jul  1 09:48:22 2010
@@ -26,13 +26,14 @@ import org.apache.camel.component.mock.M
  */
 public class FileConsumerAbsoluteRootPathDefaultMoveTest extends 
ContextTestSupport {
 
+    // run this test manually
+
     private String base = "/tmp/mytemp";
 
     public void testDummy() {
         // noop
     }
 
-    // TODO: Enable to test locally
     public void xxxTestConsumeFromAbsolutePath() throws Exception {
         deleteDirectory(base);
 

Modified: 
camel/trunk/camel-core/src/test/java/org/apache/camel/processor/aggregator/AggregatorExceptionInPredicateTest.java
URL: 
http://svn.apache.org/viewvc/camel/trunk/camel-core/src/test/java/org/apache/camel/processor/aggregator/AggregatorExceptionInPredicateTest.java?rev=959576&r1=959575&r2=959576&view=diff
==============================================================================
--- 
camel/trunk/camel-core/src/test/java/org/apache/camel/processor/aggregator/AggregatorExceptionInPredicateTest.java
 (original)
+++ 
camel/trunk/camel-core/src/test/java/org/apache/camel/processor/aggregator/AggregatorExceptionInPredicateTest.java
 Thu Jul  1 09:48:22 2010
@@ -42,11 +42,7 @@ public class AggregatorExceptionInPredic
     private void testExceptionInFlow(String startUri) throws Exception {
         // failed first aggregation
         
-        // TODO Following assertion should be true while it is not. Instead
-        // exception handler set in BatchProcessor is used and it logs
-        // by default.
-        
-        // getMockEndpoint("mock:handled").expectedMessageCount(1);
+        getMockEndpoint("mock:handled").expectedMessageCount(1);
 
         // second aggregated
         getMockEndpoint("mock:result").expectedBodiesReceived("Hello World");