Merge pull request #127 from omniscientjs/issue-126

Expose `defaultProps` on component creator
omniscientjs · Nov 12, 2015 · 4195b11 · 4195b11
2 parents 41260c0 + 67fbd3f
commit 4195b11
Show file tree

Hide file tree

Showing 2 changed files with 19 additions and 3 deletions.
diff --git a/component.js b/component.js
@@ -304,9 +304,7 @@ function factory (initialOptions) {
         create = assign(create, methodStatics);
       }
 
-      create.type = Component;
-
-      return create;
+      return assign(create, Component, { type: Component });
     }
   }
 

diff --git a/tests/component-test.js b/tests/component-test.js
@@ -643,6 +643,24 @@ describe('component', function () {
 
       Creator.type.should.equal(Type);
     });
+
+    it('will pass default props', function () {
+      var expectedPropValue = 'default-prop-value';
+
+      var lifecycleMethods = {
+        getDefaultProps: function () {
+          return { direction: expectedPropValue };
+        }
+      };
+      var Component = component(lifecycleMethods, function (props) {
+        should.equal(props.direction, expectedPropValue);
+        this.props.direction.should.equal(expectedPropValue);
+        return DOM.div();
+      });
+
+      render(Component());
+      render(React.createElement(Component)); // jsx
+    });
   });
 
   describe('should not re-render', function () {