Improved test coverage for HelpFormatter.printHelp() with a null or empty cmdLineSyntax parameter

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

Author: ebourg

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

@@ -155,6 +155,31 @@ public void testPrintOptions() throws Exception
        assertEquals("multiple wrapped options", expected, sb.toString());
    }
 
+    public void testPrintHelpWithEmptySyntax()
+    {
+        HelpFormatter formatter = new HelpFormatter();
+        try
+        {
+            formatter.printHelp(null, new Options());
+            fail("null command line syntax should be rejected");
+        }
+        catch (IllegalArgumentException e)
+        {
+            // expected
+        }
+
+        try
+        {
+            formatter.printHelp("", new Options());
+            fail("empty command line syntax should be rejected");
+        }
+        catch (IllegalArgumentException e)
+        {
+            // expected
+        }
+    }
+
+
    public void testAutomaticUsage() throws Exception
    {
        HelpFormatter hf = new HelpFormatter();
@@ -295,13 +320,7 @@ public void testPrintOptionWithEmptyArgNameUsage()
 
     public void testRtrim()
     {
-        HelpFormatter formatter = new HelpFormatter()
-        {
-            public String rtrim(String s)
-            {
-                return super.rtrim(s);
-            }
-        };
+        HelpFormatter formatter = new HelpFormatter();
 
         assertEquals(null, formatter.rtrim(null));
         assertEquals("", formatter.rtrim(""));