trailing-comma: check for a closing parenthesis (#4457)

fixes #4456
palantir · Jan 29, 2019 · 8486bf0 · 8486bf0
1 parent 4958030
commit 8486bf0
Showing 1 changed file with 12 additions and 5 deletions.
diff --git a/src/rules/trailingCommaRule.ts b/src/rules/trailingCommaRule.ts
@@ -229,12 +229,19 @@ class TrailingCommaWalker extends Lint.AbstractWalker<Options> {
                 case ts.SyntaxKind.ConstructorType:
                 case ts.SyntaxKind.FunctionType:
                 case ts.SyntaxKind.CallSignature:
-                    this.checkList(
-                        (node as ts.SignatureDeclaration).parameters,
-                        getChildOfKind(node, ts.SyntaxKind.CloseParenToken, this.sourceFile)!.end,
-                        "functions",
-                        isRestParameter,
+                    const closingParen = getChildOfKind(
+                        node,
+                        ts.SyntaxKind.CloseParenToken,
+                        this.sourceFile,
                     );
+                    if (closingParen !== undefined) {
+                        this.checkList(
+                            (node as ts.SignatureDeclaration).parameters,
+                            closingParen.end,
+                            "functions",
+                            isRestParameter,
+                        );
+                    }
                     break;
                 case ts.SyntaxKind.TypeLiteral:
                     this.checkTypeLiteral(node as ts.TypeLiteralNode);