Fix a bad bug in findAll (affecting FindAll, FindAllChildren, FindAll…

…Els, etc). Increase test coverage.

htree.go

@@ -89,9 +89,7 @@ func FindAll(tree *html.Node, pred func(*html.Node) bool) Seq {
 
 func findAll(node *html.Node, pred func(*html.Node) bool, yield func(*html.Node) bool) bool {
 	if pred(node) {
-		if !yield(node) {
-			return false
-		}
+		return yield(node)
 	}
 	return findAllChildren(node, pred, yield)
 }

htree_test.go

@@ -144,4 +144,29 @@ func TestHTML(t *testing.T) {
 			t.Errorf("got %v, want %v", fields, want)
 		}
 	})
+
+	t.Run("FindAllChildEls", func(t *testing.T) {
+		bodyEl := FindEl(root, func(n *html.Node) bool { return n.DataAtom == atom.Body })
+		if bodyEl == nil {
+			t.Fatal("no body")
+		}
+
+		var (
+			seq = FindAllChildEls(bodyEl, func(n *html.Node) bool { return n.DataAtom == atom.Div })
+			got []string
+		)
+		for n := range seq {
+			got = append(got, ElAttr(n, "class"))
+		}
+
+		want := []string{
+			"vector-header-container",
+			"mw-page-container",
+			"vector-settings",
+		}
+
+		if !slices.Equal(got, want) {
+			t.Errorf("got %v, want %v", got, want)
+		}
+	})
 }