Skip to content

Commit 74d7196

Browse files
committed
fix(core dom): Do not break querySelectorAllAndMe, if passed element is not a real element but something like a text node.
1 parent 744ec6e commit 74d7196

File tree

2 files changed

+10
-1
lines changed

2 files changed

+10
-1
lines changed

Diff for: src/core/dom.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -60,7 +60,7 @@ const toNodeArray = (nodes) => {
6060
* @returns {Array} - The DOM nodes found.
6161
*/
6262
const querySelectorAllAndMe = (el, selector) => {
63-
if (!el) {
63+
if (!el || !el.querySelectorAll) {
6464
return [];
6565
}
6666

Diff for: src/core/dom.test.js

+9
Original file line numberDiff line numberDiff line change
@@ -174,6 +174,15 @@ describe("core.dom tests", () => {
174174

175175
done();
176176
});
177+
178+
it("return empty list, if the element is not a real element.", (done) => {
179+
const res = dom.querySelectorAllAndMe("text", ".selector");
180+
expect(Array.isArray(res)).toBe(true);
181+
expect(res.length).toBe(0);
182+
183+
done();
184+
});
185+
177186
});
178187

179188
describe("wrap tests", () => {

0 commit comments

Comments
 (0)