Fix: Don't warn for member expression functions (fixes #2402)

eslint · May 8, 2015 · c11c408 · c11c408 · michaelficarra · May 8, 2015
1 parent 5beee9d
commit c11c408
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/lib/rules/no-wrap-func.js b/lib/rules/no-wrap-func.js
@@ -27,7 +27,7 @@ module.exports = function(context) {
             return;
         }
 
-        if (!/CallExpression|NewExpression/.test(node.parent.type)) {
+        if (!/CallExpression|NewExpression|MemberExpression/.test(node.parent.type)) {
             previousToken = context.getTokenBefore(node);
             nextToken = context.getTokenAfter(node);
             if (previousToken.value === "(" && nextToken.value === ")") {

diff --git a/tests/lib/rules/no-wrap-func.js b/tests/lib/rules/no-wrap-func.js
@@ -23,6 +23,7 @@ eslintTester.addRuleTest("lib/rules/no-wrap-func", {
         "(function() {})()",
         "var a = function() {}",
         "new Object(function() {})",
+        "(function () {thing()}).bar()",
         {
             code: "opts.ease = opts.ease || (a => a * a * a)",
             ecmaFeatures: { arrowFunctions: true }