fix vnode type coercion

preactjs · Oct 12, 2023 · dfb6c32 · dfb6c32
1 parent 4b1a7e9
commit dfb6c32
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 8 deletions.
diff --git a/src/index.d.ts b/src/index.d.ts
@@ -195,9 +195,8 @@ export function createElement(
 		| null,
 	...children: ComponentChildren[]
 ): VNode<
-	| (JSXInternal.DOMAttributes<HTMLInputElement> &
-			ClassAttributes<HTMLInputElement>)
-	| null
+	| JSXInternal.DOMAttributes<HTMLInputElement> &
+			ClassAttributes<HTMLInputElement>
 >;
 export function createElement<
 	P extends JSXInternal.HTMLAttributes<T>,
@@ -206,15 +205,15 @@ export function createElement<
 	type: keyof JSXInternal.IntrinsicElements,
 	props: (ClassAttributes<T> & P) | null,
 	...children: ComponentChildren[]
-): VNode<(ClassAttributes<T> & P) | null>;
+): VNode<ClassAttributes<T> & P>;
 export function createElement<
 	P extends JSXInternal.SVGAttributes<T>,
 	T extends HTMLElement
 >(
 	type: keyof JSXInternal.IntrinsicElements,
 	props: (ClassAttributes<T> & P) | null,
 	...children: ComponentChildren[]
-): VNode<(ClassAttributes<T> & P) | null>;
+): VNode<ClassAttributes<T> & P>;
 export function createElement<T extends HTMLElement>(
 	type: string,
 	props:

diff --git a/test/ts/VNode-test.tsx b/test/ts/VNode-test.tsx
@@ -39,9 +39,15 @@ describe('VNode TS types', () => {
 		expect(actual).to.include.all.keys('type', 'props', 'key');
 	});
 
-	it('has a nodeName of string when html element', () => {
-		const div = <div>Hi!</div>;
-		expect(div.type).to.equal('div');
+	it('is returned by h', () => {
+		const actual = <div className="wow" />;
+		expect(actual).to.include.all.keys('type', 'props', 'key');
+	});
+
+	it('createElement conforms to the VNode type', () => {
+		const arr: VNode[] = [];
+		arr.push(createElement('div', null));
+		expect(true).to.be.true;
 	});
 
 	it('has a nodeName equal to the construction function when SFC', () => {