eslint-plugin-cypress 6.2.2 → 6.2.3

package/lib/rules/no-unnecessary-waiting.js

@@ -79,13 +79,17 @@ function isIdentifierNumberConstArgument(node, scope) {
   // we don't know if WAIT is a number or alias '@someRequest', so don't fail
   if (definition.type === 'ImportBinding') return false
 
-  const param = definition.node.params[definition.index]
+  // Get the identifier's parent node to check for a default value.
+  // For `function wait (amount = 1) { ... }`, `definition.name` is the `amount` Identifier,
+  // and its parent is the AssignmentPattern `amount = 1`.
+  // This also handles destructured parameters like `function wait ({ ms = 1 }) { ... }`.
+  const nameParent = definition.name.parent
 
   // function wait (amount) { cy.wait(amount) }
   // we can't know the type of value, so don't fail
-  if (!param || param.type !== 'AssignmentPattern') return false
+  if (!nameParent || nameParent.type !== 'AssignmentPattern') return false
 
   // function wait (amount = 1) { cy.wait(amount) } or
   // function wait (amount = '@alias') { cy.wait(amount) }
-  return typeof param.right.value === 'number'
+  return typeof nameParent.right.value === 'number'
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "eslint-plugin-cypress",
-  "version": "6.2.2",
+  "version": "6.2.3",
   "description": "An ESLint plugin for projects using Cypress",
   "main": "./lib/index.js",
   "exports": {