Fix: no-useless-rest-spread crashed (fixes #17)

mysticatea · Oct 18, 2017 · 30a5928 · 30a5928
1 parent bfb5fa7
commit 30a5928
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/lib/rules/no-useless-rest-spread.js b/lib/rules/no-useless-rest-spread.js
@@ -115,8 +115,9 @@ module.exports = {
             const parentType = node.parent.type
             const argumentType = node.argument.type
             const isArray = argumentType.startsWith("Array")
+            const isObject = !isArray && argumentType.startsWith("Object")
             const isRedundant = (
-                argumentType === parentType ||
+                ((isArray || isObject) && argumentType === parentType) ||
                 (isArray && FUNC_TYPE.test(parentType))
             )
 

diff --git a/tests/lib/rules/no-useless-rest-spread.js b/tests/lib/rules/no-useless-rest-spread.js
@@ -35,6 +35,7 @@ ruleTester.run("no-useless-rest-spread", rule, {
         "foo(...{a}, ...{b})",
         "let list = [...{a}]",
         "let obj = {...[a]}",
+        "f(...g())",
     ],
     invalid: [
         {