Index: maven-commandline-plugin/src/test/org/codelutin/util/ParserTest.java diff -u maven-commandline-plugin/src/test/org/codelutin/util/ParserTest.java:1.2 maven-commandline-plugin/src/test/org/codelutin/util/ParserTest.java:1.3 --- maven-commandline-plugin/src/test/org/codelutin/util/ParserTest.java:1.2 Wed Dec 12 21:05:00 2007 +++ maven-commandline-plugin/src/test/org/codelutin/util/ParserTest.java Wed Dec 12 22:12:51 2007 @@ -43,137 +43,40 @@ public void testOptions() throws Exception { parser.doParse(); - assertError(parser, 0, 0, 0); + assertError(parser, 0, 0); parser.doParse("--true"); - assertError(parser, 0, 0, 1); + assertError(parser, 0, 1); parser.doParse("-l"); - assertError(parser, 0, 0, 1); + assertError(parser, 0, 1); assertFalse(parser.isOptionEnabled("list")); parser.doParse("-l","regions"); - assertError(parser, 1,0, 0); + assertError(parser, 1, 0); assertTrue(parser.isOptionEnabled("list")); } - public void testArguments() throws Exception { - - parser.doParse(); - assertError(parser, 0, 0, 0); - - parser.doParse("true"); - assertError(parser, 0, 1, 0); - assertTrue(parser.isArgumentEnabled("bool")); - - parser.doParse("true", "true"); - assertError(parser, 0, 2, 0); - assertTrue(parser.isArgumentEnabled("bool")); - - parser.doParse("true", "true", "yo"); - assertError(parser, 0, 3, 0); - assertTrue(parser.isArgumentEnabled("bool")); - assertTrue(parser.isArgumentEnabled("strin")); - - parser.doParse("true", "true", "5"); - assertError(parser, 0, 3, 0); - assertTrue(parser.isArgumentEnabled("bool")); - assertTrue(parser.isArgumentEnabled("inte")); - - parser.doParse("5", "6", "true", "true"); - assertError(parser, 0, 3, 1); - assertTrue(parser.isArgumentEnabled("bool")); - assertTrue(parser.isArgumentEnabled("inte")); - - parser.doParse("true", "true", "5.5", "6"); - assertError(parser, 0, 3, 1); - assertTrue(parser.isArgumentEnabled("bool")); - assertTrue(parser.isArgumentEnabled("inte")); - - parser.doParse("5.5", "6.2", "true", "true"); - assertError(parser, 0, 2, 2); - assertTrue(parser.isArgumentEnabled("bool")); - assertFalse(parser.isArgumentEnabled("inte")); - - parser.doParse("true", "true", "true"); - assertError(parser, 0, 2, 1); - assertTrue(parser.isArgumentEnabled("bool")); - - parser.doParse("true", "true", "yo", "yo"); - assertError(parser, 0, 3, 1); - assertTrue(parser.isArgumentEnabled("bool")); - - parser.doParse("true", "true", "true", "yo", "yo"); - assertError(parser, 0, 3, 2); - assertTrue(parser.isArgumentEnabled("bool")); - assertTrue(parser.isArgumentEnabled("strin")); - - parser.doParse("true", "yo", "/home"); - assertError(parser, 0, 3, 0); - assertTrue(parser.isArgumentEnabled("bool")); - assertTrue(parser.isArgumentEnabled("strin")); - assertTrue(parser.isArgumentEnabled("fil")); - - parser.doParse("true", "/home", "yo"); - assertError(parser, 0, 3, 0); - assertTrue(parser.isArgumentEnabled("bool")); - assertTrue(parser.isArgumentEnabled("strin")); - assertTrue(parser.isArgumentEnabled("fil")); - - parser.doParse("true", "/home", "/home"); - assertError(parser, 0, 2, 1); - assertTrue(parser.isArgumentEnabled("bool")); - assertFalse(parser.isArgumentEnabled("strin")); - assertTrue(parser.isArgumentEnabled("fil")); - - parser.doParse("true", "home", "/home"); - assertError(parser, 0, 3, 0); - assertTrue(parser.isArgumentEnabled("bool")); - assertTrue(parser.isArgumentEnabled("strin")); - assertTrue(parser.isArgumentEnabled("fil")); - - parser.doParse("true", "home", "/home", "java.lang.Class"); - assertError(parser, 0, 4, 0); - assertTrue(parser.isArgumentEnabled("bool")); - assertTrue(parser.isArgumentEnabled("strin")); - assertTrue(parser.isArgumentEnabled("fil")); - assertTrue(parser.isArgumentEnabled("cla")); - - parser.doParse("true", "home", "/home", "java.lang.Class", "java.lang.Short", "java.lang.Double", "java.lang.Long", "java.lang.Integer"); - assertError(parser, 0, 8, 0); - assertTrue(parser.isArgumentEnabled("bool")); - assertTrue(parser.isArgumentEnabled("strin")); - assertTrue(parser.isArgumentEnabled("fil")); - assertTrue(parser.isArgumentEnabled("cla")); - - parser.doParse("true", "home", "/home", "java.lang.Class", "java.lang.Short", "java.lang.Double", "java.lang.Long", "java.lang.Intege"); - assertError(parser, 0, 7, 1); - assertTrue(parser.isArgumentEnabled("bool")); - assertTrue(parser.isArgumentEnabled("strin")); - assertTrue(parser.isArgumentEnabled("fil")); - assertTrue(parser.isArgumentEnabled("cla")); - } - public void testUnfoundOptions() throws Exception { parser.doParse("true", "true", "-h"); - assertError(parser, 1, 0, 0); + assertError(parser, 1, 1); parser.doParse("true", "true", "-l", "exports"); - assertError(parser, 1, 0, 0); + assertError(parser, 1, 1); parser.doParse("-l", "exports"); - assertError(parser, 1, 0, 0); + assertError(parser, 1, 0); parser.doParse("true", "-zzh", "yo", "ya", "-yu", "yy", "uu", "-h"); - assertError(parser, 1, 0, 2); + assertError(parser, 1, 3); } - protected static void assertError(OptionParser parser, int nbOptions, int nbArguments, int nbErrors) throws IOException { + protected static void assertError(OptionParser parser, int nbOptions, int nbErrors) throws IOException { Writer writer = new StringWriter(); parser.printErrors(writer); System.out.println(writer.toString()); @@ -181,7 +84,6 @@ writer.close(); assertEquals(nbErrors != 0, parser.hasFailed()); assertEquals(nbOptions, parser.getOptions().length); - assertEquals(nbArguments, parser.getArguments().length); assertEquals(nbErrors, parser._errors.length); }