Support flow type spread (#241)

* Support Flow Type Spread Props

* Addressing feedback

* Properly spreading in locally resolved types

* Clean up comment

* Pass Flow

* Sort imports

* Update tests

Author: elicwhite

src/Documentation.js

@@ -51,7 +51,9 @@ class Documentation {
     if (this._props.size > 0) {
       obj.props = {};
       for (var [name, descriptor] of this._props) {
-        obj.props[name] = descriptor;
+        if (Object.keys(descriptor).length > 0) {
+          obj.props[name] = descriptor;
+        }
       }
     }
 

src/__tests__/__snapshots__/main-test.js.snap

@@ -263,10 +263,20 @@ Object {
 
 exports[`main fixtures processes component "component_11.js" without errors 1`] = `
 Object {
+  "composes": Array [
+    "OtherProps",
+  ],
   "description": "",
   "displayName": "MyComponent",
   "methods": Array [],
   "props": Object {
+    "fooProp": Object {
+      "description": "fooProp is spread in from a locally resolved type",
+      "flowType": Object {
+        "name": "string",
+      },
+      "required": false,
+    },
     "prop1": Object {
       "description": "The first prop",
       "flowType": Object {

src/__tests__/fixtures/component_11.js

@@ -14,7 +14,26 @@
 
 import React from 'react';
 
+import { OtherComponentProps as OtherProps } from 'NonExistentFile';
+
+type OtherLocalProps = {|
+   /**
+   * fooProp is spread in from a locally resolved type
+   */
+  fooProp?: string,
+|}
+
 type Props = {|
+  /**
+   * Spread props defined locally
+   */
+  ...OtherLocalProps,
+
+  /**
+   * Spread props from another file
+   */
+  ...OtherProps,
+
   /**
    * The first prop
    */

src/handlers/__tests__/flowTypeDocBlockHandler-test.js

@@ -56,14 +56,17 @@ describe('flowTypeHandler', () => {
         foo: {
           flowType: {},
           required: true,
+          description: '',
         },
         bar: {
           flowType: {},
           required: true,
+          description: '',
         },
         hal: {
           flowType: {},
           required: true,
+          description: '',
         },
         });
     });
@@ -83,10 +86,12 @@ describe('flowTypeHandler', () => {
         foo: {
           flowType: {},
           required: true,
+          description: '',
         },
         bar: {
           flowType: {},
           required: false,
+          description: '',
         },
       });
     });
@@ -106,10 +111,12 @@ describe('flowTypeHandler', () => {
         foo: {
           flowType: {},
           required: true,
+          description: '',
         },
         bar: {
           flowType: {},
           required: true,
+          description: '',
         },
       });
     });
@@ -128,6 +135,7 @@ describe('flowTypeHandler', () => {
         foo: {
           flowType: {},
           required: true,
+          description: '',
         },
       });
     });
@@ -195,6 +203,7 @@ describe('flowTypeHandler', () => {
       foo: {
         flowType: {},
         required: true,
+        description: '',
       },
     });
   });