Adding comma delimited whitespace to the exception message that lists missing required options as requested in CLI-149. I didn't add the requested -, as it could be -- or some other prefix. Unit tests also added.

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

Author: hen

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

@@ -317,9 +317,10 @@ protected void checkRequiredOptions()
             while (iter.hasNext())
             {
                 buff.append(iter.next());
+                buff.append(", ");
             }
 
-            throw new MissingOptionException(buff.toString());
+            throw new MissingOptionException(buff.substring(0, buff.length() - 2));
         }
     }
 

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

@@ -113,7 +113,7 @@ public void testMissingOptionsException() throws ParseException {
             new PosixParser().parse(options, new String[0]);
             fail("Expected MissingOptionException to be thrown");
         } catch (MissingOptionException e) {
-            assertEquals("Missing required options: fx", e.getMessage());
+            assertEquals("Missing required options: f, x", e.getMessage());
         }
     }
 

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

@@ -105,12 +105,38 @@ public void testMissingRequiredOption()
             CommandLine cl = parser.parse(_options,args);
             fail( "exception should have been thrown" );
         }
+        catch (MissingOptionException e)
+        {
+            assertEquals( "Incorrect exception message", "Missing required option: b", e.getMessage() );
+        }
+        catch (ParseException e)
+        {
+            fail( "expected to catch MissingOptionException" );
+        }
+    }
+
+    public void testMissingRequiredOptions()
+    {
+        String[] args = new String[] { "-a" };
+
+        _options.addOption( OptionBuilder.withLongOpt( "cfile" )
+                                     .hasArg()
+                                     .isRequired()
+                                     .withDescription( "set the value of [c]" )
+                                     .create( 'c' ) );
+
+        try
+        {
+            CommandLine cl = parser.parse(_options,args);
+            fail( "exception should have been thrown" );
+        }
+        catch (MissingOptionException e)
+        {
+            assertEquals( "Incorrect exception message", "Missing required options: b, c", e.getMessage() );
+        }
         catch (ParseException e)
         {
-            if( !( e instanceof MissingOptionException ) )
-            {
-                fail( "expected to catch MissingOptionException" );
-            }
+            fail( "expected to catch MissingOptionException" );
         }
     }