happy-dom 13.6.2 → 13.7.1

package/src/fetch/utilities/FetchBodyUtility.ts

@@ -9,29 +9,12 @@ import DOMExceptionNameEnum from '../../exception/DOMExceptionNameEnum.js';
 import IRequestBody from '../types/IRequestBody.js';
 import IResponseBody from '../types/IResponseBody.js';
 import { Buffer } from 'buffer';
+import Stream from 'stream';
 
 /**
  * Fetch body utility.
  */
 export default class FetchBodyUtility {
-	/**
-	 * Wraps a given value in a browser ReadableStream.
-	 *
-	 * This method creates a ReadableStream and immediately enqueues and closes it
-	 * with the provided value, useful for stream API compatibility.
-	 *
-	 * @param value The value to be wrapped in a ReadableStream.
-	 * @returns ReadableStream
-	 */
-	public static toReadableStream(value): ReadableStream {
-		return new ReadableStream({
-			start(controller) {
-				controller.enqueue(value);
-				controller.close();
-			}
-		});
-	}
-
 	/**
 	 * Parses body and returns stream and type.
 	 *
@@ -115,8 +98,8 @@ export default class FetchBodyUtility {
 	 * It creates a pass through stream and pipes the original stream to it.
 	 *
 	 * @param requestOrResponse Request or Response.
-	 * @param requestOrResponse.body
-	 * @param requestOrResponse.bodyUsed
+	 * @param requestOrResponse.body Body.
+	 * @param requestOrResponse.bodyUsed Body used.
 	 * @returns New stream.
 	 */
 	public static cloneBodyStream(requestOrResponse: {
@@ -130,7 +113,25 @@ export default class FetchBodyUtility {
 			);
 		}
 
+		// If a buffer is set, use it to create a new stream.
+		if (requestOrResponse[PropertySymbol.buffer]) {
+			return this.toReadableStream(requestOrResponse[PropertySymbol.buffer]);
+		}
+
+		// Pipe underlying node stream if it exists.
+		if (requestOrResponse.body[PropertySymbol.nodeStream]) {
+			const stream1 = new Stream.PassThrough();
+			const stream2 = new Stream.PassThrough();
+			requestOrResponse.body[PropertySymbol.nodeStream].pipe(stream1);
+			requestOrResponse.body[PropertySymbol.nodeStream].pipe(stream2);
+			// Sets the body of the cloned request/response to the first pass through stream.
+			requestOrResponse.body = this.nodeToWebStream(stream1);
+			// Returns the clone.
+			return this.nodeToWebStream(stream2);
+		}
+
 		// Uses the tee() method to clone the ReadableStream
+		// This requires the stream to be consumed in parallel which is not the case for the fetch API
 		const [stream1, stream2] = requestOrResponse.body.tee();
 
 		// Sets the body of the cloned request to the first pass through stream.
@@ -198,4 +199,50 @@ export default class FetchBodyUtility {
 			);
 		}
 	}
+	/**
+	 * Wraps a given value in a browser ReadableStream.
+	 *
+	 * This method creates a ReadableStream and immediately enqueues and closes it
+	 * with the provided value, useful for stream API compatibility.
+	 *
+	 * @param value The value to be wrapped in a ReadableStream.
+	 * @returns ReadableStream
+	 */
+	public static toReadableStream(value): ReadableStream {
+		return new ReadableStream({
+			start(controller) {
+				controller.enqueue(value);
+				controller.close();
+			}
+		});
+	}
+
+	/**
+	 * Wraps a Node.js stream into a browser-compatible ReadableStream.
+	 *
+	 * Enables the use of Node.js streams where browser ReadableStreams are required.
+	 * Handles 'data', 'end', and 'error' events from the Node.js stream.
+	 *
+	 * @param nodeStream The Node.js stream to be converted.
+	 * @returns ReadableStream
+	 */
+	public static nodeToWebStream(nodeStream: Stream): ReadableStream {
+		const readableStream = new ReadableStream({
+			start(controller) {
+				nodeStream.on('data', (chunk) => {
+					controller.enqueue(chunk);
+				});
+
+				nodeStream.on('end', () => {
+					controller.close();
+				});
+
+				nodeStream.on('error', (err) => {
+					controller.error(err);
+				});
+			}
+		});
+		readableStream[PropertySymbol.nodeStream] = nodeStream;
+		return readableStream;
+	}
 }

