fix(eslint-plugin): [no-unnec-type-arg] throwing on call/new expr (#1217

) Co-authored-by: Brad Zacher <brad.zacher@gmail.com>
typescript-eslint · Nov 17, 2019 · 42a48de · 42a48de
1 parent ba89168
commit 42a48de
Show file tree

Hide file tree

Showing 2 changed files with 19 additions and 1 deletion.
diff --git a/packages/eslint-plugin/src/rules/no-unnecessary-type-arguments.ts b/packages/eslint-plugin/src/rules/no-unnecessary-type-arguments.ts
@@ -114,7 +114,11 @@ function getTypeParametersFromNode(
     return getTypeParametersFromType(node.typeName, checker);
   }
 
-  return getTypeParametersFromCall(node, checker);
+  if (ts.isCallExpression(node) || ts.isNewExpression(node)) {
+    return getTypeParametersFromCall(node, checker);
+  }
+
+  return undefined;
 }
 
 function getTypeParametersFromType(

diff --git a/packages/eslint-plugin/tests/rules/no-unnecessary-type-arguments.test.ts b/packages/eslint-plugin/tests/rules/no-unnecessary-type-arguments.test.ts
@@ -53,6 +53,9 @@ ruleTester.run('no-unnecessary-type-arguments', rule, {
     `declare const C: unknown;
       class D<TD = number> extends C { }`,
     `let a: A<number>`,
+    `class Foo<T> {}
+      const foo = new Foo<number>();`,
+    `type Foo<T> = import('foo').Foo<T>;`,
   ],
   invalid: [
     {
@@ -123,5 +126,16 @@ ruleTester.run('no-unnecessary-type-arguments', rule, {
       output: `interface I<T = number> { }
         class Impl implements I { }`,
     },
+    {
+      code: `class Foo<T = number> {}
+        const foo = new Foo<number>();`,
+      errors: [
+        {
+          messageId: 'unnecessaryTypeParameter',
+        },
+      ],
+      output: `class Foo<T = number> {}
+        const foo = new Foo();`,
+    },
   ],
 });