fix(#255): handle falsy children values (#256)

src/Helmet.js

@@ -198,6 +198,10 @@ const Helmet = (Component) => class HelmetWrapper extends React.Component {
         let arrayTypeChildren = {};
 
         React.Children.forEach(children, (child) => {
+            if (!child || !child.props) {
+                return;
+            }
+
             const {children: nestedChildren, ...childProps} = child.props;
             const newChildProps = convertReactPropstoHtmlAttributes(childProps);
 

test/HelmetDeclarativeTest.js

@@ -3173,6 +3173,24 @@ describe("Helmet - Declarative API", () => {
             expect(renderInvalid).to.throw(Error, "Helmet expects a string as a child of <script>. Did you forget to wrap your children in braces? ( <script>{``}</script> ) Refer to our API for more information.");
         });
 
+        it("handles undefined children", (done) => {
+            const charSet = undefined;
+
+            ReactDOM.render(
+                <Helmet>
+                    {charSet && <meta charSet={charSet} />}
+                    <title>Test Title</title>
+                </Helmet>,
+                container
+            );
+
+            requestIdleCallback(() => {
+                expect(document.title).to.equal("Test Title");
+
+                done();
+            });
+        });
+
         it("recognizes valid tags regardless of attribute ordering", (done) => {
             ReactDOM.render(
                 <Helmet>