add test for issue nahsra#453

use 4.2.0-SNAPSHOT
rbri · May 27, 2024 · 2eee0b3 · 2eee0b3
1 parent 1afa168
commit 2eee0b3
Show file tree

Hide file tree

Showing 2 changed files with 36 additions and 1 deletion.
diff --git a/pom.xml b/pom.xml
@@ -128,7 +128,7 @@
         <dependency>
             <groupId>org.htmlunit</groupId>
             <artifactId>neko-htmlunit</artifactId>
-            <version>4.1.0</version>
+            <version>4.2.0-SNAPSHOT</version>
         </dependency>
         <dependency>
             <groupId>org.slf4j</groupId>

diff --git a/src/test/java/org/owasp/validator/html/test/AntiSamyTest.java b/src/test/java/org/owasp/validator/html/test/AntiSamyTest.java
@@ -2666,4 +2666,39 @@ public void testBangCommentsWhenPreservingComments() throws ScanException, Polic
             .getCleanHTML(),
         not(containsString("mxss")));
   }
+
+  static final String test453 =
+      "<html lang=\"en\">\n"
+          + "<head>\n"
+          + "</head>\n"
+          + "<table>\n"
+          + "<SELECT NAME=\"Lang\">\n"
+          + "<OPTION VALUE=\"da\">Dansk</OPTION>\n"
+          + "<OPTION VALUE=\"en\" selected=selected>English</OPTION>\n"
+          + "</SELECT>\n"
+          + "</table>\n"
+          + "</html>";
+
+  @Test
+  public void testGithubIssue453() throws ScanException, PolicyException {
+    String cleaned = as.scan(test453, policy, AntiSamy.SAX).getCleanHTML();
+    cleaned = cleaned.replaceAll("\r?\n", "");
+    cleaned = cleaned.replaceAll("\\s\\s+", " ");
+
+    assertThat(
+        cleaned,
+        containsString("<body> <table> "
+                + "<select name=\"Lang\"> "
+                + "<option value=\"da\">Dansk</option> "));
+
+    cleaned = as.scan(test453, policy, AntiSamy.DOM).getCleanHTML();
+    cleaned = cleaned.replaceAll("\r?\n", "");
+    cleaned = cleaned.replaceAll("\\s\\s+", " ");
+
+    assertThat(
+        cleaned,
+        containsString("<body> <table> "
+                + "<select name=\"Lang\"> "
+                + "<option value=\"da\">Dansk</option> "));
+  }
 }