updated unit tests (apache#76)

Author: waso

src/changes/changes.xml

@@ -62,7 +62,7 @@
       <action type="add" dev="ggregory" due-to="Alex Nordlund" issue="CLI-282">
         TypeHandler should throw ParseException for an unsupported class.
       </action>
-      <action type="add" dev="ggregory" due-to="waso, Gary Gregory">
+      <action type="add" dev="ggregory" due-to="Waldemar Sojka, Gary Gregory">
         Added setter for Builder.option #33.
       </action>
       <!-- UPDATE -->

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

@@ -81,6 +81,26 @@ public void testBuilderInsufficientParams2() {
         Option.builder(null).desc("desc").build();
     }
 
+    @Test(expected = IllegalArgumentException.class)
+    public void testBuilderInvalidOptionName1() {
+        Option.builder().option("invalid?");
+    }
+
+    @Test(expected = IllegalArgumentException.class)
+    public void testBuilderInvalidOptionName2() {
+        Option.builder().option("invalid@");
+    }
+
+    @Test(expected = IllegalArgumentException.class)
+    public void testBuilderInvalidOptionName3() {
+        Option.builder("invalid?");
+    }
+
+    @Test(expected = IllegalArgumentException.class)
+    public void testBuilderInvalidOptionName4() {
+        Option.builder("invalid@");
+    }
+
     @Test
     public void testBuilderMethods() {
         final char defaultSeparator = (char) 0;