Add similar supprt for no-used-prop-types

Author: jseminck

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

@@ -724,6 +724,50 @@ module.exports = {
       });
     }
 
+    /**
+     * Marks all props found inside IntersectionTypeAnnotation as declared.
+     * Since InterSectionTypeAnnotations can be nested, this handles recursively. 
+     * 
+     * Modifies the declaredPropTypes object
+     * @param {ASTNode} propTypes 
+     * @param {Object} declaredPropTypes 
+     * @returns {Boolean} True if propTypes should be ignored (e.g. when a type can't be resolved, when it is imported)
+     */
+    function declarePropTypesForIntersectionTypeAnnotation(propTypes, declaredPropTypes) {
+      let ignorePropsValidation = false;
+
+      propTypes.types.forEach(annotation => {
+        const typeNode = typeScope(annotation.id.name);
+
+        if (!typeNode) {
+          ignorePropsValidation = true;
+          return;
+        }
+
+        if (typeNode.type === 'IntersectionTypeAnnotation') {
+          ignorePropsValidation = declarePropTypesForIntersectionTypeAnnotation(typeNode, declaredPropTypes);
+        } else {
+          iterateProperties(typeNode.properties, (key, value) => {
+            if (!value) {
+              ignorePropsValidation = true;
+              return;
+            }
+
+            let types = buildTypeAnnotationDeclarationTypes(value, key);
+            if (types === true) {
+              types = {};
+            }
+            types.fullName = key;
+            types.name = key;
+            types.node = value;
+            declaredPropTypes.push(types);
+          });
+        }
+      });
+
+      return ignorePropsValidation;
+    }
+
     /**
      * Mark a prop type as declared
      * @param {ASTNode} node The AST node being checked.
@@ -791,24 +835,7 @@ module.exports = {
           }
           break;
         case 'IntersectionTypeAnnotation':
-          propTypes.types.forEach(annotation => {
-            const propsType = typeScope(annotation.id.name);
-            iterateProperties(propsType.properties, (key, value) => {
-              if (!value) {
-                ignorePropsValidation = true;
-                return;
-              }
-
-              let types = buildTypeAnnotationDeclarationTypes(value, key);
-              if (types === true) {
-                types = {};
-              }
-              types.fullName = key;
-              types.name = key;
-              types.node = value;
-              declaredPropTypes.push(types);
-            });
-          });
+          ignorePropsValidation = declarePropTypesForIntersectionTypeAnnotation(propTypes, declaredPropTypes);
           break;
         case null:
           break;

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

@@ -839,6 +839,38 @@ ruleTester.run('no-unused-prop-types', rule, {
       }
       `,
       parser: 'babel-eslint'
+    }, {
+      code: `
+        type PropsA = { foo: string };
+        type PropsB = { bar: string };
+        type PropsC = { zap: string };
+        type Props = PropsA & PropsB;
+
+        class Bar extends React.Component {
+          props: Props & PropsC;
+          
+          render() {
+            return <div>{this.props.foo} - {this.props.bar} - {this.props.zap}</div>
+          }
+        }
+      `,
+      parser: 'babel-eslint'
+    }, {
+      code: `
+        import type { PropsA } from "./myPropsA";
+        type PropsB = { bar: string };
+        type PropsC = { zap: string };
+        type Props = PropsA & PropsB;
+
+        class Bar extends React.Component {
+          props: Props & PropsC;
+          
+          render() {
+            return <div>{this.props.foo} - {this.props.bar} - {this.props.zap}</div>
+          }
+        }
+      `,
+      parser: 'babel-eslint'
     }, {
       code: [
         'import type Props from "fake";',
@@ -2743,6 +2775,25 @@ ruleTester.run('no-unused-prop-types', rule, {
       errors: [
         {message: '\'b\' PropType is defined but prop is never used'}
       ]
+    }, {
+      code: `
+        type PropsA = { foo: string };
+        type PropsB = { bar: string };
+        type PropsC = { zap: string };
+        type Props = PropsA & PropsB;
+
+        class Bar extends React.Component {
+          props: Props & PropsC;
+          
+          render() {
+            return <div>{this.props.foo} - {this.props.bar}</div>
+          }
+        }
+      `,
+      parser: 'babel-eslint',
+      errors: [
+        {message: '\'zap\' PropType is defined but prop is never used'}
+      ]
     }, {
       code: [
         'class Hello extends React.Component {',