bug fix: function expression as computed property

eslint · Oct 10, 2017 · 7365f7d · 7365f7d
1 parent 3bb0529
commit 7365f7d
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 1 deletion.
diff --git a/lib/rules/require-jsdoc.js b/lib/rules/require-jsdoc.js
@@ -85,7 +85,7 @@ module.exports = {
             FunctionExpression(node) {
                 if (
                     (options.MethodDefinition && node.parent.type === "MethodDefinition") ||
-                    (options.FunctionExpression && (node.parent.type === "VariableDeclarator" || node.parent.type === "Property"))
+                    (options.FunctionExpression && (node.parent.type === "VariableDeclarator" || (node.parent.type === "Property" && node === node.parent.value)))
                 ) {
                     checkJsDoc(node);
                 }

diff --git a/tests/lib/rules/require-jsdoc.js b/tests/lib/rules/require-jsdoc.js
@@ -216,6 +216,15 @@ ruleTester.run("require-jsdoc", rule, {
                     FunctionExpression: true
                 }
             }]
+        },
+        {
+            code: " var foo = { [function() {}]: 1 };",
+            options: [{
+                require: {
+                    FunctionExpression: true
+                }
+            }],
+            parserOptions: { ecmaVersion: 6 }
         }
     ],