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 2022/05/05 17:33:37 UTC

[commons-compress] branch master updated: Fix SevenZArchiveEntryTest.

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 2b643c3e Fix SevenZArchiveEntryTest.
2b643c3e is described below

commit 2b643c3ea8ca54e802793544164d71981a9b128b
Author: Gary Gregory <ga...@gmail.com>
AuthorDate: Thu May 5 13:33:32 2022 -0400

    Fix SevenZArchiveEntryTest.
---
 .../archivers/sevenz/SevenZArchiveEntryTest.java   | 40 +++++++++++++---------
 1 file changed, 23 insertions(+), 17 deletions(-)

diff --git a/src/test/java/org/apache/commons/compress/archivers/sevenz/SevenZArchiveEntryTest.java b/src/test/java/org/apache/commons/compress/archivers/sevenz/SevenZArchiveEntryTest.java
index 0f4205d9..7f6cde9b 100644
--- a/src/test/java/org/apache/commons/compress/archivers/sevenz/SevenZArchiveEntryTest.java
+++ b/src/test/java/org/apache/commons/compress/archivers/sevenz/SevenZArchiveEntryTest.java
@@ -17,26 +17,32 @@
  */
 package org.apache.commons.compress.archivers.sevenz;
 
-import org.junit.Assert;
-import org.junit.jupiter.api.Test;
+import static org.junit.jupiter.api.Assertions.assertEquals;
+import static org.junit.jupiter.api.Assertions.assertNotEquals;
+import static org.junit.jupiter.api.Assertions.assertThrows;
 
 import java.util.Arrays;
 
+import org.junit.jupiter.api.Test;
+
+/**
+ * Tests {@link SevenZArchiveEntry}.
+ */
 public class SevenZArchiveEntryTest {
 
     @Test
     public void shouldThrowIfNoLastModifiedDateIsSet() {
-        new SevenZArchiveEntry().getLastModifiedDate();
+        assertThrows(UnsupportedOperationException.class, () -> new SevenZArchiveEntry().getLastModifiedDate());
     }
 
     @Test
     public void shouldThrowIfNoCreationDateIsSet() {
-        new SevenZArchiveEntry().getCreationDate();
+        assertThrows(UnsupportedOperationException.class, () -> new SevenZArchiveEntry().getCreationDate());
     }
 
     @Test
     public void shouldThrowIfNoAccessDateIsSet() {
-        new SevenZArchiveEntry().getAccessDate();
+        assertThrows(UnsupportedOperationException.class, () -> new SevenZArchiveEntry().getAccessDate());
     }
 
     @Test
@@ -44,8 +50,8 @@ public class SevenZArchiveEntryTest {
         final SevenZArchiveEntry z1 = new SevenZArchiveEntry();
         final SevenZArchiveEntry z2 = new SevenZArchiveEntry();
         z2.setContentMethods(Arrays.asList(new SevenZMethodConfiguration(SevenZMethod.COPY)));
-        Assert.assertNotEquals(z1, z2);
-        Assert.assertNotEquals(z2, z1);
+        assertNotEquals(z1, z2);
+        assertNotEquals(z2, z1);
     }
 
     @Test
@@ -55,8 +61,8 @@ public class SevenZArchiveEntryTest {
         z1.setContentMethods(Arrays.asList(new SevenZMethodConfiguration(SevenZMethod.COPY)));
         z2.setContentMethods(Arrays.asList(new SevenZMethodConfiguration(SevenZMethod.DELTA_FILTER),
                                            new SevenZMethodConfiguration(SevenZMethod.LZMA2)));
-        Assert.assertNotEquals(z1, z2);
-        Assert.assertNotEquals(z2, z1);
+        assertNotEquals(z1, z2);
+        assertNotEquals(z2, z1);
     }
 
     @Test
@@ -67,8 +73,8 @@ public class SevenZArchiveEntryTest {
                                            new SevenZMethodConfiguration(SevenZMethod.LZMA2)));
         z2.setContentMethods(Arrays.asList(new SevenZMethodConfiguration(SevenZMethod.DELTA_FILTER),
                                            new SevenZMethodConfiguration(SevenZMethod.LZMA2)));
-        Assert.assertEquals(z1, z2);
-        Assert.assertEquals(z2, z1);
+        assertEquals(z1, z2);
+        assertEquals(z2, z1);
     }
 
     @Test
@@ -79,8 +85,8 @@ public class SevenZArchiveEntryTest {
                                            new SevenZMethodConfiguration(SevenZMethod.DELTA_FILTER)));
         z2.setContentMethods(Arrays.asList(new SevenZMethodConfiguration(SevenZMethod.DELTA_FILTER),
                                            new SevenZMethodConfiguration(SevenZMethod.LZMA2)));
-        Assert.assertNotEquals(z1, z2);
-        Assert.assertNotEquals(z2, z1);
+        assertNotEquals(z1, z2);
+        assertNotEquals(z2, z1);
     }
 
     @Test
@@ -91,10 +97,10 @@ public class SevenZArchiveEntryTest {
         z1.setContentMethods(Arrays.asList(new SevenZMethodConfiguration(SevenZMethod.LZMA2, 1)));
         z2.setContentMethods(Arrays.asList(new SevenZMethodConfiguration(SevenZMethod.LZMA2, 2)));
         z3.setContentMethods(Arrays.asList(new SevenZMethodConfiguration(SevenZMethod.LZMA2, 2)));
-        Assert.assertNotEquals(z1, z2);
-        Assert.assertNotEquals(z2, z1);
-        Assert.assertEquals(z3, z2);
-        Assert.assertEquals(z2, z3);
+        assertNotEquals(z1, z2);
+        assertNotEquals(z2, z1);
+        assertEquals(z3, z2);
+        assertEquals(z2, z3);
     }
 
 }