fix(check-param-names, require-param): ignore TSIndexSignature; fixes #529

Author: brettz9

README.md

@@ -2066,6 +2066,16 @@ const bboxToObj = function ({x, y, width, height}) {
   return {x, y, width, height};
 };
 // Options: [{"checkTypesPattern":"SVGRect"}]
+
+class CSS {
+  /**
+   * Set one or more CSS properties for the set of matched elements.
+   *
+   * @param {Object} propertyObject - An object of property-value pairs to set.
+   */
+  setCssObject(propertyObject: {[key: string]: string | number}): void {
+  }
+}
 ````
 
 

src/jsdocUtils.js

@@ -51,7 +51,7 @@ const flattenRoots = (params, root = '') => {
         hasRestElement = true;
       }
       acc.push(root ? `${root}.${cur.name}` : cur.name);
-    } else {
+    } else if (typeof cur !== 'undefined') {
       rests.push(false);
       acc.push(root ? `${root}.${cur}` : cur);
     }
@@ -74,6 +74,9 @@ const getPropertiesFromPropertySignature = (propSignature): T => {
       return getPropertiesFromPropertySignature(member);
     })];
   }
+  if (propSignature.type === 'TSIndexSignature') {
+    return undefined;
+  }
 
   return propSignature.key.name;
 };

test/rules/assertions/checkParamNames.js

@@ -948,5 +948,19 @@ export default {
         },
       ],
     },
+    {
+      code: `
+      class CSS {
+        /**
+         * Set one or more CSS properties for the set of matched elements.
+         *
+         * @param {Object} propertyObject - An object of property-value pairs to set.
+         */
+        setCssObject(propertyObject: {[key: string]: string | number}): void {
+        }
+      }
+      `,
+      parser: require.resolve('@typescript-eslint/parser'),
+    },
   ],
 };