fix(check-param-names, require-param): properly check object parameters with default values; fixes gajus#635

Author: brettz9

README.md

@@ -2456,6 +2456,22 @@ function quux ({ foo: { bar } }) {}
  * @param {FooBar} options.baz
  */
 function quux ({ foo: { bar }, baz: { cfg } }) {}
+
+/**
+ * Item
+ *
+ * @param {object} props
+ * @param {object} props.data - case data
+ * @param {string} props.data.className - additional css class
+ * @param props.val
+ */
+export default function Item({
+  data: {
+    className,
+  } = {},
+  val = 4
+}) {
+}
 ````
 
 

src/jsdocUtils.js

@@ -140,6 +140,13 @@ const getFunctionParameterNames = (functionNode : Object) : Array<T> => {
           return getParamName(prop, isProperty);
         })];
       }
+      if (param.value.type === 'AssignmentPattern' &&
+        param.value.left.type === 'ObjectPattern'
+      ) {
+        return [param.key.name, param.value.left.properties.map((prop) => {
+          return getParamName(prop, isProperty);
+        })];
+      }
 
       // As function parameters, these do not allow dynamic properties, etc.
       /* istanbul ignore else */

test/rules/assertions/checkParamNames.js

@@ -1382,5 +1382,27 @@ export default {
       function quux ({ foo: { bar }, baz: { cfg } }) {}
       `,
     },
+    {
+      code: `
+      /**
+       * Item
+       *
+       * @param {object} props
+       * @param {object} props.data - case data
+       * @param {string} props.data.className - additional css class
+       * @param props.val
+       */
+      export default function Item({
+        data: {
+          className,
+        } = {},
+        val = 4
+      }) {
+      }
+      `,
+      parserOptions: {
+        sourceType: 'module',
+      },
+    },
   ],
 };