Moved the real world test from PosixParserTest to ApplicationTest

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

Author: ebourg

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

@@ -218,4 +218,71 @@ public void testMan()
         hf.printHelp(60, cmdLine, null, options, null);
     }
 
+
+    /**
+     * Real world test with long and short options.
+     */
+    public void testNLT() throws Exception {
+        Option help = new Option("h", "help", false, "print this message");
+        Option version = new Option("v", "version", false, "print version information");
+        Option newRun = new Option("n", "new", false, "Create NLT cache entries only for new items");
+        Option trackerRun = new Option("t", "tracker", false, "Create NLT cache entries only for tracker items");
+
+        Option timeLimit = OptionBuilder.withLongOpt("limit").hasArg()
+                                        .withValueSeparator()
+                                        .withDescription("Set time limit for execution, in minutes")
+                                        .create("l");
+
+        Option age = OptionBuilder.withLongOpt("age").hasArg()
+                                  .withValueSeparator()
+                                  .withDescription("Age (in days) of cache item before being recomputed")
+                                  .create("a");
+
+        Option server = OptionBuilder.withLongOpt("server").hasArg()
+                                     .withValueSeparator()
+                                     .withDescription("The NLT server address")
+                                     .create("s");
+
+        Option numResults = OptionBuilder.withLongOpt("results").hasArg()
+                                         .withValueSeparator()
+                                         .withDescription("Number of results per item")
+                                         .create("r");
+
+        Option configFile = OptionBuilder.withLongOpt("file").hasArg()
+                                         .withValueSeparator()
+                                         .withDescription("Use the specified configuration file")
+                                         .create();
+
+        Options options = new Options();
+        options.addOption(help);
+        options.addOption(version);
+        options.addOption(newRun);
+        options.addOption(trackerRun);
+        options.addOption(timeLimit);
+        options.addOption(age);
+        options.addOption(server);
+        options.addOption(numResults);
+        options.addOption(configFile);
+
+        // create the command line parser
+        CommandLineParser parser = new PosixParser();
+
+        String[] args = new String[] {
+                "-v",
+                "-l",
+                "10",
+                "-age",
+                "5",
+                "-file",
+                "filename"
+            };
+
+        CommandLine line = parser.parse(options, args);
+        assertTrue(line.hasOption("v"));
+        assertEquals(line.getOptionValue("l"), "10");
+        assertEquals(line.getOptionValue("limit"), "10");
+        assertEquals(line.getOptionValue("a"), "5");
+        assertEquals(line.getOptionValue("age"), "5");
+        assertEquals(line.getOptionValue("file"), "filename");
+    }
 }

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

@@ -30,73 +30,6 @@ public void setUp()
         parser = new PosixParser();
     }
 
-    /**
-     * Real world test with long and short options.
-     */
-    public void testLongOptionWithShort() throws Exception {
-        Option help = new Option("h", "help", false, "print this message");
-        Option version = new Option("v", "version", false, "print version information");
-        Option newRun = new Option("n", "new", false, "Create NLT cache entries only for new items");
-        Option trackerRun = new Option("t", "tracker", false, "Create NLT cache entries only for tracker items");
-
-        Option timeLimit = OptionBuilder.withLongOpt("limit").hasArg()
-                                        .withValueSeparator()
-                                        .withDescription("Set time limit for execution, in minutes")
-                                        .create("l");
-
-        Option age = OptionBuilder.withLongOpt("age").hasArg()
-                                  .withValueSeparator()
-                                  .withDescription("Age (in days) of cache item before being recomputed")
-                                  .create("a");
-
-        Option server = OptionBuilder.withLongOpt("server").hasArg()
-                                     .withValueSeparator()
-                                     .withDescription("The NLT server address")
-                                     .create("s");
-
-        Option numResults = OptionBuilder.withLongOpt("results").hasArg()
-                                         .withValueSeparator()
-                                         .withDescription("Number of results per item")
-                                         .create("r");
-
-        Option configFile = OptionBuilder.withLongOpt("file").hasArg()
-                                         .withValueSeparator()
-                                         .withDescription("Use the specified configuration file")
-                                         .create();
-
-        Options options = new Options();
-        options.addOption(help);
-        options.addOption(version);
-        options.addOption(newRun);
-        options.addOption(trackerRun);
-        options.addOption(timeLimit);
-        options.addOption(age);
-        options.addOption(server);
-        options.addOption(numResults);
-        options.addOption(configFile);
-
-        // create the command line parser
-        CommandLineParser parser = new PosixParser();
-
-        String[] args = new String[] {
-                "-v",
-                "-l",
-                "10",
-                "-age",
-                "5",
-                "-file",
-                "filename"
-            };
-
-        CommandLine line = parser.parse(options, args);
-        assertTrue(line.hasOption("v"));
-        assertEquals(line.getOptionValue("l"), "10");
-        assertEquals(line.getOptionValue("limit"), "10");
-        assertEquals(line.getOptionValue("a"), "5");
-        assertEquals(line.getOptionValue("age"), "5");
-        assertEquals(line.getOptionValue("file"), "filename");
-    }
-
     public void testLongWithEqualSingleDash() throws Exception
     {
         // not supported by the PosixParser