Merge pull request #1563 from justinanastos/fix/prop-types-union-flowtype-1468

Fix crash when using Unions in flow propTypes

Author: ljharb

lib/rules/no-unused-prop-types.js

@@ -780,6 +780,10 @@ module.exports = {
           return declarePropTypesForObjectTypeAnnotation(annotation, declaredPropTypes);
         }
 
+        if (annotation.type === 'UnionTypeAnnotation') {
+          return true;
+        }
+
         const typeNode = typeScope(annotation.id.name);
 
         if (!typeNode) {

lib/rules/prop-types.js

@@ -726,6 +726,10 @@ module.exports = {
           return declarePropTypesForObjectTypeAnnotation(annotation, declaredPropTypes);
         }
 
+        if (annotation.type === 'UnionTypeAnnotation') {
+          return true;
+        }
+
         const typeNode = typeScope(annotation.id.name);
 
         if (!typeNode) {

tests/lib/rules/no-unused-prop-types.js

@@ -1097,6 +1097,28 @@ ruleTester.run('no-unused-prop-types', rule, {
         '}'
       ].join('\n'),
       parser: 'babel-eslint'
+    }, {
+      code: [
+        'type PropsUnionA = {',
+        '  a: string,',
+        '  b?: void,',
+        '};',
+        'type PropsUnionB = {',
+        '  a?: void,',
+        '  b: string,',
+        '};',
+        'type Props = {',
+        '  name: string,',
+        '} & (PropsUnionA | PropsUnionB);',
+        'class Hello extends React.Component {',
+        '  props: Props;',
+        '  render() {',
+        '    const {name} = this.props;',
+        '    return name;',
+        '  }',
+        '}'
+      ].join('\n'),
+      parser: 'babel-eslint'
     }, {
       code: [
         'Card.propTypes = {',

tests/lib/rules/prop-types.js

@@ -1129,6 +1129,28 @@ ruleTester.run('prop-types', rule, {
         '}'
       ].join('\n'),
       parser: 'babel-eslint'
+    }, {
+      code: [
+        'type PropsUnionA = {',
+        '  a: string,',
+        '  b?: void,',
+        '};',
+        'type PropsUnionB = {',
+        '  a?: void,',
+        '  b: string,',
+        '};',
+        'type Props = {',
+        '  name: string,',
+        '} & (PropsUnionA | PropsUnionB);',
+        'class Hello extends React.Component {',
+        '  props: Props;',
+        '  render() {',
+        '    const {name} = this.props;',
+        '    return name;',
+        '  }',
+        '}'
+      ].join('\n'),
+      parser: 'babel-eslint'
     }, {
       code: [
         'Card.propTypes = {',