fix: [#1285] Prevent contains from accessing properties of undefined arguments

Author: rofly

packages/happy-dom/src/nodes/node/Node.ts

@@ -298,7 +298,10 @@ export default class Node extends EventTarget implements INode {
 	 * @param otherNode Node to test with.
 	 * @returns "true" if this node contains the other node.
 	 */
-	public contains(otherNode: INode): boolean {
+	public contains(otherNode: INode | undefined): boolean {
+		if (otherNode === undefined) {
+			return false;
+		}
 		return NodeUtility.isInclusiveAncestor(this, otherNode);
 	}
 

packages/happy-dom/test/nodes/node/Node.test.ts

@@ -330,6 +330,11 @@ describe('Node', () => {
 
 			expect(div.contains(null)).toBe(false);
 		});
+		it('Returns "false" if match node is undefined.', () => {
+			const div = document.createElement('div');
+
+			expect(div.contains(undefined)).toBe(false);
+		});
 	});
 
 	describe('getRootNode()', () => {