Sort members

Author: garydgregory

src/main/java/org/apache/commons/cli/Option.java

@@ -136,19 +136,6 @@ public Option build() {
             return get();
         }
 
-        /**
-         * Constructs an Option with the values declared by this {@link Builder}.
-         *
-         * @return the new {@link Option}.
-         * @throws IllegalArgumentException if neither {@code opt} or {@code longOpt} has been set.
-         */
-        public Option get() {
-            if (option == null && longOption == null) {
-                throw new IllegalArgumentException("Either opt or longOpt must be specified");
-            }
-            return new Option(this);
-        }
-
         /**
          * Sets the converter for the option.
          * <p>
@@ -197,6 +184,19 @@ public Builder desc(final String description) {
             return this;
         }
 
+        /**
+         * Constructs an Option with the values declared by this {@link Builder}.
+         *
+         * @return the new {@link Option}.
+         * @throws IllegalArgumentException if neither {@code opt} or {@code longOpt} has been set.
+         */
+        public Option get() {
+            if (option == null && longOption == null) {
+                throw new IllegalArgumentException("Either opt or longOpt must be specified");
+            }
+            return new Option(this);
+        }
+
         /**
          * Tests whether the Option will require an argument.
          *

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

@@ -36,21 +36,6 @@ Licensed to the Apache Software Foundation (ASF) under one or more
 @SuppressWarnings("deprecation") // tests some deprecated classes
 class OptionsTest {
 
-    @Test
-    void testRequiredOptionInGroupShouldNotBeInRequiredList() {
-        final String key = "a";
-        final Option option = new Option(key, "along", false, "Option A");
-        option.setRequired(true);
-        final Options options = new Options();
-        options.addOption(option);
-        assertTrue(options.getRequiredOptions().contains(key));
-        final OptionGroup optionGroup = new OptionGroup();
-        optionGroup.addOption(option);
-        options.addOptionGroup(optionGroup);
-        assertFalse(options.getOption(key).isRequired());
-        assertFalse(options.getRequiredOptions().contains(key), "Option in group shouldn't be in required options list.");
-    }
-
     private void assertToStrings(final Option option) {
         // Should never throw.
         // Should return a String, not null.
@@ -280,6 +265,21 @@ void testMissingOptionsException() throws ParseException {
         assertEquals("Missing required options: f, x", e.getMessage());
     }
 
+    @Test
+    void testRequiredOptionInGroupShouldNotBeInRequiredList() {
+        final String key = "a";
+        final Option option = new Option(key, "along", false, "Option A");
+        option.setRequired(true);
+        final Options options = new Options();
+        options.addOption(option);
+        assertTrue(options.getRequiredOptions().contains(key));
+        final OptionGroup optionGroup = new OptionGroup();
+        optionGroup.addOption(option);
+        options.addOptionGroup(optionGroup);
+        assertFalse(options.getOption(key).isRequired());
+        assertFalse(options.getRequiredOptions().contains(key), "Option in group shouldn't be in required options list.");
+    }
+
     @Test
     void testSimple() {
         final Options options = new Options();