package/src/fetch/utilities/FetchRequestHeaderUtility.ts

@@ -6,6 +6,7 @@ import Headers from '../Headers.js';
 import Request from '../Request.js';
 import IHeaders from '../types/IHeaders.js';
 import FetchCORSUtility from './FetchCORSUtility.js';
+import { URL } from 'url';
 
 const FORBIDDEN_HEADER_NAMES = [
 	'accept-charset',
@@ -77,10 +78,8 @@ export default class FetchRequestHeaderUtility {
 		request: Request;
 	}): { [key: string]: string } {
 		const headers = new Headers(options.request.headers);
-		const isCORS = FetchCORSUtility.isCORS(
-			options.window.location,
-			options.request[PropertySymbol.url]
-		);
+		const originURL = new URL(options.window.location.href);
+		const isCORS = FetchCORSUtility.isCORS(originURL, options.request[PropertySymbol.url]);
 
 		// TODO: Maybe we need to add support for OPTIONS request with 'Access-Control-Allow-*' headers?
 		if (
@@ -107,7 +106,7 @@ export default class FetchRequestHeaderUtility {
 			(options.request.credentials === 'same-origin' && !isCORS)
 		) {
 			const cookies = options.browserFrame.page.context.cookieContainer.getCookies(
-				options.window.location,
+				originURL,
 				false
 			);
 			if (cookies.length > 0) {

package/src/fetch/utilities/FetchRequestReferrerUtility.ts

@@ -55,7 +55,8 @@ export default class FetchRequestReferrerUtility {
 		if (referrer === '' || referrer === 'no-referrer' || referrer === 'client') {
 			return referrer;
 		} else if (referrer) {
-			const referrerURL = referrer instanceof URL ? referrer : new URL(referrer, window.location);
+			const referrerURL =
+				referrer instanceof URL ? referrer : new URL(referrer, window.location.href);
 			return referrerURL.origin === window.location.origin ? referrerURL : 'client';
 		}
 

package/src/form-data/FormData.ts

@@ -59,7 +59,7 @@ export default class FormData implements Iterable<[string, string | File]> {
 									this.append(node.name, file);
 								}
 							}
-						} else {
+						} else if (node.value) {
 							this.append(node.name, node.value);
 						}
 					}
@@ -73,7 +73,7 @@ export default class FormData implements Iterable<[string, string | File]> {
 	 *
 	 * @param callback Callback.
 	 */
-	public forEach(callback: (key: string, value: string | File, thisArg: FormData) => void): void {
+	public forEach(callback: (value: string | File, key: string, thisArg: FormData) => void): void {
 		for (const entry of this.#entries) {
 			callback.call(this, entry.value, entry.name, this);
 		}

package/src/index.ts

@@ -40,8 +40,10 @@ import ClipboardEvent from './event/events/ClipboardEvent.js';
 import CustomEvent from './event/events/CustomEvent.js';
 import ErrorEvent from './event/events/ErrorEvent.js';
 import FocusEvent from './event/events/FocusEvent.js';
+import HashChangeEvent from './event/events/HashChangeEvent.js';
 import InputEvent from './event/events/InputEvent.js';
 import KeyboardEvent from './event/events/KeyboardEvent.js';
+import MediaQueryListEvent from './event/events/MediaQueryListEvent.js';
 import MouseEvent from './event/events/MouseEvent.js';
 import ProgressEvent from './event/events/ProgressEvent.js';
 import SubmitEvent from './event/events/SubmitEvent.js';
@@ -115,7 +117,7 @@ import Storage from './storage/Storage.js';
 import NodeFilter from './tree-walker/NodeFilter.js';
 import NodeIterator from './tree-walker/NodeIterator.js';
 import TreeWalker from './tree-walker/TreeWalker.js';
-import Location from './url/Location.js';
+import Location from './location/Location.js';
 import URL from './url/URL.js';
 import BrowserWindow from './window/BrowserWindow.js';
 import GlobalWindow from './window/GlobalWindow.js';
@@ -138,8 +140,10 @@ import type IClipboardEventInit from './event/events/IClipboardEventInit.js';
 import type ICustomEventInit from './event/events/ICustomEventInit.js';
 import type IErrorEventInit from './event/events/IErrorEventInit.js';
 import type IFocusEventInit from './event/events/IFocusEventInit.js';
+import type IHashChangeEventInit from './event/events/IHashChangeEventInit.js';
 import type IInputEventInit from './event/events/IInputEventInit.js';
 import type IKeyboardEventInit from './event/events/IKeyboardEventInit.js';
+import type IMediaQueryListInit from './event/events/IMediaQueryListInit.js';
 import type IMouseEventInit from './event/events/IMouseEventInit.js';
 import type IProgressEventInit from './event/events/IProgressEventInit.js';
 import type ISubmitEventInit from './event/events/ISubmitEventInit.js';
@@ -237,9 +241,11 @@ export type {
 	IHTMLTemplateElement,
 	IHTMLTextAreaElement,
 	IHTMLVideoElement,
+	IHashChangeEventInit,
 	IHeaders,
 	IInputEventInit,
 	IKeyboardEventInit,
+	IMediaQueryListInit,
 	IMouseEventInit,
 	INode,
 	INodeList,
@@ -388,12 +394,14 @@ export {
 	HTMLElement as HTMLUListElement,
 	HTMLUnknownElement,
 	HTMLVideoElement,
+	HashChangeEvent,
 	Headers,
 	History,
 	Image,
 	InputEvent,
 	KeyboardEvent,
 	Location,
+	MediaQueryListEvent,
 	MouseEvent,
 	MutationObserver,
 	MutationRecord,

package/src/location/Location.ts

@@ -0,0 +1,257 @@
+import IBrowserFrame from '../browser/types/IBrowserFrame.js';
+import HashChangeEvent from '../event/events/HashChangeEvent.js';
+import * as PropertySymbol from '../PropertySymbol.js';
+import { URL } from 'url';
+
+/**
+ * Location.
+ */
+export default class Location {
+	// Public properties
+	public [Symbol.toStringTag] = 'Location';
+
+	// Private properties
+	#browserFrame: IBrowserFrame;
+	#url: URL;
+
+	/**
+	 * Constructor.
+	 *
+	 * @param browserFrame Browser frame.
+	 * @param url URL.
+	 */
+	constructor(browserFrame: IBrowserFrame, url: string) {
+		this.#browserFrame = browserFrame;
+		this.#url = new URL(url);
+	}
+
+	/**
+	 * Returns hash.
+	 *
+	 * @returns Hash.
+	 */
+	public get hash(): string {
+		return this.#url.hash;
+	}
+
+	/**
+	 * Sets hash.
+	 *
+	 * @param hash Value.
+	 */
+	public set hash(hash: string) {
+		const oldURL = this.#url.href;
+		this.#url.hash = hash;
+		const newURL = this.#url.href;
+		if (newURL !== oldURL) {
+			this.#browserFrame.window?.dispatchEvent(
+				new HashChangeEvent('hashchange', { oldURL, newURL })
+			);
+		}
+	}
+
+	/**
+	 * Returns host.
+	 *
+	 * @returns Host.
+	 */
+	public get host(): string {
+		return this.#url.host;
+	}
+
+	/**
+	 * Sets host.
+	 *
+	 * @param host Value.
+	 */
+	public set host(host: string) {
+		const url = new URL(this.#url.href);
+		url.host = host;
+		this.#browserFrame
+			.goto(url.href)
+			.catch((error) => this.#browserFrame.page.console.error(error));
+	}
+
+	/**
+	 * Returns hostname.
+	 *
+	 * @returns Hostname.
+	 */
+	public get hostname(): string {
+		return this.#url.hostname;
+	}
+
+	/**
+	 * Sets hostname.
+	 *
+	 * @param hostname Value.
+	 */
+	public set hostname(hostname: string) {
+		const url = new URL(this.#url.href);
+		url.hostname = hostname;
+		this.#browserFrame
+			.goto(url.href)
+			.catch((error) => this.#browserFrame.page.console.error(error));
+	}
+
+	/**
+	 * Override set href.
+	 */
+	public get href(): string {
+		return this.#url.href;
+	}
+
+	/**
+	 * Override set href.
+	 */
+	public set href(url: string) {
+		this.#browserFrame.goto(url).catch((error) => this.#browserFrame.page.console.error(error));
+	}
+
+	/**
+	 * Returns origin.
+	 *
+	 * @returns Origin.
+	 */
+	public get origin(): string {
+		return this.#url.origin;
+	}
+
+	/**
+	 * Returns pathname
+	 *
+	 * @returns Pathname.
+	 */
+	public get pathname(): string {
+		return this.#url.pathname;
+	}
+
+	/**
+	 * Sets pathname.
+	 *
+	 * @param pathname Value.
+	 */
+	public set pathname(pathname: string) {
+		const url = new URL(this.#url.href);
+		url.pathname = pathname;
+		this.#browserFrame
+			.goto(url.href)
+			.catch((error) => this.#browserFrame.page.console.error(error));
+	}
+
+	/**
+	 * Returns port.
+	 *
+	 * @returns Port.
+	 */
+	public get port(): string {
+		return this.#url.port;
+	}
+
+	/**
+	 * Sets port.
+	 *
+	 * @param port Value.
+	 */
+	public set port(port: string) {
+		const url = new URL(this.#url.href);
+		url.port = port;
+		this.#browserFrame
+			.goto(url.href)
+			.catch((error) => this.#browserFrame.page.console.error(error));
+	}
+
+	/**
+	 * Returns protocol.
+	 *
+	 * @returns Protocol.
+	 */
+	public get protocol(): string {
+		return this.#url.protocol;
+	}
+
+	/**
+	 * Sets protocol.
+	 *
+	 * @param protocol Value.
+	 */
+	public set protocol(protocol: string) {
+		const url = new URL(this.#url.href);
+		url.protocol = protocol;
+		this.#browserFrame
+			.goto(url.href)
+			.catch((error) => this.#browserFrame.page.console.error(error));
+	}
+
+	/**
+	 * Returns search.
+	 *
+	 * @returns Search.
+	 */
+	public get search(): string {
+		return this.#url.search;
+	}
+
+	/**
+	 * Sets search.
+	 *
+	 * @param search Value.
+	 */
+	public set search(search: string) {
+		const url = new URL(this.#url.href);
+		url.search = search;
+		this.#browserFrame
+			.goto(url.href)
+			.catch((error) => this.#browserFrame.page.console.error(error));
+	}
+
+	/**
+	 * Replaces the current resource with the one at the provided URL. The difference from the assign() method is that after using replace() the current page will not be saved in session History, meaning the user won't be able to use the back button to navigate to it.
+	 *
+	 * @param url URL.
+	 */
+	public replace(url: string): void {
+		this.href = url;
+	}
+
+	/**
+	 * Loads the resource at the URL provided in parameter.
+	 *
+	 * @param url URL.
+	 */
+	public assign(url: string): void {
+		this.href = url;
+	}
+
+	/**
+	 * Reloads the resource from the current URL.
+	 */
+	public reload(): void {
+		this.#browserFrame
+			.goto(this.href)
+			.catch((error) => this.#browserFrame.page.console.error(error));
+	}
+
+	/**
+	 * Replaces the current URL state with the provided one without navigating to the new URL.
+	 *
+	 * @param browserFrame Browser frame that must match the current one as validation.
+	 * @param url URL.
+	 */
+	public [PropertySymbol.setURL](browserFrame: IBrowserFrame, url: string): void {
+		if (this.#browserFrame !== browserFrame) {
+			throw new Error('Failed to set URL. Browser frame mismatch.');
+		}
+
+		this.#url.href = url;
+	}
+
+	/**
+	 * Returns the URL as a string.
+	 *
+	 * @returns URL as a string.
+	 */
+	public toString(): string {
+		return this.#url.toString();
+	}
+}

package/src/named-node-map/NamedNodeMap.ts

@@ -225,6 +225,7 @@ export default class NamedNodeMap implements INamedNodeMap {
 	 */
 	protected [PropertySymbol.isValidPropertyName](name: string): boolean {
 		return (
+			!!name &&
 			!this.constructor.prototype.hasOwnProperty(name) &&
 			(isNaN(Number(name)) || name.includes('.'))
 		);