Merge pull request #675 from rofrischmann/feature/allow-empty-style-prop-in-felaComponent

feat(FelaComponent): Allow style prop with empty value

Author: robinweser

packages/fela-bindings/src/FelaComponentFactory.js

@@ -34,12 +34,6 @@ export default function FelaComponentFactory(
       'The `render` prop in FelaComponent is deprecated. It will be removed in react-fela@11.0.0.\nPlease always use `children` instead. See http://fela.js.org/docs/api/bindings/fela-component'
     )
 
-    if (!style) {
-      throw new Error(
-        'A valid `style` prop must be passed to FelaComponent in order to render.\nSee http://fela.js.org/docs/api/bindings/fela-component'
-      )
-    }
-
     const renderFn = renderer =>
       createElement(FelaTheme, undefined, theme => {
         // TODO: could optimise perf by not calling combineRules if not neccessary

packages/fela-integration/src/jest-react-fela_react-fela/__tests__/FelaComponent-test.js

@@ -219,4 +219,16 @@ describe('Using the FelaComponent component', () => {
       )
     ).toMatchSnapshot()
   })
+
+  it('should render without style', () => {
+    expect(
+      createSnapshot(
+        <FelaComponent>
+          {({ className }) => (
+            <div className={className}>I am an unstyled div</div>
+          )}
+        </FelaComponent>
+      )
+    ).toMatchSnapshot()
+  })
 })

packages/fela-integration/src/jest-react-fela_react-fela/__tests__/__snapshots__/FelaComponent-test.js.snap

@@ -169,3 +169,10 @@ exports[`Using the FelaComponent component should render children using the corr
 <h1 className=a b>Hello World</h1>;
 "
 `;
+
+exports[`Using the FelaComponent component should render without style 1`] = `
+"
+
+<div className>I am an unstyled div</div>;
+"
+`;