fix(eslint-plugin): [no-unnec-type-arg] undefined symbol crash (#1007)

typescript-eslint · Sep 24, 2019 · cdf9294 · cdf9294
1 parent 6279c5b
commit cdf9294
Show file tree

Hide file tree

Showing 2 changed files with 7 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
@@ -121,7 +121,12 @@ function getTypeParametersFromType(
   type: ts.EntityName | ts.Expression | ts.ClassDeclaration,
   checker: ts.TypeChecker,
 ): readonly ts.TypeParameterDeclaration[] | undefined {
-  const sym = getAliasedSymbol(checker.getSymbolAtLocation(type)!, checker);
+  const symAtLocation = checker.getSymbolAtLocation(type);
+  if (symAtLocation === undefined) {
+    return undefined;
+  }
+
+  const sym = getAliasedSymbol(symAtLocation, checker);
   if (sym === undefined || sym.declarations === undefined) {
     return undefined;
   }

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
@@ -52,6 +52,7 @@ ruleTester.run('no-unnecessary-type-arguments', rule, {
       class D<TD = number> extends C { }`,
     `declare const C: unknown;
       class D<TD = number> extends C { }`,
+    `let a: A<number>`,
   ],
   invalid: [
     {