fix(core): render empty string for null __html (#360)

Author: JoviDeCroock

.changeset/yellow-ghosts-bow.md

@@ -0,0 +1,5 @@
+---
+'preact-render-to-string': patch
+---
+
+Correctly render `null` as an `__html` value as an empty string

src/index.js

@@ -55,7 +55,7 @@ export function renderToString(vnode, context, _rendererState) {
 	parent[CHILDREN] = [vnode];
 
 	try {
-		return _renderToString(
+		const rendered = _renderToString(
 			vnode,
 			context || EMPTY_OBJ,
 			false,
@@ -64,6 +64,11 @@ export function renderToString(vnode, context, _rendererState) {
 			false,
 			_rendererState
 		);
+
+		if (Array.isArray(rendered)) {
+			return rendered.join('');
+		}
+		return rendered;
 	} catch (e) {
 		if (e.then) {
 			throw new Error('Use "renderToStringAsync" for suspenseful rendering.');
@@ -682,7 +687,6 @@ function _renderToString(
 
 	if (Array.isArray(html)) return [startTag, ...html, endTag];
 	else if (typeof html !== 'string') return [startTag, html, endTag];
-
 	return startTag + html + endTag;
 }
 

test/render.test.jsx

@@ -387,6 +387,16 @@ describe('render', () => {
 			);
 		});
 
+		it('should accept nullish __html', () => {
+			const Test = (props) => (
+				<Fragment>
+					<div>hi</div>
+					<div dangerouslySetInnerHTML={{ __html: null }} />
+				</Fragment>
+			);
+			expect(render(<Test />)).to.equal('<div>hi</div><div></div>');
+		});
+
 		it('should apply defaultProps', () => {
 			const Test = (props) => <div {...props} />;
 			Test.defaultProps = {