MissingArgumentException now references the option with the missing argument

git-svn-id: https://svn.apache.org/repos/asf/commons/proper/cli/branches/cli-1.x@678688 13f79535-47bb-0310-9956-ffa450edef68

Author: ebourg

src/java/org/apache/commons/cli/MissingArgumentException.java

@@ -26,6 +26,9 @@
  */
 public class MissingArgumentException extends ParseException
 {
+    /** The option requiring additional arguments */
+    private Option option;
+
     /**
      * Construct a new <code>MissingArgumentException</code>
      * with the specified detail message.
@@ -36,4 +39,28 @@ public MissingArgumentException(String message)
     {
         super(message);
     }
+
+    /**
+     * Construct a new <code>MissingArgumentException</code>
+     * with the specified detail message.
+     *
+     * @param option the option requiring an argument
+     * @since 1.2
+     */
+    public MissingArgumentException(Option option)
+    {
+        this("Missing argument for option: " + option.getKey());
+        this.option = option;
+    }
+
+    /**
+     * Return the option requiring an argument that wasn't provided
+     * on the command line.
+     *
+     * @since 1.2
+     */
+    public Option getOption()
+    {
+        return option;
+    }
 }

src/java/org/apache/commons/cli/Parser.java

@@ -355,8 +355,7 @@ public void processArgs(Option opt, ListIterator iter)
 
         if ((opt.getValues() == null) && !opt.hasOptionalArg())
         {
-            throw new MissingArgumentException("Missing argument for option:"
-                                               + opt.getKey());
+            throw new MissingArgumentException(opt);
         }
     }
 

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

@@ -102,6 +102,7 @@ public void testMissingArg() throws Exception
         catch (MissingArgumentException e)
         {
             caught = true;
+            assertEquals("option missing an argument", "b", e.getOption().getOpt());
         }
 
         assertTrue( "Confirm MissingArgumentException caught", caught );

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

@@ -122,6 +122,7 @@ public void testMissingArg() throws Exception
         catch (MissingArgumentException e)
         {
             caught = true;
+            assertEquals("option missing an argument", "b", e.getOption().getOpt());
         }
 
         assertTrue( "Confirm MissingArgumentException caught", caught );

src/test/org/apache/commons/cli/bug/BugCLI71Test.java

@@ -67,8 +67,8 @@ public void testLackOfError() throws Exception {
         try {
             CommandLine line = parser.parse( options, args);
             fail("MissingArgumentException expected");
-        } catch(MissingArgumentException mae) {
-            // expected
+        } catch(MissingArgumentException e) {
+            assertEquals("option missing an argument", "k", e.getOption().getOpt());
         }
     }