Fix component detection ignore async functions

Author: yannickcr

lib/util/Components.js

@@ -496,6 +496,10 @@ function componentRule(rule, context) {
     },
 
     FunctionExpression: function(node) {
+      if (node.async) {
+        components.add(node, 0);
+        return;
+      }
       var component = utils.getParentComponent();
       if (
         !component ||
@@ -509,6 +513,10 @@ function componentRule(rule, context) {
     },
 
     FunctionDeclaration: function(node) {
+      if (node.async) {
+        components.add(node, 0);
+        return;
+      }
       node = utils.getParentComponent();
       if (!node) {
         return;
@@ -517,6 +525,10 @@ function componentRule(rule, context) {
     },
 
     ArrowFunctionExpression: function(node) {
+      if (node.async) {
+        components.add(node, 0);
+        return;
+      }
       var component = utils.getParentComponent();
       if (
         !component ||

tests/lib/rules/prop-types.js

@@ -1361,6 +1361,30 @@ ruleTester.run('prop-types', rule, {
       ].join('\n'),
       options: [{skipUndeclared: false}],
       parserOptions: parserOptions
+    }, {
+      // Async functions can't be components.
+      code: [
+        'var Hello = async function(props) {',
+        '  return <div>Hello {props.name}</div>;',
+        '}'
+      ].join('\n'),
+      parser: 'babel-eslint'
+    }, {
+      // Async functions can't be components.
+      code: [
+        'async function Hello(props) {',
+        '  return <div>Hello {props.name}</div>;',
+        '}'
+      ].join('\n'),
+      parser: 'babel-eslint'
+    }, {
+      // Async functions can't be components.
+      code: [
+        'var Hello = async (props) => {',
+        '  return <div>Hello {props.name}</div>;',
+        '}'
+      ].join('\n'),
+      parser: 'babel-eslint'
     }
   ],