fix: regression with checking TS MethodDefinition params

gajus · Apr 13, 2022 · 041de5f · 041de5f
1 parent 288363e
commit 041de5f
Show file tree

Hide file tree

Showing 2 changed files with 25 additions and 1 deletion.
diff --git a/src/jsdocUtils.js b/src/jsdocUtils.js
@@ -271,7 +271,7 @@ const getFunctionParameterNames = (
     throw new Error(`Unsupported function signature format: \`${param.type}\`.`);
   };
 
-  return functionNode.params.map((param) => {
+  return (functionNode.params || functionNode.value.params).map((param) => {
     return getParamName(param);
   });
 };

diff --git a/test/rules/assertions/requireParam.js b/test/rules/assertions/requireParam.js
@@ -3366,5 +3366,29 @@ export default {
         sourceType: 'module',
       },
     },
+    {
+      code: `
+      /**
+       * [A description]
+       */
+      class A {
+        /**
+         * @param config
+         */
+        constructor (config: SomeConfig) {
+          super(config);
+        }
+      }
+      `,
+      ignoreReadme: true,
+      options: [
+        {
+          contexts: [
+            'MethodDefinition',
+          ],
+        },
+      ],
+      parser: require.resolve('@typescript-eslint/parser'),
+    },
   ],
 };