fix(eslint-plugin): [space-before-function-paren] handle abstract functions (typescript-eslint#2199)

adrielcodeco · bradzacher · web-flow · commit 88a3edfce834 · 2020-07-06T09:59:27.000-07:00
Co-authored-by: Brad Zacher &lt;brad.zacher@gmail.com&gt;
diff --git a/packages/eslint-plugin/src/rules/space-before-function-paren.ts b/packages/eslint-plugin/src/rules/space-before-function-paren.ts
@@ -75,16 +75,18 @@ export default util.createRule<Options, MessageIds>({
       node:
         | TSESTree.ArrowFunctionExpression
         | TSESTree.FunctionDeclaration
-        | TSESTree.FunctionExpression,
+        | TSESTree.FunctionExpression
+        | TSESTree.TSAbstractMethodDefinition,
     ): boolean {
-      if (node.id) {
+      if ('id' in node && node.id != null) {
         return true;
       }
 
       const parent = node.parent!;
 
       return (
         parent.type === AST_NODE_TYPES.MethodDefinition ||
+        parent.type === AST_NODE_TYPES.TSAbstractMethodDefinition ||
         (parent.type === AST_NODE_TYPES.Property &&
           (parent.kind === 'get' || parent.kind === 'set' || parent.method))
       );
@@ -99,7 +101,8 @@ export default util.createRule<Options, MessageIds>({
       node:
         | TSESTree.ArrowFunctionExpression
         | TSESTree.FunctionDeclaration
-        | TSESTree.FunctionExpression,
+        | TSESTree.FunctionExpression
+        | TSESTree.TSAbstractMethodDefinition,
     ): FuncOption {
       if (node.type === AST_NODE_TYPES.ArrowFunctionExpression) {
         // Always ignore non-async functions and arrow functions without parens, e.g. async foo => bar
@@ -113,7 +116,7 @@ export default util.createRule<Options, MessageIds>({
         return overrideConfig.named ?? baseConfig;
 
         // `generator-star-spacing` should warn anonymous generators. E.g. `function* () {}`
-      } else if (!node.generator) {
+      } else if (!('generator' in node) || node.generator === false) {
         return overrideConfig.anonymous ?? baseConfig;
       }
 
@@ -129,7 +132,8 @@ export default util.createRule<Options, MessageIds>({
       node:
         | TSESTree.ArrowFunctionExpression
         | TSESTree.FunctionDeclaration
-        | TSESTree.FunctionExpression,
+        | TSESTree.FunctionExpression
+        | TSESTree.TSAbstractMethodDefinition,
     ): void {
       const functionConfig = getConfigForFunction(node);
 
@@ -161,7 +165,7 @@ export default util.createRule<Options, MessageIds>({
       } else if (
         !hasSpacing &&
         functionConfig === 'always' &&
-        (!node.typeParameters || node.id)
+        (!node.typeParameters || ('id' in node && node != null))
       ) {
         context.report({
           node,
@@ -176,6 +180,7 @@ export default util.createRule<Options, MessageIds>({
       ArrowFunctionExpression: checkFunction,
       FunctionDeclaration: checkFunction,
       FunctionExpression: checkFunction,
+      TSAbstractMethodDefinition: checkFunction,
     };
   },
 });
diff --git a/packages/eslint-plugin/tests/rules/space-before-function-paren.test.ts b/packages/eslint-plugin/tests/rules/space-before-function-paren.test.ts
@@ -148,6 +148,11 @@ ruleTester.run('space-before-function-paren', rule, {
       options: ['never'],
       parserOptions: { ecmaVersion: 8 },
     },
+    'abstract class Foo { constructor () {} abstract method () }',
+    {
+      code: 'abstract class Foo { constructor() {} abstract method() }',
+      options: ['never'],
+    },
   ],
 
   invalid: [
