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
The following commit(s) were added to refs/heads/master by this push:
new 0f85792c Fix broken tests.
0f85792c is described below
commit 0f85792cac60ea302a215c8830ea4f358d31ec5a
Author: Gary Gregory <[email protected]>
AuthorDate: Thu May 5 15:35:21 2022 -0400
Fix broken tests.
---
.../lz4/FramedLZ4CompressorInputStreamTest.java | 30 +++++++++-------------
1 file changed, 12 insertions(+), 18 deletions(-)
diff --git
a/src/test/java/org/apache/commons/compress/compressors/lz4/FramedLZ4CompressorInputStreamTest.java
b/src/test/java/org/apache/commons/compress/compressors/lz4/FramedLZ4CompressorInputStreamTest.java
index 08f12a32..c9b1f6c1 100644
---
a/src/test/java/org/apache/commons/compress/compressors/lz4/FramedLZ4CompressorInputStreamTest.java
+++
b/src/test/java/org/apache/commons/compress/compressors/lz4/FramedLZ4CompressorInputStreamTest.java
@@ -19,33 +19,28 @@
package org.apache.commons.compress.compressors.lz4;
import static org.hamcrest.CoreMatchers.containsString;
+import static org.hamcrest.MatcherAssert.assertThat;
import static org.junit.Assert.assertArrayEquals;
import static org.junit.Assert.assertEquals;
import static org.junit.Assert.assertFalse;
-import static org.hamcrest.MatcherAssert.assertThat;
+import static org.junit.Assert.assertThrows;
import static org.junit.Assert.assertTrue;
import static org.junit.Assert.fail;
import java.io.BufferedInputStream;
import java.io.ByteArrayInputStream;
import java.io.File;
-import java.io.InputStream;
import java.io.IOException;
+import java.io.InputStream;
import java.nio.file.Files;
import java.util.Arrays;
import org.apache.commons.compress.AbstractTestCase;
import org.apache.commons.compress.compressors.CompressorStreamFactory;
import org.apache.commons.compress.utils.IOUtils;
-import org.junit.Rule;
import org.junit.jupiter.api.Test;
-import org.junit.rules.ExpectedException;
-
-public final class FramedLZ4CompressorInputStreamTest
- extends AbstractTestCase {
- @Rule
- public final ExpectedException thrown = ExpectedException.none();
+public final class FramedLZ4CompressorInputStreamTest extends AbstractTestCase
{
@Test
public void testMatches() throws IOException {
@@ -158,9 +153,7 @@ public final class FramedLZ4CompressorInputStreamTest
@Test
public void rejectsNonLZ4Stream() throws IOException {
- try (InputStream a = new
FramedLZ4CompressorInputStream(Files.newInputStream(getFile("bla.tar").toPath())))
{
- fail("expected exception");
- }
+ assertThrows(IOException.class, () -> new
FramedLZ4CompressorInputStream(Files.newInputStream(getFile("bla.tar").toPath())));
}
@Test
@@ -635,12 +628,13 @@ public final class FramedLZ4CompressorInputStreamTest
return to;
}
- private void expectIOException(final String fileName) throws IOException {
- thrown.expect(IOException.class);
- try (InputStream is =
Files.newInputStream(getFile(fileName).toPath())) {
- final FramedLZ4CompressorInputStream in = new
FramedLZ4CompressorInputStream(is);
- IOUtils.toByteArray(in);
- }
+ private void expectIOException(final String fileName) {
+ assertThrows(IOException.class, () -> {
+ try (InputStream is =
Files.newInputStream(getFile(fileName).toPath())) {
+ final FramedLZ4CompressorInputStream in = new
FramedLZ4CompressorInputStream(is);
+ IOUtils.toByteArray(in);
+ }
+ });
}
}