Ensure the wrap width is never exceeded and cut the words longer that the width if necessary (CLI-193)

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

Author: ebourg

src/main/java/org/apache/commons/cli/HelpFormatter.java

@@ -889,7 +889,8 @@ protected StringBuffer renderWrappedText(StringBuffer sb, int width,
      * Finds the next text wrap position after <code>startPos</code> for the
      * text in <code>text</code> with the column width <code>width</code>.
      * The wrap point is the last position before startPos+width having a 
-     * whitespace character (space, \n, \r).
+     * whitespace character (space, \n, \r). If there is no whitespace character
+     * before startPos+width, it will return startPos+width.
      *
      * @param text The text being searched for the wrap position
      * @param width width of the wrapped text
@@ -900,8 +901,8 @@ protected StringBuffer renderWrappedText(StringBuffer sb, int width,
      */
     protected int findWrapPos(String text, int width, int startPos)
     {
-        int pos = -1;
-
+        int pos;
+        
         // the line ends before the max wrap pos or a new line char found
         if (((pos = text.indexOf('\n', startPos)) != -1 && pos <= width)
                 || ((pos = text.indexOf('\t', startPos)) != -1 && pos <= width))
@@ -931,17 +932,10 @@ else if (startPos + width >= text.length())
             return pos;
         }
 
-        // must look for the first whitespace chearacter after startPos 
-        // + width
+        // if we didn't find one, simply chop at startPos+width
         pos = startPos + width;
-
-        while ((pos <= text.length()) && ((c = text.charAt(pos)) != ' ')
-               && (c != '\n') && (c != '\r'))
-        {
-            ++pos;
-        }
-
-        return (pos == text.length()) ? (-1) : pos;
+        
+        return pos == text.length() ? -1 : pos;
     }
 
     /**

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

@@ -46,9 +46,27 @@ public void testFindWrapPos() throws Exception
         // starting from 8 must give -1 - the wrap pos is after end
         assertEquals("wrap position 2", -1, hf.findWrapPos(text, 8, 8));
 
-        // if there is no a good position before width to make a wrapping look for the next one
+        // words longer than the width are cut
         text = "aaaa aa";
-        assertEquals("wrap position 3", 4, hf.findWrapPos(text, 3, 0));
+        assertEquals("wrap position 3", 3, hf.findWrapPos(text, 3, 0));
+        
+        // last word length is equal to the width
+        text = "aaaaaa aaaaaa";
+        assertEquals("wrap position 4", 6, hf.findWrapPos(text, 6, 0));
+        assertEquals("wrap position 4", -1, hf.findWrapPos(text, 6, 7));
+    }
+
+    public void testRenderWrappedTextWordCut()
+    {
+        int width = 7;
+        int padding = 0;
+        String text = "Thisisatest.";
+        String expected = "Thisisa" + EOL + 
+                          "test.";
+        
+        StringBuffer sb = new StringBuffer();
+        new HelpFormatter().renderWrappedText(sb, width, padding, text);
+        assertEquals("cut and wrap", expected, sb.toString());
     }
 
     public void testRenderWrappedTextSingleLine()