Fixing the previous commit so the right variable is used. There's no reason to share the options class between methods, so deleting the setUp()

git-svn-id: https://svn.apache.org/repos/asf/commons/proper/cli/branches/cli-1.x@745755 13f79535-47bb-0310-9956-ffa450edef68

Author: hen

src/test/org/apache/commons/cli/bug/BugCLI162Test.java

@@ -32,14 +32,9 @@
 
 public class BugCLI162Test extends TestCase {
 
-    private Options options;
-
-    public void setUp() {
-        options = new Options();
-        options.addOption("h", "help", false, "This is a looooong description");
-    }
-
     public void testInfiniteLoop() {
+        Options options = new Options();
+        options.addOption("h", "help", false, "This is a looooong description");
         HelpFormatter formatter = new HelpFormatter();
         formatter.setWidth(20);
         formatter.printHelp("app", options); // used to hang & crash
@@ -229,7 +224,7 @@ public void testPrintHelpLongLines() throws ParseException, IOException {
                 "Converts the JDBC file in the first argument to an SMFD file specified in the second argument.");
         option.setArgs(2);
         commandLineOptions.addOption(option);
-        new HelpFormatter().printHelp(this.getClass().getName(), options);
+        new HelpFormatter().printHelp(this.getClass().getName(), commandLineOptions);
     }
 
 }