Fix: Fixes a crash when using a defaultProp that is not listed in props (#437)

Author: ankri

loaders/utils/__tests__/__snapshots__/getProps.spec.js.snap

@@ -24,6 +24,22 @@ Object {
 }
 `;
 
+exports[`should not crash when using doctrine to parse a default prop that isn't in the props list 1`] = `
+Object {
+  "description": "The only true button.
+",
+  "doclets": Object {},
+  "methods": Array [],
+  "props": Object {
+    "crash": Object {
+      "description": "",
+      "tags": Object {},
+    },
+  },
+  "tags": Object {},
+}
+`;
+
 exports[`should remove non-public methods 1`] = `
 Object {
   "doclets": Object {},

loaders/utils/__tests__/getProps.spec.js

@@ -107,3 +107,17 @@ alert('Hello world');
 
 	expect(result).toMatchSnapshot();
 });
+
+it('should not crash when using doctrine to parse a default prop that isn\'t in the props list', () => {
+	const result = getProps({
+		description: 'The only true button.',
+		methods: [],
+		props: {
+			crash: {
+				description: undefined,
+			},
+		},
+	});
+
+	expect(result).toMatchSnapshot();
+});

loaders/utils/getProps.js

@@ -68,7 +68,8 @@ module.exports = function getProps(doc) {
 		Object.keys(doc.props).forEach(propName => {
 			const prop = doc.props[propName];
 			const doclets = getDocletsObject(prop.description);
-			const documentation = doctrine.parse(prop.description);
+			// when a prop is listed in defaultProps but not in props the prop.description is undefined
+			const documentation = doctrine.parse(prop.description || '');
 
 			// documentation.description is the description without tags
 			doc.props[propName].description = documentation.description;