happy-dom 17.6.2 → 18.0.0

package/src/svg/SVGPointList.ts

@@ -15,8 +15,8 @@ export default class SVGPointList {
 	[index: number]: SVGPoint;
 
 	public [PropertySymbol.window]: BrowserWindow;
-	public [PropertySymbol.getAttribute]: (() => string | null) | null = null;
-	public [PropertySymbol.setAttribute]: ((value: string) => void) | null = null;
+	public [PropertySymbol.getAttribute]: () => string | null;
+	public [PropertySymbol.setAttribute]: (value: string) => void;
 	public [PropertySymbol.readOnly]: boolean = false;
 	private [PropertySymbol.cache]: { items: SVGPoint[]; attributeValue: string } = {
 		items: [],
@@ -48,8 +48,8 @@ export default class SVGPointList {
 
 		this[PropertySymbol.window] = window;
 		this[PropertySymbol.readOnly] = !!options.readOnly;
-		this[PropertySymbol.getAttribute] = options.getAttribute || null;
-		this[PropertySymbol.setAttribute] = options.setAttribute || null;
+		this[PropertySymbol.getAttribute] = options.getAttribute;
+		this[PropertySymbol.setAttribute] = options.setAttribute;
 
 		const methodBinder = new ClassMethodBinder(this, [SVGPointList]);
 
@@ -60,7 +60,7 @@ export default class SVGPointList {
 				}
 				if (property in target || typeof property === 'symbol') {
 					methodBinder.bind(property);
-					return target[property];
+					return (<any>target)[property];
 				}
 				const index = Number(property);
 				if (!isNaN(index)) {
@@ -70,23 +70,23 @@ export default class SVGPointList {
 			set(target, property, newValue): boolean {
 				methodBinder.bind(property);
 				if (typeof property === 'symbol') {
-					target[property] = newValue;
+					(<any>target)[property] = newValue;
 					return true;
 				}
 				const index = Number(property);
 				if (isNaN(index)) {
-					target[property] = newValue;
+					(<any>target)[property] = newValue;
 				}
 				return true;
 			},
 			deleteProperty(target, property): boolean {
 				if (typeof property === 'symbol') {
-					delete target[property];
+					delete (<any>target)[property];
 					return true;
 				}
 				const index = Number(property);
 				if (isNaN(index)) {
-					delete target[property];
+					delete (<any>target)[property];
 				}
 				return true;
 			},
@@ -115,7 +115,7 @@ export default class SVGPointList {
 
 				return false;
 			},
-			getOwnPropertyDescriptor(target, property): PropertyDescriptor {
+			getOwnPropertyDescriptor(target, property): PropertyDescriptor | undefined {
 				if (property in target || typeof property === 'symbol') {
 					return;
 				}
@@ -219,8 +219,8 @@ export default class SVGPointList {
 		};
 
 		this[PropertySymbol.cache].items = [newItem];
-		this[PropertySymbol.cache].attributeValue = newItem[PropertySymbol.attributeValue];
-		this[PropertySymbol.setAttribute](newItem[PropertySymbol.attributeValue]);
+		this[PropertySymbol.cache].attributeValue = newItem[PropertySymbol.attributeValue]!;
+		this[PropertySymbol.setAttribute](newItem[PropertySymbol.attributeValue] || '');
 
 		return newItem;
 	}
@@ -231,7 +231,7 @@ export default class SVGPointList {
 	 * @param index Index.
 	 * @returns The item at the index.
 	 **/
-	public getItem(index: number | string): SVGPoint {
+	public getItem(index: number | string): SVGPoint | null {
 		const items = this[PropertySymbol.getItemList]();
 		if (typeof index === 'number') {
 			return items[index] ? items[index] : null;
@@ -505,7 +505,7 @@ export default class SVGPointList {
 				const y = parts[i + 1] !== undefined ? ' ' + parseFloat(parts[i + 1]) : '';
 				const item = new SVGPoint(PropertySymbol.illegalConstructor, this[PropertySymbol.window], {
 					readOnly: this[PropertySymbol.readOnly],
-					getAttribute: () => item[PropertySymbol.attributeValue],
+					getAttribute: (): string | null => item[PropertySymbol.attributeValue],
 					setAttribute: () => {
 						this[PropertySymbol.cache].attributeValue = this[PropertySymbol.getItemList]()
 							.map((item) => item[PropertySymbol.attributeValue] || '0 0')

package/src/svg/SVGPreserveAspectRatio.ts

@@ -84,7 +84,7 @@ export default class SVGPreserveAspectRatio {
 			return SVGPreserveAspectRatioAlignEnum.xMidYMid;
 		}
 
-		const align = attributeValue.split(/\s+/)[0];
+		const align = <'unknown'>attributeValue.split(/\s+/)[0];
 
 		if (SVGPreserveAspectRatioAlignEnum[align] === undefined) {
 			return SVGPreserveAspectRatioAlignEnum.xMidYMid;
@@ -136,7 +136,7 @@ export default class SVGPreserveAspectRatio {
 			return SVGPreserveAspectRatioMeetOrSliceEnum.meet;
 		}
 
-		const meetOrSlice = attributeValue.split(/\s+/)[1];
+		const meetOrSlice = <'unknown'>attributeValue.split(/\s+/)[1];
 
 		if (!meetOrSlice || SVGPreserveAspectRatioMeetOrSliceEnum[meetOrSlice] === undefined) {
 			return SVGPreserveAspectRatioMeetOrSliceEnum.meet;

package/src/svg/SVGStringList.ts

@@ -14,10 +14,10 @@ export default class SVGStringList {
 	[index: number]: string;
 
 	public [PropertySymbol.window]: BrowserWindow;
-	public [PropertySymbol.getAttribute]: (() => string | null) | null = null;
-	public [PropertySymbol.setAttribute]: ((value: string) => void) | null = null;
+	public [PropertySymbol.getAttribute]: () => string | null;
+	public [PropertySymbol.setAttribute]: (value: string) => void;
 	public [PropertySymbol.readOnly]: boolean = false;
-	private [PropertySymbol.cache]: { items: string[]; attributeValue: string } = {
+	private [PropertySymbol.cache]: { items: string[]; attributeValue: string | null } = {
 		items: [],
 		attributeValue: ''
 	};
@@ -47,8 +47,8 @@ export default class SVGStringList {
 
 		this[PropertySymbol.window] = window;
 		this[PropertySymbol.readOnly] = !!options.readOnly;
-		this[PropertySymbol.getAttribute] = options.getAttribute || null;
-		this[PropertySymbol.setAttribute] = options.setAttribute || null;
+		this[PropertySymbol.getAttribute] = options.getAttribute;
+		this[PropertySymbol.setAttribute] = options.setAttribute;
 
 		if (illegalConstructorSymbol !== PropertySymbol.illegalConstructor) {
 			throw new TypeError('Illegal constructor');
@@ -63,7 +63,7 @@ export default class SVGStringList {
 				}
 				if (property in target || typeof property === 'symbol') {
 					methodBinder.bind(property);
-					return target[property];
+					return (<any>target)[property];
 				}
 				const index = Number(property);
 				if (!isNaN(index)) {
@@ -73,23 +73,23 @@ export default class SVGStringList {
 			set(target, property, newValue): boolean {
 				methodBinder.bind(property);
 				if (typeof property === 'symbol') {
-					target[property] = newValue;
+					(<any>target)[property] = newValue;
 					return true;
 				}
 				const index = Number(property);
 				if (isNaN(index)) {
-					target[property] = newValue;
+					(<any>target)[property] = newValue;
 				}
 				return true;
 			},
 			deleteProperty(target, property): boolean {
 				if (typeof property === 'symbol') {
-					delete target[property];
+					delete (<any>target)[property];
 					return true;
 				}
 				const index = Number(property);
 				if (isNaN(index)) {
-					delete target[property];
+					delete (<any>target)[property];
 				}
 				return true;
 			},
@@ -118,7 +118,7 @@ export default class SVGStringList {
 
 				return false;
 			},
-			getOwnPropertyDescriptor(target, property): PropertyDescriptor {
+			getOwnPropertyDescriptor(target, property): PropertyDescriptor | undefined {
 				if (property in target || typeof property === 'symbol') {
 					return;
 				}
@@ -178,7 +178,7 @@ export default class SVGStringList {
 	 * @param newItem New item.
 	 * @returns The item being replaced.
 	 */
-	public initialize(newItem: string): string {
+	public initialize(newItem: string): string | undefined {
 		if (this[PropertySymbol.readOnly]) {
 			throw new this[PropertySymbol.window].TypeError(
 				`Failed to execute 'initialize' on 'SVGStringList': The object is read-only.`
@@ -204,7 +204,7 @@ export default class SVGStringList {
 	 * @param index Index.
 	 * @returns The item at the index.
 	 **/
-	public getItem(index: number | string): string {
+	public getItem(index: number | string): string | null {
 		const items = this[PropertySymbol.getItemList]();
 		if (typeof index === 'number') {
 			return items[index] ? items[index] : null;
@@ -403,7 +403,7 @@ export default class SVGStringList {
 		}
 
 		// It is possible to make this statement shorter by using Array.from() and Set, but this is faster when comparing using a bench test.
-		const items = [];
+		const items: string[] = [];
 		const trimmed = attributeValue.trim();
 
 		if (trimmed) {

package/src/svg/SVGTransform.ts

@@ -22,8 +22,8 @@ export default class SVGTransform {
 
 	// Internal properties
 	public [PropertySymbol.window]: BrowserWindow;
-	public [PropertySymbol.getAttribute]: () => string;
-	public [PropertySymbol.setAttribute]: (value: string) => void;
+	public [PropertySymbol.getAttribute]: (() => string | null) | null = null;
+	public [PropertySymbol.setAttribute]: ((value: string) => void) | null = null;
 	public [PropertySymbol.attributeValue]: string | null = null;
 	public [PropertySymbol.readOnly]: boolean = false;
 	public [PropertySymbol.matrix]: SVGMatrix | null = null;
@@ -199,7 +199,7 @@ export default class SVGTransform {
 			this[PropertySymbol.attributeValue] = matrix[PropertySymbol.attributeValue];
 
 			if (this[PropertySymbol.setAttribute]) {
-				this[PropertySymbol.setAttribute](this[PropertySymbol.attributeValue]);
+				this[PropertySymbol.setAttribute](this[PropertySymbol.attributeValue] || '');
 			}
 		}
 	}

package/src/svg/SVGTransformList.ts

@@ -16,10 +16,10 @@ export default class SVGTransformList {
 	[index: number]: SVGTransform;
 
 	public [PropertySymbol.window]: BrowserWindow;
-	public [PropertySymbol.getAttribute]: (() => string | null) | null = null;
-	public [PropertySymbol.setAttribute]: ((value: string) => void) | null = null;
+	public [PropertySymbol.getAttribute]: () => string | null;
+	public [PropertySymbol.setAttribute]: (value: string) => void;
 	public [PropertySymbol.readOnly]: boolean = false;
-	private [PropertySymbol.cache]: { items: SVGTransform[]; attributeValue: string } = {
+	private [PropertySymbol.cache]: { items: SVGTransform[]; attributeValue: string | null } = {
 		items: [],
 		attributeValue: ''
 	};
@@ -61,7 +61,7 @@ export default class SVGTransformList {
 				}
 				if (property in target || typeof property === 'symbol') {
 					methodBinder.bind(property);
-					return target[property];
+					return (<any>target)[property];
 				}
 				const index = Number(property);
 				if (!isNaN(index)) {
@@ -71,23 +71,23 @@ export default class SVGTransformList {
 			set(target, property, newValue): boolean {
 				methodBinder.bind(property);
 				if (typeof property === 'symbol') {
-					target[property] = newValue;
+					(<any>target)[property] = newValue;
 					return true;
 				}
 				const index = Number(property);
 				if (isNaN(index)) {
-					target[property] = newValue;
+					(<any>target)[property] = newValue;
 				}
 				return true;
 			},
 			deleteProperty(target, property): boolean {
 				if (typeof property === 'symbol') {
-					delete target[property];
+					delete (<any>target)[property];
 					return true;
 				}
 				const index = Number(property);
 				if (isNaN(index)) {
-					delete target[property];
+					delete (<any>target)[property];
 				}
 				return true;
 			},
@@ -116,7 +116,7 @@ export default class SVGTransformList {
 
 				return false;
 			},
-			getOwnPropertyDescriptor(target, property): PropertyDescriptor {
+			getOwnPropertyDescriptor(target, property): PropertyDescriptor | undefined {
 				if (property in target || typeof property === 'symbol') {
 					return;
 				}
@@ -221,7 +221,7 @@ export default class SVGTransformList {
 
 		this[PropertySymbol.cache].items = [newItem];
 		this[PropertySymbol.cache].attributeValue = newItem[PropertySymbol.attributeValue];
-		this[PropertySymbol.setAttribute](newItem[PropertySymbol.attributeValue]);
+		this[PropertySymbol.setAttribute](newItem[PropertySymbol.attributeValue] || '');
 
 		return newItem;
 	}
@@ -232,7 +232,7 @@ export default class SVGTransformList {
 	 * @param index Index.
 	 * @returns The item at the index.
 	 **/
-	public getItem(index: number | string): SVGTransform {
+	public getItem(index: number | string): SVGTransform | null {
 		const items = this[PropertySymbol.getItemList]();
 		if (typeof index === 'number') {
 			return items[index] ? items[index] : null;
@@ -509,7 +509,7 @@ export default class SVGTransformList {
 					this[PropertySymbol.window],
 					{
 						readOnly: this[PropertySymbol.readOnly],
-						getAttribute: () => item[PropertySymbol.attributeValue],
+						getAttribute: (): string | null => item[PropertySymbol.attributeValue],
 						setAttribute: () => {
 							this[PropertySymbol.cache].attributeValue = this[PropertySymbol.getItemList]()
 								.map((item) => item[PropertySymbol.attributeValue] || EMPTY_MATRIX)

package/src/tree-walker/NodeIterator.ts

@@ -11,9 +11,9 @@ import NodeFilter from './NodeFilter.js';
  * https://developer.mozilla.org/en-US/docs/Web/API/NodeIterator
  */
 export default class NodeIterator {
-	#root: Node = null;
+	#root: Node;
 	#whatToShow = -1;
-	#filter: INodeFilter = null;
+	#filter: INodeFilter | null = null;
 	#walker: TreeWalker;
 	#atRoot = true;
 
@@ -24,7 +24,7 @@ export default class NodeIterator {
 	 * @param [whatToShow] What to show.
 	 * @param [filter] Filter.
 	 */
-	constructor(root: Node, whatToShow = -1, filter: INodeFilter = null) {
+	constructor(root: Node, whatToShow = -1, filter: INodeFilter | null = null) {
 		this.#root = root;
 		this.#whatToShow = whatToShow;
 		this.#filter = filter;
@@ -54,7 +54,7 @@ export default class NodeIterator {
 	 *
 	 * @returns Filter.
 	 */
-	public get filter(): INodeFilter {
+	public get filter(): INodeFilter | null {
 		return this.#filter;
 	}
 
@@ -63,7 +63,7 @@ export default class NodeIterator {
 	 *
 	 * @returns Current node.
 	 */
-	public nextNode(): Node {
+	public nextNode(): Node | null {
 		if (this.#atRoot) {
 			this.#atRoot = false;
 			if (this.#walker[PropertySymbol.filterNode](this.#root) !== NodeFilter.FILTER_ACCEPT) {
@@ -79,7 +79,7 @@ export default class NodeIterator {
 	 *
 	 * @returns Current node.
 	 */
-	public previousNode(): Node {
+	public previousNode(): Node | null {
 		return this.#walker.previousNode();
 	}
 }

package/src/tree-walker/TreeWalker.ts

@@ -22,10 +22,10 @@ enum TraverseSiblingsTypeEnum {
  * @see https://dom.spec.whatwg.org/#interface-treewalker
  */
 export default class TreeWalker {
-	public root: Node = null;
+	public root: Node;
 	public whatToShow = -1;
-	public filter: INodeFilter = null;
-	public currentNode: Node = null;
+	public filter: INodeFilter | null = null;
+	public currentNode: Node | null = null;
 
 	/**
 	 * Constructor.
@@ -34,7 +34,7 @@ export default class TreeWalker {
 	 * @param [whatToShow] What to show.
 	 * @param [filter] Filter.
 	 */
-	constructor(root: Node, whatToShow = -1, filter: INodeFilter = null) {
+	constructor(root: Node, whatToShow = -1, filter: INodeFilter | null = null) {
 		if (!(root instanceof Node)) {
 			throw new DOMException('Parameter 1 was not of type Node.');
 		}
@@ -50,7 +50,7 @@ export default class TreeWalker {
 	 *
 	 * @returns Current node.
 	 */
-	public parentNode(): Node {
+	public parentNode(): Node | null {
 		let node = this.currentNode;
 		while (node !== null && node !== this.root) {
 			node = node.parentNode;
@@ -67,7 +67,7 @@ export default class TreeWalker {
 	 *
 	 * @returns Current node.
 	 */
-	public firstChild(): Node {
+	public firstChild(): Node | null {
 		return this.#traverseChildren(TraverseChildrenTypeEnum.first);
 	}
 
@@ -76,7 +76,7 @@ export default class TreeWalker {
 	 *
 	 * @returns Current node.
 	 */
-	public lastChild(): Node {
+	public lastChild(): Node | null {
 		return this.#traverseChildren(TraverseChildrenTypeEnum.last);
 	}
 
@@ -85,7 +85,7 @@ export default class TreeWalker {
 	 *
 	 * @returns Current node.
 	 */
-	public nextSibling(): Node {
+	public nextSibling(): Node | null {
 		return this.#traverseSiblings(TraverseSiblingsTypeEnum.next);
 	}
 
@@ -94,7 +94,7 @@ export default class TreeWalker {
 	 *
 	 * @returns Current node.
 	 */
-	public previousSibling(): Node {
+	public previousSibling(): Node | null {
 		return this.#traverseSiblings(TraverseSiblingsTypeEnum.previous);
 	}
 
@@ -103,18 +103,18 @@ export default class TreeWalker {
 	 *
 	 * @returns Current node.
 	 */
-	public previousNode(): Node {
+	public previousNode(): Node | null {
 		let node = this.currentNode;
 
 		while (node !== this.root) {
-			let sibling = node.previousSibling;
+			let sibling = node?.previousSibling || null;
 
 			while (sibling !== null) {
 				let node = sibling;
 				let result = this[PropertySymbol.filterNode](node);
 
 				while (result !== NodeFilter.FILTER_REJECT && node[PropertySymbol.nodeArray].length) {
-					node = node.lastChild;
+					node = node.lastChild!;
 					result = this[PropertySymbol.filterNode](node);
 				}
 
@@ -126,11 +126,11 @@ export default class TreeWalker {
 				sibling = node.previousSibling;
 			}
 
-			if (node === this.root || node.parentNode === null) {
+			if (node === this.root || node!.parentNode === null) {
 				return null;
 			}
 
-			node = node.parentNode;
+			node = node!.parentNode;
 
 			if (this[PropertySymbol.filterNode](node) === NodeFilter.FILTER_ACCEPT) {
 				this.currentNode = node;
@@ -147,13 +147,17 @@ export default class TreeWalker {
 	 * @returns Current node.
 	 */
 	public nextNode(): Node | null {
-		let node = this.currentNode;
+		let node: Node | null = this.currentNode;
 		let result = NodeFilter.FILTER_ACCEPT;
 
+		if (node === null) {
+			return null;
+		}
+
 		while (true) {
-			while (result !== NodeFilter.FILTER_REJECT && node[PropertySymbol.nodeArray].length) {
-				node = node.firstChild;
-				result = this[PropertySymbol.filterNode](node);
+			while (result !== NodeFilter.FILTER_REJECT && node![PropertySymbol.nodeArray].length) {
+				node = node!.firstChild;
+				result = this[PropertySymbol.filterNode](node!);
 
 				if (result === NodeFilter.FILTER_ACCEPT) {
 					this.currentNode = node;
@@ -166,7 +170,7 @@ export default class TreeWalker {
 					return null;
 				}
 
-				const sibling = node.nextSibling;
+				const sibling: Node | null = node.nextSibling;
 
 				if (sibling !== null) {
 					node = sibling;
@@ -199,7 +203,7 @@ export default class TreeWalker {
 	 * @returns Child nodes.
 	 */
 	public [PropertySymbol.filterNode](node: Node): number {
-		const mask = NodeFilterMask[node.nodeType];
+		const mask = NodeFilterMask[<1>node.nodeType];
 
 		if (mask && (this.whatToShow & mask) == 0) {
 			return NodeFilter.FILTER_SKIP;
@@ -221,7 +225,12 @@ export default class TreeWalker {
 	 * @returns Node.
 	 */
 	#traverseChildren(type: TraverseChildrenTypeEnum): Node | null {
-		let node: Node = this.currentNode;
+		let node = this.currentNode;
+
+		if (!node) {
+			return null;
+		}
+
 		node = type === TraverseChildrenTypeEnum.first ? node.firstChild : node.lastChild;
 
 		while (node !== null) {
@@ -248,7 +257,7 @@ export default class TreeWalker {
 					node = sibling;
 					break;
 				}
-				const parent = node.parentNode;
+				const parent: Node | null = node.parentNode;
 				if (parent === null || parent === this.root || parent === this.currentNode) {
 					return null;
 				}
@@ -266,9 +275,9 @@ export default class TreeWalker {
 	 * @returns Node.
 	 */
 	#traverseSiblings(type: TraverseSiblingsTypeEnum): Node | null {
-		let node: Node = this.currentNode;
+		let node: Node | null = this.currentNode;
 
-		if (node === this.root) {
+		if (!node || node === this.root) {
 			return null;
 		}
 

package/src/validity-state/ValidityState.ts

@@ -80,7 +80,7 @@ export default class ValidityState {
 			this.element instanceof HTMLInputElement &&
 			this.element.hasAttribute('pattern') &&
 			this.element.value.length > 0 &&
-			this.element.value.replace(new RegExp(this.element.getAttribute('pattern')), '').length > 0
+			this.element.value.replace(new RegExp(this.element.getAttribute('pattern')!), '').length > 0
 		);
 	}