[Fix] no-object-type-as-default-prop: enable rule for components with many parameters

JulienR1 · JulienR1 · commit 62e763810b5d · 2024-06-12T20:49:45.000-04:00
diff --git a/lib/rules/no-object-type-as-default-prop.js b/lib/rules/no-object-type-as-default-prop.js
@@ -30,7 +30,7 @@ const messages = {
 function hasUsedObjectDestructuringSyntax(params) {
   return (
     params != null
-    && params.length === 1
+    && params.length >= 1
     && params[0].type === 'ObjectPattern'
   );
 }
diff --git a/tests/lib/rules/no-object-type-as-default-prop.js b/tests/lib/rules/no-object-type-as-default-prop.js
@@ -143,6 +143,11 @@ ruleTester.run('no-object-type-as-default-prop', rule, {
         return null;
       };
     `,
+    `
+      const Foo = ({bar = 1}, context) => {
+        return null;
+      };
+    `,
     `
       export default function NotAComponent({foo = {}}) {}
     `
@@ -183,6 +188,24 @@ ruleTester.run('no-object-type-as-default-prop', rule, {
         }
       `,
       errors: expectedViolations,
+    },
+    {
+      code: `
+        const Foo = ({
+          a = {},
+          b = ['one', 'two'],
+          c = /regex/i,
+          d = () => {},
+          e = function() {},
+          f = class {},
+          g = new Thing(),
+          h = <Thing />,
+          i = Symbol('foo')
+        }, context) => {
+          return null;
+        }
+      `,
+      errors: expectedViolations,
     }
   )),
 });

Original file line number	Diff line number	Diff line change
`@@ -30,7 +30,7 @@ const messages = {`
`30`	`30`	`function hasUsedObjectDestructuringSyntax(params) {`
`31`	`31`	`return (`
`32`	`32`	`params != null`
`33`		`- && params.length === 1`
	`33`	`+ && params.length >= 1`
`34`	`34`	`&& params[0].type === 'ObjectPattern'`
`35`	`35`	`);`
`36`	`36`	`}`