[explicit-function-return-types] default parameters where a type anno…

…tation is present should count as typed function expressions fix typescript-eslint#8950
kirkwaiblinger · May 4, 2024 · fd81f5d · fd81f5d
1 parent f248e68
commit fd81f5d
Show file tree

Hide file tree

Showing 2 changed files with 84 additions and 0 deletions.
diff --git a/packages/eslint-plugin/src/util/explicitReturnTypeUtils.ts b/packages/eslint-plugin/src/util/explicitReturnTypeUtils.ts
@@ -87,12 +87,22 @@ function isTypedParent(
   return (
     isTypeAssertion(parent) ||
     isVariableDeclaratorWithTypeAnnotation(parent) ||
+    isDefaultFunctionParameterWithTypeAnnotation(parent) ||
     isPropertyDefinitionWithTypeAnnotation(parent) ||
     isFunctionArgument(parent, callee) ||
     isTypedJSX(parent)
   );
 }
 
+function isDefaultFunctionParameterWithTypeAnnotation(
+  node: TSESTree.Node,
+): boolean {
+  return (
+    node.type === AST_NODE_TYPES.AssignmentPattern &&
+    node.left.typeAnnotation != null
+  );
+}
+
 /**
  * Checks if a node belongs to:
  * ```

diff --git a/packages/eslint-plugin/tests/rules/explicit-function-return-type.test.ts b/packages/eslint-plugin/tests/rules/explicit-function-return-type.test.ts
@@ -770,7 +770,32 @@ class Bar {
 }
       `,
     },
+    {
+      code: `
+type CallBack = () => void;
+
+function f(gotcha: CallBack = () => {}): void {}
+      `,
+      options: [{ allowTypedFunctionExpressions: true }],
+    },
+    {
+      code: `
+type CallBack = () => void;
+
+const f = (gotcha: CallBack = () => {}): void => {};
+      `,
+      options: [{ allowTypedFunctionExpressions: true }],
+    },
+    {
+      code: `
+type ObjectWithCallback = { callback: () => void };
+
+const f = (gotcha: ObjectWithCallback = { callback: () => {} }): void => {};
+      `,
+      options: [{ allowTypedFunctionExpressions: true }],
+    },
   ],
+
   invalid: [
     {
       code: `
@@ -1940,5 +1965,54 @@ let foo = (() => () => {})()();
         },
       ],
     },
+    {
+      code: `
+type CallBack = () => void;
+
+function f(gotcha: CallBack = () => {}): void {}
+      `,
+      options: [{ allowTypedFunctionExpressions: false }],
+      errors: [
+        {
+          messageId: 'missingReturnType',
+          line: 4,
+          column: 34,
+          endLine: 4,
+          endColumn: 36,
+        },
+      ],
+    },
+    {
+      code: `
+type CallBack = () => void;
+
+const f = (gotcha: CallBack = () => {}): void => {};
+      `,
+      options: [{ allowTypedFunctionExpressions: false }],
+      errors: [
+        {
+          messageId: 'missingReturnType',
+          line: 4,
+          column: 34,
+          endLine: 4,
+          endColumn: 36,
+        },
+      ],
+    },
+    {
+      code: `
+type ObjectWithCallback = { callback: () => void };
+
+const f = (gotcha: ObjectWithCallback = { callback: () => {} }): void => {};
+      `,
+      options: [{ allowTypedFunctionExpressions: false }],
+      errors: [
+        {
+          messageId: 'missingReturnType',
+          line: 4,
+          column: 43,
+        },
+      ],
+    },
   ],
 });