Sort members

Author: garydgregory

src/test/java/org/apache/commons/cli/CommandLineTest.java

@@ -31,21 +31,6 @@ Licensed to the Apache Software Foundation (ASF) under one or more
 @SuppressWarnings("deprecation") // tests some deprecated classes
 public class CommandLineTest {
 
-    @Test
-    public void testDeprecatedOption() {
-        final CommandLine.Builder builder = new CommandLine.Builder();
-        builder.addArg("foo").addArg("bar");
-        final Option opt = Option.builder().option("T").deprecated().build();
-        builder.addOption(opt);
-        final AtomicReference<Option> handler = new AtomicReference<>();
-        final CommandLine cmd = builder.setDeprecatedHandler(handler::set).build();
-        cmd.getOptionValue(opt.getOpt());
-        assertSame(opt, handler.get());
-        handler.set(null);
-        cmd.getOptionValue("Nope");
-        assertNull(handler.get());
-    }
-
     @Test
     public void testBuilder() {
         final CommandLine.Builder builder = new CommandLine.Builder();
@@ -83,6 +68,21 @@ public void testBuilderNullOption() {
         assertEquals(0, cmd.getOptions().length);
     }
 
+    @Test
+    public void testDeprecatedOption() {
+        final CommandLine.Builder builder = new CommandLine.Builder();
+        builder.addArg("foo").addArg("bar");
+        final Option opt = Option.builder().option("T").deprecated().build();
+        builder.addOption(opt);
+        final AtomicReference<Option> handler = new AtomicReference<>();
+        final CommandLine cmd = builder.setDeprecatedHandler(handler::set).build();
+        cmd.getOptionValue(opt.getOpt());
+        assertSame(opt, handler.get());
+        handler.set(null);
+        cmd.getOptionValue("Nope");
+        assertNull(handler.get());
+    }
+
     @Test
     public void testGetOptionProperties() throws Exception {
         final String[] args = {"-Dparam1=value1", "-Dparam2=value2", "-Dparam3", "-Dparam4=value4", "-D", "--property", "foo=bar"};

src/test/java/org/apache/commons/cli/OptionsTest.java

@@ -123,14 +123,6 @@ public void testAddOptions2X() {
         assertThrows(IllegalArgumentException.class, () -> options.addOptions(options));
     }
 
-    @Test
-    public void testDuplicateLong() {
-        final Options opts = new Options();
-        opts.addOption("a", "--a", false, "toggle -a");
-        opts.addOption("a", "--a", false, "toggle -a*");
-        assertEquals("toggle -a*", opts.getOption("a").getDescription(), "last one in wins");
-    }
-
     @Test
     public void testDeprecated() {
         final Options opts = new Options();
@@ -148,6 +140,14 @@ public void testDeprecated() {
         assertEquals("Option 'c': Deprecated for removal since 2.0: Use X.", opts.getOption("c").toDeprecatedString());
     }
 
+    @Test
+    public void testDuplicateLong() {
+        final Options opts = new Options();
+        opts.addOption("a", "--a", false, "toggle -a");
+        opts.addOption("a", "--a", false, "toggle -a*");
+        assertEquals("toggle -a*", opts.getOption("a").getDescription(), "last one in wins");
+    }
+
     @Test
     public void testDuplicateSimple() {
         final Options opts = new Options();