Merge pull request #2225 from dwelle/display_name_nested

[Fix] `display-name`: fix false negative around nested functions

Author: ljharb

lib/rules/display-name.js

@@ -160,21 +160,27 @@ module.exports = {
         if (ignoreTranspilerName || !hasTranspilerName(node)) {
           return;
         }
-        markDisplayNameAsDeclared(node);
+        if (components.get(node)) {
+          markDisplayNameAsDeclared(node);
+        }
       },
 
       FunctionDeclaration: function(node) {
         if (ignoreTranspilerName || !hasTranspilerName(node)) {
           return;
         }
-        markDisplayNameAsDeclared(node);
+        if (components.get(node)) {
+          markDisplayNameAsDeclared(node);
+        }
       },
 
       ArrowFunctionExpression: function(node) {
         if (ignoreTranspilerName || !hasTranspilerName(node)) {
           return;
         }
-        markDisplayNameAsDeclared(node);
+        if (components.get(node)) {
+          markDisplayNameAsDeclared(node);
+        }
       },
 
       MethodDefinition: function(node) {

tests/lib/rules/display-name.js

@@ -693,5 +693,65 @@ ruleTester.run('display-name', rule, {
     errors: [{
       message: 'Component definition is missing display name'
     }]
+  }, {
+    code: `
+      module.exports = function () {
+        function a () {}
+        const b = function b () {}
+        const c = function () {}
+        const d = () => {}
+        const obj = {
+          a: function a () {},
+          b: function b () {},
+          c () {},
+          d: () => {},
+        }
+        return React.createElement("div", {}, "text content");
+      }
+    `,
+    errors: [{
+      message: 'Component definition is missing display name'
+    }]
+  }, {
+    code: `
+      module.exports = () => {
+        function a () {}
+        const b = function b () {}
+        const c = function () {}
+        const d = () => {}
+        const obj = {
+          a: function a () {},
+          b: function b () {},
+          c () {},
+          d: () => {},
+        }
+
+        return React.createElement("div", {}, "text content");
+      }
+    `,
+    errors: [{
+      message: 'Component definition is missing display name'
+    }]
+  }, {
+    code: `
+      export default class extends React.Component {
+        render() {
+          function a () {}
+          const b = function b () {}
+          const c = function () {}
+          const d = () => {}
+          const obj = {
+            a: function a () {},
+            b: function b () {},
+            c () {},
+            d: () => {},
+          }
+          return <div>Hello {this.props.name}</div>;
+        }
+      }
+    `,
+    errors: [{
+      message: 'Component definition is missing display name'
+    }]
   }]
 });