chore: [#1678] Fixes problem with query selector

capricorn86 · Jan 8, 2025 · 3f545e8 · 3f545e8
1 parent 55110a1
commit 3f545e8
Show file tree

Hide file tree

Showing 3 changed files with 19 additions and 3 deletions.
diff --git a/packages/happy-dom/src/query-selector/SelectorItem.ts b/packages/happy-dom/src/query-selector/SelectorItem.ts
@@ -7,6 +7,8 @@ import ISelectorAttribute from './ISelectorAttribute.js';
 import ISelectorMatch from './ISelectorMatch.js';
 import ISelectorPseudo from './ISelectorPseudo.js';
 
+const SPACE_REGEXP = /\s+/;
+
 /**
  * Selector item.
  */
@@ -417,7 +419,7 @@ export default class SelectorItem {
 			return null;
 		}
 
-		const classList = element.className.split(' ');
+		const classList = element.className.split(SPACE_REGEXP);
 		let priorityWeight = 0;
 
 		for (const className of this.classNames) {

diff --git a/packages/happy-dom/test/html-parser/HTMLParser.test.ts b/packages/happy-dom/test/html-parser/HTMLParser.test.ts
@@ -2137,6 +2137,10 @@ describe('HTMLParser', () => {
                  visually-hidden" type="button">{{message_gui_replies}}</button>
             </div>`
 			);
+
+			const element = result.querySelector('div > .comment_collapse');
+
+			expect(element).toBe(result.children[0].children[1]);
 		});
 	});
 });
diff --git a/packages/happy-dom/test/query-selector/QuerySelector.test.ts b/packages/happy-dom/test/query-selector/QuerySelector.test.ts
@@ -18,7 +18,7 @@ describe('QuerySelector', () => {
 		document = window.document;
 	});
 
-	describe('querySelectorAll', () => {
+	describe('querySelectorAll()', () => {
 		it('Throws an error for invalid selectors.', () => {
 			const container = document.createElement('div');
 			expect(() => container.querySelectorAll(<string>(<unknown>12))).toThrow(
@@ -1214,7 +1214,7 @@ describe('QuerySelector', () => {
 		});
 	});
 
-	describe('querySelector', () => {
+	describe('querySelector()', () => {
 		it('Throws an error for invalid selectors.', () => {
 			const container = document.createElement('div');
 			expect(() => container.querySelector(<string>(<unknown>12))).toThrow(
@@ -1628,6 +1628,16 @@ describe('QuerySelector', () => {
 			expect(document.querySelector(':focus')).toBe(div);
 			expect(document.querySelector(':focus-visible')).toBe(div);
 		});
+
+		it('Handles class names with line breaks', () => {
+			const div = document.createElement('div');
+			div.innerHTML = `
+                <div class="class1
+                class2"></div>
+            `;
+
+			expect(div.querySelector('.class1.class2')).toBe(div.children[0]);
+		});
 	});
 
 	describe('matches()', () => {