universal selector should only match element node

Author: syjer

src/main/java/ch/digitalfondue/jfiveparse/Selector.java

@@ -160,12 +160,12 @@ public Selector element(String name) {
     }
 
     /**
-     * Universal selector "*". It matches any node.
+     * Universal selector "*". It matches any _element_ node.
      *
      * @return
      */
     public Selector universal() {
-        matchers.add(n -> true);
+        matchers.add(n -> n.getNodeType() == Node.ELEMENT_NODE);
         return this;
     }
 

src/test/java/ch/digitalfondue/jfiveparse/NodeMatchersTest.java

@@ -50,6 +50,15 @@ void universalTest() {
         assertEquals(doc.getElementsByTagName("body").get(0).getChildNodes().get(1).getChildNodes().get(0), universal.get(0));
     }
 
+    @Test
+    void universalWithSpaces() {
+        Document doc = parser.parse(" <div> </div> <div id=myid> <span> <i> </i> </span> </div> <div> </div> ");
+        // div > *
+        var universal = doc.getElementsByTagName("body").get(0).getAllNodesMatching(Selector.select().element("div").withChild().universal().toMatcher());
+        assertEquals(1, universal.size());
+        assertEquals("span", universal.get(0).getNodeName());
+    }
+
     @Test
     void firstElementTest() {
         Document doc = parser.parse("text<div></div><div id=myid></div><div></div>");