Add more tests for tab expansion

These show more clearly what the code is supposed to do.

diff --git a/src/test/java/com/gitblit/tests/StringUtilsTest.java b/src/test/java/com/gitblit/tests/StringUtilsTest.java
index b836c71fbb7da92f9ca280c9c32630a9ef3b0538..cc579888b6701d20a3fbba54eb208fa5bfe69ef6 100644 (file)
--- a/src/test/java/com/gitblit/tests/StringUtilsTest.java
+++ b/src/test/java/com/gitblit/tests/StringUtilsTest.java
@@ -61,9 +61,27 @@ public class StringUtilsTest extends GitblitUnitTest {
 \r
        @Test\r
        public void testEscapeForHtml() throws Exception {\r
-               String input = "& < > \"";\r
-               String outputNoChange = "&amp; &lt; &gt; &quot;";\r
-               String outputChange = "&amp;&nbsp;&lt;&nbsp;&gt;&nbsp;&quot;";\r
+               String input = "\t & < > \"";\r
+               String outputNoChange = "\t &amp; &lt; &gt; &quot;";\r
+               String outputChange = "&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&amp;&nbsp;&lt;&nbsp;&gt;&nbsp;&quot;";\r
+               assertEquals(outputNoChange, StringUtils.escapeForHtml(input, false));\r
+               assertEquals(outputChange, StringUtils.escapeForHtml(input, true));\r
+\r
+               input = "a\tb";\r
+               outputNoChange = "a\tb";\r
+               outputChange = "a&nbsp;&nbsp;&nbsp;b";\r
+               assertEquals(outputNoChange, StringUtils.escapeForHtml(input, false));\r
+               assertEquals(outputChange, StringUtils.escapeForHtml(input, true));\r
+\r
+               input = "\ta b\t";\r
+               outputNoChange = "\ta b\t";\r
+               outputChange = "&nbsp;&nbsp;&nbsp;&nbsp;a&nbsp;b&nbsp;";\r
+               assertEquals(outputNoChange, StringUtils.escapeForHtml(input, false));\r
+               assertEquals(outputChange, StringUtils.escapeForHtml(input, true));\r
+\r
+               input = "\t <> \t";\r
+               outputNoChange = "\t &lt;&gt; \t";\r
+               outputChange = "&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&lt;&gt;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;";\r
                assertEquals(outputNoChange, StringUtils.escapeForHtml(input, false));\r
                assertEquals(outputChange, StringUtils.escapeForHtml(input, true));\r
 \r