fix: [#1176] Use localName correctly in Document.createElement() and Document.createElementNS()

Author: capricorn86

packages/happy-dom/src/PropertySymbol.ts

@@ -149,3 +149,5 @@ export const content = Symbol('content');
 export const mode = Symbol('mode');
 export const host = Symbol('host');
 export const setURL = Symbol('setURL');
+export const localName = Symbol('localName');
+export const registedClass = Symbol('registedClass');

packages/happy-dom/src/config/ElementTag.ts

@@ -0,0 +1,119 @@
+export default <{ [key: string]: string }>{
+	a: 'HTMLAnchorElement',
+	abbr: 'HTMLElement',
+	address: 'HTMLElement',
+	area: 'HTMLElement',
+	article: 'HTMLElement',
+	aside: 'HTMLElement',
+	audio: 'HTMLAudioElement',
+	b: 'HTMLElement',
+	base: 'HTMLBaseElement',
+	bdi: 'HTMLElement',
+	bdo: 'HTMLElement',
+	blockquaote: 'HTMLElement',
+	body: 'HTMLElement',
+	template: 'HTMLTemplateElement',
+	form: 'HTMLFormElement',
+	input: 'HTMLInputElement',
+	textarea: 'HTMLTextAreaElement',
+	script: 'HTMLScriptElement',
+	img: 'HTMLImageElement',
+	link: 'HTMLLinkElement',
+	style: 'HTMLStyleElement',
+	label: 'HTMLLabelElement',
+	slot: 'HTMLSlotElement',
+	meta: 'HTMLMetaElement',
+	blockquote: 'HTMLElement',
+	br: 'HTMLElement',
+	button: 'HTMLButtonElement',
+	canvas: 'HTMLElement',
+	caption: 'HTMLElement',
+	cite: 'HTMLElement',
+	code: 'HTMLElement',
+	col: 'HTMLElement',
+	colgroup: 'HTMLElement',
+	data: 'HTMLElement',
+	datalist: 'HTMLElement',
+	dd: 'HTMLElement',
+	del: 'HTMLElement',
+	details: 'HTMLElement',
+	dfn: 'HTMLElement',
+	dialog: 'HTMLDialogElement',
+	div: 'HTMLElement',
+	dl: 'HTMLElement',
+	dt: 'HTMLElement',
+	em: 'HTMLElement',
+	embed: 'HTMLElement',
+	fieldset: 'HTMLElement',
+	figcaption: 'HTMLElement',
+	figure: 'HTMLElement',
+	footer: 'HTMLElement',
+	h1: 'HTMLElement',
+	h2: 'HTMLElement',
+	h3: 'HTMLElement',
+	h4: 'HTMLElement',
+	h5: 'HTMLElement',
+	h6: 'HTMLElement',
+	head: 'HTMLElement',
+	header: 'HTMLElement',
+	hgroup: 'HTMLElement',
+	hr: 'HTMLElement',
+	html: 'HTMLElement',
+	i: 'HTMLElement',
+	iframe: 'HTMLIFrameElement',
+	ins: 'HTMLElement',
+	kbd: 'HTMLElement',
+	legend: 'HTMLElement',
+	li: 'HTMLElement',
+	main: 'HTMLElement',
+	map: 'HTMLElement',
+	mark: 'HTMLElement',
+	math: 'HTMLElement',
+	menu: 'HTMLElement',
+	menuitem: 'HTMLElement',
+	meter: 'HTMLElement',
+	nav: 'HTMLElement',
+	noscript: 'HTMLElement',
+	object: 'HTMLElement',
+	ol: 'HTMLElement',
+	optgroup: 'HTMLOptGroupElement',
+	option: 'HTMLOptionElement',
+	output: 'HTMLElement',
+	p: 'HTMLElement',
+	param: 'HTMLElement',
+	picture: 'HTMLElement',
+	pre: 'HTMLElement',
+	progress: 'HTMLElement',
+	q: 'HTMLElement',
+	rb: 'HTMLElement',
+	rp: 'HTMLElement',
+	rt: 'HTMLElement',
+	rtc: 'HTMLElement',
+	ruby: 'HTMLElement',
+	s: 'HTMLElement',
+	samp: 'HTMLElement',
+	section: 'HTMLElement',
+	select: 'HTMLSelectElement',
+	small: 'HTMLElement',
+	source: 'HTMLElement',
+	span: 'HTMLElement',
+	strong: 'HTMLElement',
+	sub: 'HTMLElement',
+	summary: 'HTMLElement',
+	sup: 'HTMLElement',
+	table: 'HTMLElement',
+	tbody: 'HTMLElement',
+	td: 'HTMLElement',
+	tfoot: 'HTMLElement',
+	th: 'HTMLElement',
+	thead: 'HTMLElement',
+	time: 'HTMLElement',
+	title: 'HTMLElement',
+	tr: 'HTMLElement',
+	track: 'HTMLElement',
+	u: 'HTMLElement',
+	ul: 'HTMLElement',
+	var: 'HTMLElement',
+	video: 'HTMLVideoElement',
+	wbr: 'HTMLElement'
+};

packages/happy-dom/src/config/HTMLElementLocalNameToClass.ts

@@ -1,5 +1,6 @@
 export default {
 	html: 'http://www.w3.org/1999/xhtml',
 	svg: 'http://www.w3.org/2000/svg',
-	mathML: 'http://www.w3.org/1998/Math/MathML'
+	mathML: 'http://www.w3.org/1998/Math/MathML',
+	xmlns: 'http://www.w3.org/2000/xmlns/'
 };

packages/happy-dom/src/config/PlainTextElements.ts packages/happy-dom/src/config/HTMLElementPlainText.ts

@@ -10,6 +10,7 @@ export default class CustomElementRegistry {
 	public [PropertySymbol.registry]: {
 		[k: string]: { elementClass: typeof HTMLElement; extends: string };
 	} = {};
+	public [PropertySymbol.registedClass]: Map<typeof HTMLElement, string> = new Map();
 	public [PropertySymbol.callbacks]: { [k: string]: (() => void)[] } = {};
 
 	/**
@@ -27,16 +28,27 @@ export default class CustomElementRegistry {
 	): void {
 		if (!this.#isValidCustomElementName(name)) {
 			throw new DOMException(
-				"Failed to execute 'define' on 'CustomElementRegistry': \"" +
-					name +
-					'" is not a valid custom element name.'
+				`Failed to execute 'define' on 'CustomElementRegistry': "${name}" is not a valid custom element name`
+			);
+		}
+
+		if (this[PropertySymbol.registry][name]) {
+			throw new DOMException(
+				`Failed to execute 'define' on 'CustomElementRegistry': the name "${name}" has already been used with this registry`
+			);
+		}
+
+		if (this[PropertySymbol.registedClass].has(elementClass)) {
+			throw new DOMException(
+				"Failed to execute 'define' on 'CustomElementRegistry': this constructor has already been used with this registry"
 			);
 		}
 
 		this[PropertySymbol.registry][name] = {
 			elementClass,
 			extends: options && options.extends ? options.extends.toLowerCase() : null
 		};
+		this[PropertySymbol.registedClass].set(elementClass, name);
 
 		// ObservedAttributes should only be called once by CustomElementRegistry (see #117)
 		if (elementClass.prototype.attributeChangedCallback) {
@@ -98,13 +110,7 @@ export default class CustomElementRegistry {
 	 * @returns Found tag name or `null`.
 	 */
 	public getName(elementClass: typeof HTMLElement): string | null {
-		// For loops are faster than find()
-		for (const name of Object.keys(this[PropertySymbol.registry])) {
-			if (this[PropertySymbol.registry][name].elementClass === elementClass) {
-				return name;
-			}
-		}
-		return null;
+		return this[PropertySymbol.registedClass].get(elementClass) || null;
 	}
 
 	/**