added tests

Author: Claudenw

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

@@ -469,10 +469,7 @@ public <T> T getParsedOptionValue(final Option option, final T defaultValue) thr
      */
     @SuppressWarnings("unchecked")
     public <T> T getParsedOptionValue(final Option option, final Supplier<T> defaultValue) throws ParseException {
-        if (option == null) {
-            return null;
-        }
-        final String res = getOptionValue(option);
+        final String res = option == null ? null : getOptionValue(option);
 
         try {
             return res == null ? defaultValue.get() : (T) option.getConverter().apply(res);

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

@@ -165,6 +165,26 @@ public void testGetParsedOptionValueWithOption() throws Exception {
         assertEquals("foo", cmd.getParsedOptionValue(optF));
     }
 
+    @Test
+    public void testGetParsedOptionValueUsingDefault() throws Exception {
+        final Options options = new Options();
+        final Option optI = Option.builder("i").hasArg().type(Number.class).build();
+        final Option optF = Option.builder("f").hasArg().build();
+        options.addOption(optI);
+        options.addOption(optF);
+
+        final CommandLineParser parser = new DefaultParser();
+        final CommandLine cmd = parser.parse(options, new String[] {"-i", "123"});
+
+        assertEquals(123, ((Number) cmd.getParsedOptionValue(optI)).intValue());
+        assertEquals("foo", cmd.getParsedOptionValue(optF, "foo"));
+        assertEquals("foo", cmd.getParsedOptionValue(optF, ()->"foo"));
+        assertEquals("foo", cmd.getParsedOptionValue("f", "foo"));
+        assertEquals("foo", cmd.getParsedOptionValue("f", ()->"foo"));
+        assertEquals("foo", cmd.getParsedOptionValue('f', "foo"));
+        assertEquals("foo", cmd.getParsedOptionValue('f', ()->"foo"));
+    }
+
     @Test
     public void testNullhOption() throws Exception {
         final Options options = new Options();