fix(eslint-plugin): [naming-convention] check bodyless function param…

…eters (#2675)
typescript-eslint · Oct 18, 2020 · c505863 · c505863
1 parent d690c8d
commit c505863
Show file tree

Hide file tree

Showing 2 changed files with 18 additions and 2 deletions.
diff --git a/packages/eslint-plugin/src/rules/naming-convention.ts b/packages/eslint-plugin/src/rules/naming-convention.ts
@@ -529,10 +529,11 @@ export default util.createRule<Options, MessageIds>({
       // #endregion function
 
       // #region parameter
-
-      'FunctionDeclaration, TSDeclareFunction, FunctionExpression, ArrowFunctionExpression'(
+      'FunctionDeclaration, TSDeclareFunction, TSEmptyBodyFunctionExpression, FunctionExpression, ArrowFunctionExpression'(
         node:
           | TSESTree.FunctionDeclaration
+          | TSESTree.TSDeclareFunction
+          | TSESTree.TSEmptyBodyFunctionExpression
           | TSESTree.FunctionExpression
           | TSESTree.ArrowFunctionExpression,
       ): void {

diff --git a/packages/eslint-plugin/tests/rules/naming-convention.test.ts b/packages/eslint-plugin/tests/rules/naming-convention.test.ts
@@ -1316,5 +1316,20 @@ ruleTester.run('naming-convention', rule, {
         },
       ],
     },
+    {
+      code: `
+        declare class Foo {
+          Bar(Baz: string): void;
+        }
+      `,
+      parserOptions,
+      options: [{ selector: 'parameter', format: ['camelCase'] }],
+      errors: [
+        {
+          line: 3,
+          messageId: 'doesNotMatchFormat',
+        },
+      ],
+    },
   ],
 });