happy-dom 15.7.0 → 15.7.2

package/src/query-selector/QuerySelector.ts

@@ -13,7 +13,6 @@ import ISVGElementTagNameMap from '../config/ISVGElementTagNameMap.js';
 import ICachedQuerySelectorAllItem from '../nodes/node/ICachedQuerySelectorAllResult.js';
 import ICachedQuerySelectorItem from '../nodes/node/ICachedQuerySelectorResult.js';
 import ICachedMatchesItem from '../nodes/node/ICachedMatchesResult.js';
-import DOMExceptionNameEnum from '../exception/DOMExceptionNameEnum.js';
 
 type DocumentPositionAndElement = {
 	documentPosition: string;
@@ -78,27 +77,32 @@ export default class QuerySelector {
 		node: Element | Document | DocumentFragment,
 		selector: string
 	): NodeList<Element> {
-		if (selector === null || selector === undefined) {
-			return new NodeList<Element>(PropertySymbol.illegalConstructor, []);
-		}
-
 		const window = node[PropertySymbol.window];
 
 		if (<string>selector === '') {
-			throw new window.Error(
+			throw new window.DOMException(
 				`Failed to execute 'querySelectorAll' on '${node.constructor.name}': The provided selector is empty.`
 			);
 		}
 
-		if (typeof selector !== 'string' && typeof selector !== 'boolean') {
+		if (typeof selector === 'function') {
 			throw new window.DOMException(
-				`Failed to execute 'querySelectorAll' on '${node.constructor.name}': '${selector}' is not a valid selector.`,
-				'SyntaxError'
+				`Failed to execute 'querySelectorAll' on '${node.constructor.name}': '${selector}' is not a valid selector.`
 			);
 		}
 
+		if (typeof selector === 'symbol') {
+			throw new window.TypeError(`Cannot convert a Symbol value to a string`);
+		}
+
 		selector = String(selector);
 
+		if (INVALID_SELECTOR_REGEXP.test(selector)) {
+			throw new window.DOMException(
+				`Failed to execute 'querySelectorAll' on '${node.constructor.name}': '${selector}' is not a valid selector.`
+			);
+		}
+
 		const cache = node[PropertySymbol.cache].querySelectorAll;
 		const cachedResult = cache.get(selector);
 
@@ -109,12 +113,6 @@ export default class QuerySelector {
 			}
 		}
 
-		if (INVALID_SELECTOR_REGEXP.test(selector)) {
-			throw new window.Error(
-				`Failed to execute 'querySelectorAll' on '${node.constructor.name}': '${selector}' is not a valid selector.`
-			);
-		}
-
 		const groups = SelectorParser.getSelectorGroups(selector);
 		const items: Element[] = [];
 		const nodeList = new NodeList<Element>(PropertySymbol.illegalConstructor, items);
@@ -199,27 +197,38 @@ export default class QuerySelector {
 		node: Element | Document | DocumentFragment,
 		selector: string
 	): Element | null {
-		if (selector === null || selector === undefined) {
-			return null;
-		}
-
 		const window = node[PropertySymbol.window];
 
 		if (selector === '') {
-			throw new window.Error(
+			throw new window.DOMException(
 				`Failed to execute 'querySelector' on '${node.constructor.name}': The provided selector is empty.`
 			);
 		}
 
-		if (typeof selector !== 'string' && typeof selector !== 'boolean') {
+		if (typeof selector === 'function' || typeof selector === 'symbol') {
 			throw new window.DOMException(
-				`Failed to execute 'querySelector' on '${node.constructor.name}': '${selector}' is not a valid selector.`,
-				'SyntaxError'
+				`Failed to execute 'querySelector' on '${node.constructor.name}': '${selector}' is not a valid selector.`
+			);
+		}
+
+		if (typeof selector === 'function') {
+			throw new window.DOMException(
+				`Failed to execute 'querySelector' on '${node.constructor.name}': '${selector}' is not a valid selector.`
 			);
 		}
 
+		if (typeof selector === 'symbol') {
+			throw new window.TypeError(`Cannot convert a Symbol value to a string`);
+		}
+
 		selector = String(selector);
 
+		if (INVALID_SELECTOR_REGEXP.test(selector)) {
+			throw new window.DOMException(
+				`Failed to execute 'querySelector' on '${node.constructor.name}': '${selector}' is not a valid selector.`
+			);
+		}
+
 		const cachedResult = node[PropertySymbol.cache].querySelector.get(selector);
 
 		if (cachedResult?.result) {
@@ -229,12 +238,6 @@ export default class QuerySelector {
 			}
 		}
 
-		if (INVALID_SELECTOR_REGEXP.test(selector)) {
-			throw new window.Error(
-				`Failed to execute 'querySelector' on '${node.constructor.name}': '${selector}' is not a valid selector.`
-			);
-		}
-
 		const cachedItem: ICachedQuerySelectorItem = {
 			result: <WeakRef<Element | null>>{
 				deref: () => null
@@ -277,55 +280,57 @@ export default class QuerySelector {
 		selector: string,
 		options?: { ignoreErrors?: boolean }
 	): ISelectorMatch | null {
-		if (!selector) {
-			return null;
-		}
+		const ignoreErrors = options?.ignoreErrors;
+		const window = element[PropertySymbol.window];
 
-		if (selector === null || selector === undefined) {
+		if (selector === '*') {
 			return {
-				priorityWeight: 0
+				priorityWeight: 1
 			};
 		}
 
-		const window = element[PropertySymbol.window];
-
 		if (<string>selector === '') {
-			throw new window.Error(
+			if (ignoreErrors) {
+				return null;
+			}
+			throw new window.DOMException(
 				`Failed to execute 'matches' on '${element.constructor.name}': The provided selector is empty.`
 			);
 		}
 
-		if (typeof selector !== 'string' && typeof selector !== 'boolean') {
+		if (typeof selector === 'function') {
+			if (ignoreErrors) {
+				return null;
+			}
 			throw new window.DOMException(
-				`Failed to execute 'matches' on '${element.constructor.name}': '${selector}' is not a valid selector.`,
-				DOMExceptionNameEnum.syntaxError
+				`Failed to execute 'matches' on '${element.constructor.name}': '${selector}' is not a valid selector.`
 			);
 		}
 
-		selector = String(selector);
-
-		if (selector === '*') {
-			return {
-				priorityWeight: 1
-			};
+		if (typeof selector === 'symbol') {
+			if (ignoreErrors) {
+				return null;
+			}
+			throw new window.TypeError(`Cannot convert a Symbol value to a string`);
 		}
 
-		const ignoreErrors = options?.ignoreErrors;
-		const cachedResult = element[PropertySymbol.cache].matches.get(selector);
-
-		if (cachedResult?.result) {
-			return cachedResult.result.match;
-		}
+		selector = String(selector);
 
 		if (INVALID_SELECTOR_REGEXP.test(selector)) {
 			if (ignoreErrors) {
 				return null;
 			}
-			throw new window.Error(
+			throw new window.DOMException(
 				`Failed to execute 'matches' on '${element.constructor.name}': '${selector}' is not a valid selector.`
 			);
 		}
 
+		const cachedResult = element[PropertySymbol.cache].matches.get(selector);
+
+		if (cachedResult?.result) {
+			return cachedResult.result.match;
+		}
+
 		const cachedItem: ICachedMatchesItem = {
 			result: { match: null }
 		};

package/src/window/BrowserWindow.ts

@@ -1462,7 +1462,9 @@ export default class BrowserWindow extends EventTarget implements INodeJSGlobal
 		const mutationObservers = this[PropertySymbol.mutationObservers];
 
 		for (const mutationObserver of mutationObservers) {
-			mutationObserver.disconnect();
+			if (mutationObserver[PropertySymbol.destroy]) {
+				mutationObserver[PropertySymbol.destroy]();
+			}
 		}
 
 		this[PropertySymbol.mutationObservers] = [];