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 169277a6cfee9f4ff9e222cf3cb9a2fef53fa7cf Author: Gary Gregory <garydgreg...@gmail.com> AuthorDate: Thu May 5 13:48:06 2022 -0400 Sort members. --- .../sevenz/SevenZMethodConfigurationTest.java | 24 +++++++++++----------- 1 file changed, 12 insertions(+), 12 deletions(-) diff --git a/src/test/java/org/apache/commons/compress/archivers/sevenz/SevenZMethodConfigurationTest.java b/src/test/java/org/apache/commons/compress/archivers/sevenz/SevenZMethodConfigurationTest.java index 551b9a1e..7b60261e 100644 --- a/src/test/java/org/apache/commons/compress/archivers/sevenz/SevenZMethodConfigurationTest.java +++ b/src/test/java/org/apache/commons/compress/archivers/sevenz/SevenZMethodConfigurationTest.java @@ -25,29 +25,19 @@ import org.tukaani.xz.LZMA2Options; public class SevenZMethodConfigurationTest { - @Test - public void shouldAllowNullOptions() { - Assert.assertNull(new SevenZMethodConfiguration(SevenZMethod.LZMA2, null).getOptions()); - } - @Test public void shouldAllowLZMA2OptionsForLZMA() { Assert.assertNotNull(new SevenZMethodConfiguration(SevenZMethod.LZMA, new LZMA2Options()).getOptions()); } - @Test - public void shouldAllowNumberForLZMA() { - Assert.assertNotNull(new SevenZMethodConfiguration(SevenZMethod.LZMA, 42).getOptions()); - } - @Test public void shouldAllowLZMA2OptionsForLZMA2() { Assert.assertNotNull(new SevenZMethodConfiguration(SevenZMethod.LZMA2, new LZMA2Options()).getOptions()); } @Test - public void shouldAllowNumberForLZMA2() { - Assert.assertNotNull(new SevenZMethodConfiguration(SevenZMethod.LZMA2, 42).getOptions()); + public void shouldAllowNullOptions() { + Assert.assertNull(new SevenZMethodConfiguration(SevenZMethod.LZMA2, null).getOptions()); } @Test @@ -60,6 +50,16 @@ public class SevenZMethodConfigurationTest { Assert.assertNotNull(new SevenZMethodConfiguration(SevenZMethod.DEFLATE, 42).getOptions()); } + @Test + public void shouldAllowNumberForLZMA() { + Assert.assertNotNull(new SevenZMethodConfiguration(SevenZMethod.LZMA, 42).getOptions()); + } + + @Test + public void shouldAllowNumberForLZMA2() { + Assert.assertNotNull(new SevenZMethodConfiguration(SevenZMethod.LZMA2, 42).getOptions()); + } + @Test public void shouldNotAllowStringOptionsForLZMA() { assertThrows(IllegalArgumentException.class, () -> new SevenZMethodConfiguration(SevenZMethod.LZMA, ""));