ArgumentBuilder now resets the validator to null on reset()

git-svn-id: https://svn.apache.org/repos/asf/jakarta/commons/proper/cli/trunk@130093 13f79535-47bb-0310-9956-ffa450edef68

Author: Robert James Oxspring

src/java/org/apache/commons/cli2/builder/ArgumentBuilder.java

@@ -100,6 +100,7 @@ public final void reset() {
         maximum = Integer.MAX_VALUE;
         initialSeparator = ArgumentImpl.DEFAULT_INITIAL_SEPARATOR;
         subsequentSeparator = ArgumentImpl.DEFAULT_SUBSEQUENT_SEPARATOR;
+        validator = null;
         consumeRemaining = "--";
         defaultValues = null;
         id = 0;

src/test/org/apache/commons/cli2/option/ArgumentTest.java

@@ -319,6 +319,7 @@ public void testProcessValues_ContractedTooFew() throws OptionException {
         final ListIterator iterator = args.listIterator();
         try {
             option.processValues(commandLine, iterator, option);
+            option.validate(commandLine);
             fail("Expected MissingValueException");
         }
         catch (OptionException mve) {
@@ -549,6 +550,7 @@ public void testProcess_ConsumeNothing() throws OptionException {
 
         try {
             option.process(commandLine, iterator);
+            option.validate(commandLine);
             fail("Missing Value!");
         }
         catch (OptionException mve) {