Add test for findOne that adresses previous fix

fb55 · Jan 6, 2025 · a50fa4a · a50fa4a
1 parent fab7c6e
commit a50fa4a
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 1 deletion.
diff --git a/src/querying.spec.ts b/src/querying.spec.ts
@@ -14,6 +14,9 @@ describe("querying", () => {
     const manyNodesWide = parseDocument(
         `<body>${"<div></div>".repeat(200_000)}Text</body>`,
     );
+    const someDeepNodes = parseDocument(
+        `<body><div><div></div></div><div><p></p></div></body>`,
+    );
 
     describe("find", () => {
         it("should accept many children without RangeError", () =>
@@ -107,6 +110,15 @@ describe("querying", () => {
                 ),
             ).toBe((manyNodesWide.children[0] as Element).children[0]));
 
+        it("should find elements in children in any branch", () =>
+            expect(
+                findOne(
+                    (elem) => elem.name === "p",
+                    someDeepNodes.children,
+                    true,
+                ),
+            ).toBeTruthy());
+
         it("should not find elements in children if recurse is false", () =>
             expect(
                 findOne((elem) => elem.name === "div", manyNodesWide, false),

diff --git a/src/querying.ts b/src/querying.ts
@@ -113,7 +113,7 @@ export function findOne(
         }
         if (recurse && hasChildren(node) && node.children.length > 0) {
             const found = findOne(test, node.children, true);
-            if(found) return found
+            if (found) return found;
         }
     }