Added a test case, and tweaked the encoder / decoder properties handling in the streams.

Author: jponge

pom.xml

@@ -1,20 +1,32 @@
 <?xml version="1.0"?>
-<project xmlns="http://maven.apache.org/POM/4.0.0" 
+<project xmlns="http://maven.apache.org/POM/4.0.0"
          xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
          xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/maven-v4_0_0.xsd">
-         
+
     <modelVersion>4.0.0</modelVersion>
-    
+
     <groupId>lzma-java</groupId>
     <artifactId>lzma-java</artifactId>
-    
+
     <packaging>jar</packaging>
     <version>0.9-SNAPSHOT</version>
-    
-    <name>LZMA library for Java</name>    
+
+    <name>LZMA library for Java</name>
     <url>http://jponge.github.com/lzma-java</url>
-    
-    <dependencies/>
+
+    <dependencies>
+        <dependency>
+            <groupId>junit</groupId>
+            <artifactId>junit</artifactId>
+            <version>4.7</version>
+            <scope>test</scope>
+        </dependency>
+        <dependency>
+            <groupId>commons-io</groupId>
+            <artifactId>commons-io</artifactId>
+            <version>1.4</version>
+        </dependency>
+    </dependencies>
 
     <build>
         <plugins>

src/main/java/lzma/streams/LzmaOutputStream.java

@@ -93,8 +93,6 @@ public void run()
             {
                 if (!encoderConfigured)
                 {
-                    encoder.setDictionarySize(28);
-                    encoder.setEndMarkerMode(true);
                     encoder.writeCoderProperties(out);
                 }
                 encoder.code(fakeInputStream, out, -1, -1, null);
@@ -284,7 +282,7 @@ public LzmaOutputStream build()
             encoder.setMatchFinder(matchFinder);
             encoder.setNumFastBytes(numFastBytes);
 
-            return new LzmaOutputStream(out, encoder, true);
+            return new LzmaOutputStream(out, encoder, false);
         }
     }
 }

src/test/java/lzma/streams/RountTripTest.java

@@ -0,0 +1,87 @@
+/*
+ *  Copyright (c) 2009 Julien Ponge. All rights reserved.
+ *
+ *  <[email protected]>
+ *  http://julien.ponge.info/
+ *
+ *  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.
+ *
+ *  This work is based on the LZMA SDK by Igor Pavlov.
+ *  The LZMA SDK is placed under the public domain, and can be obtained from
+ *
+ *      http://www.7-zip.org/sdk.html
+ *
+ *  The LzmaInputStream and LzmaOutputStream classes were inspired by the
+ *  work of Christopher League, although they are not derivative works.
+ *
+ *      http://contrapunctus.net/league/haques/lzmajio/
+ */
+
+package lzma.streams;
+
+import org.junit.Test;
+import org.apache.commons.io.FileUtils;
+import static org.apache.commons.io.FileUtils.*;
+import static org.apache.commons.io.FileUtils.contentEquals;
+
+import java.io.*;
+
+import lzma.sdk.lzma.Decoder;
+import junit.framework.TestCase;
+
+public class RountTripTest extends TestCase
+{
+    private void copy(InputStream in, OutputStream out) throws IOException
+    {
+        int nread = 0;
+        final byte[] buffer = new byte[1024];
+        while ((nread = in.read(buffer)) != -1)
+        {
+            out.write(buffer, 0, nread);
+        }
+    }
+
+    public void test_round_trip() throws IOException
+    {
+        final File sourceFile = new File("LICENSE");
+        final File compressed = File.createTempFile("lzma-java", "compressed");
+        final File unCompressed = File.createTempFile("lzma-java", "uncompressed");
+
+        final LzmaOutputStream compressedOut = new LzmaOutputStream.Builder(
+                new BufferedOutputStream(new FileOutputStream(compressed)))
+                .useMaximalDictionarySize()
+                .useEndMarkerMode(true)
+                .useBT4MatchFinder()
+                .build();
+
+        final InputStream sourceIn = new BufferedInputStream(new FileInputStream(sourceFile));
+
+        copy(sourceIn, compressedOut);
+        sourceIn.close();
+        compressedOut.close();
+
+        final LzmaInputStream compressedIn = new LzmaInputStream(
+                new BufferedInputStream(new FileInputStream(compressed)),
+                new Decoder());
+
+        final OutputStream uncompressedOut = new BufferedOutputStream(
+                new FileOutputStream(unCompressed));
+
+        copy(compressedIn, uncompressedOut);
+        compressedIn.close();
+        uncompressedOut.close();
+
+        assertTrue(contentEquals(sourceFile, unCompressed));
+        assertFalse(contentEquals(sourceFile, compressed));
+    }
+}