Add test for event listeners on cloneNode

BonnierNews · Jan 14, 2025 · e824147 · e824147
1 parent 543cc8d
commit e824147
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 2 deletions.
diff --git a/lib/Element.js b/lib/Element.js
@@ -396,7 +396,6 @@ module.exports = class Element extends Node {
   }
   cloneNode(deep) {
     const $clone = this.$elm.clone();
-    // TODO check for event listeners, should be empty on $clone
     if (!deep) {
       $clone.empty();
     }

diff --git a/test/elements-test.js b/test/elements-test.js
@@ -6,7 +6,6 @@ const HTMLAnchorElement = require("../lib/HTMLAnchorElement.js");
 const HTMLFormElement = require("../lib/HTMLFormElement.js");
 const { Document } = require("../lib/index.js");
 const { Event } = require("../lib/Events.js");
-const { expect } = require("chai");
 
 const elementProperties = [
   "children",
@@ -1489,6 +1488,15 @@ describe("elements", () => {
 
       expect(elmCloneChild === elmChild).to.be.false;
     });
+
+    it("returns a clone without event listerners", () => {
+      const elm = document.getElementsByClassName("block")[0];
+      let clicked = false;
+      elm.addEventListener("click", () => (clicked = true));
+      const elmClone = elm.cloneNode(true);
+      elmClone.click();
+      expect(clicked).to.be.false;
+    });
   });
 
   describe("video element", () => {