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);