You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@commons.apache.org by gg...@apache.org on 2023/08/29 22:44:38 UTC
[commons-compress] 03/03: Use better test API
This is an automated email from the ASF dual-hosted git repository.
ggregory pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/commons-compress.git
commit c6c99897905756e2fea3712c532f94d8ce2f75f0
Author: Gary Gregory <ga...@gmail.com>
AuthorDate: Tue Aug 29 17:34:06 2023 -0400
Use better test API
---
.../commons/compress/compressors/lz4/FactoryTest.java | 14 +++++++-------
1 file changed, 7 insertions(+), 7 deletions(-)
diff --git a/src/test/java/org/apache/commons/compress/compressors/lz4/FactoryTest.java b/src/test/java/org/apache/commons/compress/compressors/lz4/FactoryTest.java
index 038b89db..b8f39221 100644
--- a/src/test/java/org/apache/commons/compress/compressors/lz4/FactoryTest.java
+++ b/src/test/java/org/apache/commons/compress/compressors/lz4/FactoryTest.java
@@ -20,10 +20,10 @@ package org.apache.commons.compress.compressors.lz4;
import static org.junit.jupiter.api.Assertions.assertArrayEquals;
-import java.io.File;
import java.io.InputStream;
import java.io.OutputStream;
import java.nio.file.Files;
+import java.nio.file.Path;
import org.apache.commons.compress.AbstractTestCase;
import org.apache.commons.compress.compressors.CompressorStreamFactory;
@@ -43,18 +43,18 @@ public class FactoryTest extends AbstractTestCase {
}
private void roundtripViaFactory(final String format) throws Exception {
- final File input = getFile("bla.tar");
+ final Path input = getPath("bla.tar");
long start = System.currentTimeMillis();
- final File outputSz = new File(dir, input.getName() + "." + format + ".lz4");
- try (OutputStream os = Files.newOutputStream(outputSz.toPath());
+ final Path outputSz = dir.toPath().resolve(input.getFileName() + "." + format + ".lz4");
+ try (OutputStream os = Files.newOutputStream(outputSz);
OutputStream los = new CompressorStreamFactory().createCompressorOutputStream(format, os)) {
- Files.copy(input.toPath(), los);
+ Files.copy(input, los);
}
// System.err.println(input.getName() + " written, uncompressed bytes: " + input.length()
// + ", compressed bytes: " + outputSz.length() + " after " + (System.currentTimeMillis() - start) + "ms");
start = System.currentTimeMillis();
- try (InputStream is = Files.newInputStream(input.toPath());
- InputStream sis = new CompressorStreamFactory().createCompressorInputStream(format, Files.newInputStream(outputSz.toPath()))) {
+ try (InputStream is = Files.newInputStream(input);
+ InputStream sis = new CompressorStreamFactory().createCompressorInputStream(format, Files.newInputStream(outputSz))) {
final byte[] expected = IOUtils.toByteArray(is);
final byte[] actual = IOUtils.toByteArray(sis);
assertArrayEquals(expected, actual);