You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@camel.apache.org by da...@apache.org on 2015/11/29 10:02:13 UTC

[2/2] camel git commit: CAMEL-9375: camel-tarfile - TarSplitter includes one extra empty entry at the end

CAMEL-9375: camel-tarfile - TarSplitter includes one extra empty entry at the end


Project: http://git-wip-us.apache.org/repos/asf/camel/repo
Commit: http://git-wip-us.apache.org/repos/asf/camel/commit/98d4f6af
Tree: http://git-wip-us.apache.org/repos/asf/camel/tree/98d4f6af
Diff: http://git-wip-us.apache.org/repos/asf/camel/diff/98d4f6af

Branch: refs/heads/master
Commit: 98d4f6af37bffed6d0af72d3a89cff11d2e9ea1b
Parents: 8dfbed1
Author: Claus Ibsen <da...@apache.org>
Authored: Sun Nov 29 10:01:59 2015 +0100
Committer: Claus Ibsen <da...@apache.org>
Committed: Sun Nov 29 10:01:59 2015 +0100

----------------------------------------------------------------------
 .../camel/dataformat/tarfile/TarSplitterRouteIssueTest.java   | 7 +++++--
 1 file changed, 5 insertions(+), 2 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/camel/blob/98d4f6af/components/camel-tarfile/src/test/java/org/apache/camel/dataformat/tarfile/TarSplitterRouteIssueTest.java
----------------------------------------------------------------------
diff --git a/components/camel-tarfile/src/test/java/org/apache/camel/dataformat/tarfile/TarSplitterRouteIssueTest.java b/components/camel-tarfile/src/test/java/org/apache/camel/dataformat/tarfile/TarSplitterRouteIssueTest.java
index 2e785ad..7bbf813 100644
--- a/components/camel-tarfile/src/test/java/org/apache/camel/dataformat/tarfile/TarSplitterRouteIssueTest.java
+++ b/components/camel-tarfile/src/test/java/org/apache/camel/dataformat/tarfile/TarSplitterRouteIssueTest.java
@@ -36,6 +36,10 @@ public class TarSplitterRouteIssueTest extends CamelTestSupport {
 
         template.sendBody("seda:decompressFiles", new File("src/test/resources/data/tarfile3.tar"));
 
+        // TODO: a bug which causes the TarSplitter to include an extra empty entry
+        // the bug is in the TarIterator hasNext / next
+        Thread.sleep(2000);
+
         assertMockEndpointsSatisfied();
     }
     
@@ -59,9 +63,8 @@ public class TarSplitterRouteIssueTest extends CamelTestSupport {
                 
                 from("seda:decompressFiles")
                     .split(new TarSplitter()).streaming().shareUnitOfWork()
-                        .log("we are splitting")
+                        .to("log:entry")
                         .to("mock:entry");
-                        //.to("file:target/tar/?fileName=decompressed.txt&fileExist=Append");
             }
         };
     }