Fix prop-types on annotated components (fixes #729)

Author: yannickcr

lib/rules/prop-types.js

@@ -118,7 +118,12 @@ module.exports = {
     function isAnnotatedFunctionPropsDeclaration(node) {
       if (node && node.params && node.params.length) {
         var tokens = context.getFirstTokens(node.params[0], 2);
-        var isAnnotated = node.params[0].typeAnnotation;
+        var isAnnotated = (
+          node.params[0].typeAnnotation &&
+          node.params[0].typeAnnotation.typeAnnotation &&
+          node.params[0].typeAnnotation.typeAnnotation.id &&
+          node.params[0].typeAnnotation.typeAnnotation.id.name === 'Props'
+        );
         var isDestructuredProps = node.params[0].type === 'ObjectPattern';
         var isProps = tokens[0].value === 'props' || (tokens[1] && tokens[1].value === 'props');
         if (isAnnotated && (isDestructuredProps || isProps)) {

tests/lib/rules/prop-types.js

@@ -1280,6 +1280,20 @@ ruleTester.run('prop-types', rule, {
         '}'
       ].join('\n'),
       parser: 'babel-eslint'
+    }, {
+      code: [
+        'type Target = { target: EventTarget }',
+        'class MyComponent extends Component {',
+        '  static propTypes = {',
+        '    children: PropTypes.any,',
+        '  }',
+        '  handler({ target }: Target) {}',
+        '  render() {',
+        '    return <div>{this.props.children}</div>;',
+        '  }',
+        '}'
+      ].join('\n'),
+      parser: 'babel-eslint'
     }
   ],