fix(view): correctly proxy react specific getter only static props

Author: solkimicreb

.npmrc

@@ -24,4 +24,15 @@ describe('static props', () => {
     expect(ViewFuncComp.propTypes).toBe(FuncComp.propTypes)
     expect(ViewFuncComp.defaultProps).toBe(FuncComp.defaultProps)
   })
+
+  test('view() should proxy react specific static getters', () => {
+    class Comp extends Component {
+      static get defaultProps () {
+        return { key: 'value' }
+      }
+    }
+
+    const ViewComp = view(Comp)
+    expect(ViewComp.defaultProps).toEqual(Comp.defaultProps)
+  })
 })

__tests__/staticProps.test.js

@@ -91,15 +91,15 @@
     "moment": "2.20.1",
     "pre-push": "0.1.1",
     "prettier": "^1.6.1",
-    "react": "^16.3.0-alpha.1",
-    "react-dom": "^16.3.0-alpha.1",
-    "react-test-renderer": "^16.3.0-alpha.1",
+    "react": "^16.3.0",
+    "react-dom": "^16.3.0",
+    "react-test-renderer": "^16.3.0",
     "rollup": "^0.56.0",
     "rollup-plugin-auto-external": "^1.0.0",
     "rollup-plugin-babel": "^3.0.2",
     "rollup-plugin-node-resolve": "^3.0.0",
     "sinon": "^4.3.0",
-    "standard": "10.0.3"
+    "standard": "^10.0.3"
   },
   "engines": {
     "node": ">=6.0.0"

package.json

@@ -11,13 +11,7 @@ export default function view (Comp, { devtool: rawDevtool } = {}) {
 
   // return a HOC which overwrites render, shouldComponentUpdate and componentWillUnmount
   // it decides when to run the new reactive methods and when to proxy to the original methods
-  return class ReactiveHOC extends BaseComp {
-    static displayName = Comp.displayName || Comp.name;
-    static contextTypes = Comp.contextTypes;
-    static childContextTypes = Comp.childContextTypes;
-    static propTypes = Comp.propTypes;
-    static defaultProps = Comp.defaultProps;
-
+  class ReactiveHOC extends BaseComp {
     constructor (props, context) {
       super(props, context)
 
@@ -81,4 +75,16 @@ export default function view (Comp, { devtool: rawDevtool } = {}) {
       unobserve(this.render)
     }
   }
+
+  ReactiveHOC.displayName = Comp.displayName || Comp.name;
+  // these are inherited by class components,
+  // but have to be copied for function components
+  if (isStatelessComp) {
+    ReactiveHOC.contextTypes = Comp.contextTypes;
+    ReactiveHOC.childContextTypes = Comp.childContextTypes;
+    ReactiveHOC.propTypes = Comp.propTypes;
+    ReactiveHOC.defaultProps = Comp.defaultProps;
+  }
+
+  return ReactiveHOC
 }