prevent to self-close if dangerouslySetInnerHTML set (#174)

Author: TOGASHI Tomoki

src/index.js

@@ -351,13 +351,13 @@ function renderToString(vnode, context, opts, inner, isSvgMode, selectValue) {
 		}
 	}
 
-	if (pieces.length) {
+	if (pieces.length || html) {
 		s += pieces.join('');
 	} else if (opts && opts.xml) {
 		return s.substring(0, s.length - 1) + ' />';
 	}
 
-	if (isVoid && !children) {
+	if (isVoid && !children && !html) {
 		s = s.replace(/>$/, ' />');
 	} else {
 		if (pretty && ~s.indexOf('\n')) s += '\n';

test/render.js

@@ -228,6 +228,15 @@ describe('render', () => {
 			expect(rendered).to.equal(expected);
 		});
 
+		it('should not self-close void elements if it has dangerouslySetInnerHTML prop', () => {
+			let rendered = render(
+					<link dangerouslySetInnerHTML={{ __html: '<foo>' }} />
+				),
+				expected = `<link><foo></link>`;
+
+			expect(rendered).to.equal(expected);
+		});
+
 		it('should serialize object styles', () => {
 			let rendered = render(<div style={{ color: 'red', border: 'none' }} />),
 				expected = `<div style="color: red; border: none;"></div>`;
@@ -814,6 +823,15 @@ describe('render', () => {
 			expect(renderXml(<a>b</a>)).to.equal(`<a>b</a>`);
 		});
 
+		it('should not self-close if it has dangerouslySetInnerHTML prop', () => {
+			expect(
+				renderXml(<a dangerouslySetInnerHTML={{ __html: 'b' }} />)
+			).to.equal(`<a>b</a>`);
+			expect(
+				renderXml(<a dangerouslySetInnerHTML={{ __html: '<b />' }} />)
+			).to.equal(`<a><b /></a>`);
+		});
+
 		it('should render boolean attributes with named values', () => {
 			expect(renderXml(<div foo bar />)).to.equal(
 				`<div foo="foo" bar="bar" />`