fix(typescript-estree): fix regression introduced in #1525 (#1543)

typescript-eslint · Jan 29, 2020 · bec4572 · bec4572
1 parent 6bebb1d
commit bec4572
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 6 deletions.
diff --git a/packages/typescript-estree/src/convert.ts b/packages/typescript-estree/src/convert.ts
@@ -379,10 +379,7 @@ export class Converter {
    * property instead of a kind property. Recursively copies all children.
    */
   private deeplyCopy(node: TSNode): any {
-    if (
-      node.kind >= SyntaxKind.FirstJSDocNode &&
-      node.kind <= SyntaxKind.LastJSDocNode
-    ) {
+    if (node.kind === ts.SyntaxKind.JSDocFunctionType) {
       throw createError(
         this.ast,
         node.pos,

diff --git a/packages/typescript-estree/tests/lib/convert.ts b/packages/typescript-estree/tests/lib/convert.ts
@@ -239,8 +239,9 @@ describe('convert', () => {
 
   it('should throw error on jsDoc node', () => {
     const jsDocCode = [
-      'type foo = ?foo<T> | ?(() => void)?',
-      'var a: function(b): c;',
+      'const x: function(new: number, string);',
+      'const x: function(this: number, string);',
+      'var g: function(number, number): number;',
     ];
 
     for (const code of jsDocCode) {