eslint-plugin-cypress 5.0.0 → 5.0.1

package/CONTRIBUTING.md

@@ -32,7 +32,7 @@ To add a new rule:
 * Run `eslint-doc-generator` to generate automated documentation sections (see [Document generation](#document-generation) below)
 * Review documentation changes
 * Run `npm run lint`
-* Run `npm test` to run [Jest](https://jestjs.io/) (or run `npm start` to run [Jest](https://jestjs.io/) in [watchAll](https://jestjs.io/docs/cli#--watchall) mode where it remains active and reruns when source changes are made)
+* Run `npm test` to run [Vitest](https://vitest.dev/)
 * Make sure all tests are passing
 * Add the rule to [flat.js](https://github.com/cypress-io/eslint-plugin-cypress/blob/master/lib/flat.js)
 * Create a git commit with a commit message similar to: `feat: add rule <description>` (see [commit message conventions](https://github.com/semantic-release/semantic-release#commit-message-format))

package/eslint.config.mjs

@@ -2,25 +2,32 @@ import globals from 'globals'
 import pluginJs from '@eslint/js'
 import eslintPlugin from 'eslint-plugin-eslint-plugin'
 import mochaPlugin from 'eslint-plugin-mocha'
+import stylistic from '@stylistic/eslint-plugin'
 
 export default [
   pluginJs.configs.recommended,
   eslintPlugin.configs['flat/recommended'],
   mochaPlugin.configs.recommended,
-  {ignores: ['test-project/']},
+  stylistic.configs.recommended,
+  { ignores: ['test-project/'] },
   {
     languageOptions: {
-      globals: globals.node
+      globals: globals.node,
     },
     rules: {
       'no-redeclare': 'off',
+      '@stylistic/arrow-parens': ['error', 'always'],
+      '@stylistic/indent': ['error', 2],
+      '@stylistic/comma-dangle': ['error', 'always-multiline'],
+      '@stylistic/quotes': ['error', 'single'],
+      '@stylistic/semi': ['error', 'never'],
       'eslint-plugin/require-meta-docs-url':
-        ['error', { 'pattern': 'https://github.com/cypress-io/eslint-plugin-cypress/blob/master/docs/rules/{{name}}.md' }],
+        ['error', { pattern: 'https://github.com/cypress-io/eslint-plugin-cypress/blob/master/docs/rules/{{name}}.md' }],
       'eslint-plugin/require-meta-docs-description': 'error',
       'eslint-plugin/meta-property-ordering': 'error',
       'eslint-plugin/test-case-property-ordering': 'error',
       'mocha/no-mocha-arrows': 'off',
-      'mocha/no-setup-in-describe': 'off'
-    }
+      'mocha/no-setup-in-describe': 'off',
+    },
   },
 ]

package/lib/flat.js

@@ -20,8 +20,8 @@ const plugin = {
   },
 }
 
-const commonGlobals =
-  Object.assign({
+const commonGlobals
+  = Object.assign({
     cy: false,
     Cypress: false,
     expect: false,
@@ -33,20 +33,20 @@ Object.assign(plugin.configs, {
   globals: {
     name: 'cypress/globals',
     plugins: {
-      cypress: plugin
+      cypress: plugin,
     },
     languageOptions: {
       globals:
         commonGlobals,
-    }
-  }
+    },
+  },
 })
 
 Object.assign(plugin.configs, {
   recommended: {
     name: 'cypress/recommended',
     plugins: {
-      cypress: plugin
+      cypress: plugin,
     },
     rules: {
       'cypress/no-assigning-return-values': 'error',
@@ -57,8 +57,8 @@ Object.assign(plugin.configs, {
     languageOptions: {
       globals:
         commonGlobals,
-    }
-  }
+    },
+  },
 })
 
 module.exports = plugin

package/lib/rules/assertion-before-screenshot.js

@@ -28,9 +28,9 @@ module.exports = {
       unexpected: 'Make an assertion on the page state before taking a screenshot',
     },
   },
-  create (context) {
+  create(context) {
     return {
-      CallExpression (node) {
+      CallExpression(node) {
         if (isCallingCyScreenshot(node) && !isPreviousAnAssertion(node)) {
           context.report({ node, messageId: 'unexpected' })
         }
@@ -39,12 +39,12 @@ module.exports = {
   },
 }
 
-function isRootCypress (node) {
+function isRootCypress(node) {
   while (node.type === 'CallExpression') {
     if (node.callee.type !== 'MemberExpression') return false
 
-    if (node.callee.object.type === 'Identifier' &&
-        node.callee.object.name === 'cy') {
+    if (node.callee.object.type === 'Identifier'
+      && node.callee.object.name === 'cy') {
       return true
     }
 
@@ -54,26 +54,26 @@ function isRootCypress (node) {
   return false
 }
 
-function getPreviousInChain (node) {
-  return node.type === 'CallExpression' &&
-         node.callee.type === 'MemberExpression' &&
-         node.callee.object.type === 'CallExpression' &&
-         node.callee.object.callee.type === 'MemberExpression' &&
-         node.callee.object.callee.property.type === 'Identifier' &&
-         node.callee.object.callee.property.name
+function getPreviousInChain(node) {
+  return node.type === 'CallExpression'
+    && node.callee.type === 'MemberExpression'
+    && node.callee.object.type === 'CallExpression'
+    && node.callee.object.callee.type === 'MemberExpression'
+    && node.callee.object.callee.property.type === 'Identifier'
+    && node.callee.object.callee.property.name
 }
 
-function getCallExpressionCypressCommand (node) {
-  return isRootCypress(node) &&
-         node.callee.property.type === 'Identifier' &&
-         node.callee.property.name
+function getCallExpressionCypressCommand(node) {
+  return isRootCypress(node)
+    && node.callee.property.type === 'Identifier'
+    && node.callee.property.name
 }
 
-function isCallingCyScreenshot (node) {
+function isCallingCyScreenshot(node) {
   return getCallExpressionCypressCommand(node) === 'screenshot'
 }
 
-function getPreviousCypressCommand (node) {
+function getPreviousCypressCommand(node) {
   const previousInChain = getPreviousInChain(node)
 
   if (previousInChain) {
@@ -97,15 +97,15 @@ function getPreviousCypressCommand (node) {
 
   const previousStatement = body[index - 1]
 
-  if (previousStatement.type !== 'ExpressionStatement' ||
-      previousStatement.expression.type !== 'CallExpression') {
+  if (previousStatement.type !== 'ExpressionStatement'
+    || previousStatement.expression.type !== 'CallExpression') {
     return null
   }
 
   return getCallExpressionCypressCommand(previousStatement.expression)
 }
 
-function isPreviousAnAssertion (node) {
+function isPreviousAnAssertion(node) {
   const previousCypressCommand = getPreviousCypressCommand(node)
 
   return assertionCommands.indexOf(previousCypressCommand) >= 0

package/lib/rules/no-assigning-return-values.js

@@ -1,7 +1,7 @@
 'use strict'
 
 // safely get nested object property
-function get (obj, propertyString = '') {
+function get(obj, propertyString = '') {
   const properties = propertyString.split('.')
 
   for (let i = 0; i < properties.length; i++) {
@@ -29,9 +29,9 @@ module.exports = {
       unexpected: 'Do not assign the return value of a Cypress command',
     },
   },
-  create (context) {
+  create(context) {
     return {
-      VariableDeclaration (node) {
+      VariableDeclaration(node) {
         if (node.declarations.some(isCypressCommandDeclaration)) {
           context.report({ node, messageId: 'unexpected' })
         }
@@ -47,7 +47,7 @@ const allowedCommands = {
   stub: true,
 }
 
-function isCypressCommandDeclaration (declarator) {
+function isCypressCommandDeclaration(declarator) {
   let object = get(declarator, 'init.callee.object')
 
   if (!object) return

package/lib/rules/no-async-before.js

@@ -15,31 +15,31 @@ module.exports = {
     },
   },
 
-  create (context) {
-    function isBeforeBlock (callExpressionNode) {
+  create(context) {
+    function isBeforeBlock(callExpressionNode) {
       const { type, name } = callExpressionNode.callee
 
       return type === 'Identifier'
-                && name === 'before' || name === 'beforeEach'
+        && (name === 'before' || name === 'beforeEach')
     }
 
-    function isBeforeAsync (node) {
+    function isBeforeAsync(node) {
       return node.arguments
-                && node.arguments.length >= 2
-                && node.arguments[1].async === true
+        && node.arguments.length >= 2
+        && node.arguments[1].async === true
     }
     const sourceCode = context.sourceCode ?? context.getSourceCode()
 
     return {
-      Identifier (node) {
+      Identifier(node) {
         if (node.name === 'cy' || node.name === 'Cypress') {
           const ancestors = sourceCode.getAncestors
             ? sourceCode.getAncestors(node)
             : context.getAncestors()
           const asyncTestBlocks = ancestors
-          .filter((n) => n.type === 'CallExpression')
-          .filter(isBeforeBlock)
-          .filter(isBeforeAsync)
+            .filter((n) => n.type === 'CallExpression')
+            .filter(isBeforeBlock)
+            .filter(isBeforeAsync)
 
           if (asyncTestBlocks.length >= 1) {
             asyncTestBlocks.forEach((node) => {

package/lib/rules/no-async-tests.js

@@ -15,31 +15,31 @@ module.exports = {
     },
   },
 
-  create (context) {
-    function isTestBlock (callExpressionNode) {
+  create(context) {
+    function isTestBlock(callExpressionNode) {
       const { type, name } = callExpressionNode.callee
 
       return type === 'Identifier'
-                && name === 'it' || name === 'test'
+        && (name === 'it' || name === 'test')
     }
 
-    function isTestAsync (node) {
+    function isTestAsync(node) {
       return node.arguments
-                && node.arguments.length >= 2
-                && node.arguments[1].async === true
+        && node.arguments.length >= 2
+        && node.arguments[1].async === true
     }
     const sourceCode = context.sourceCode ?? context.getSourceCode()
 
     return {
-      Identifier (node) {
+      Identifier(node) {
         if (node.name === 'cy' || node.name === 'Cypress') {
           const ancestors = sourceCode.getAncestors
             ? sourceCode.getAncestors(node)
             : context.getAncestors()
           const asyncTestBlocks = ancestors
-          .filter((n) => n.type === 'CallExpression')
-          .filter(isTestBlock)
-          .filter(isTestAsync)
+            .filter((n) => n.type === 'CallExpression')
+            .filter(isTestBlock)
+            .filter(isTestAsync)
 
           if (asyncTestBlocks.length >= 1) {
             asyncTestBlocks.forEach((node) => {

package/lib/rules/no-chained-get.js

@@ -1,8 +1,8 @@
-'use strict';
+'use strict'
 
-//------------------------------------------------------------------------------
+// ------------------------------------------------------------------------------
 // Rule Definition
-//------------------------------------------------------------------------------
+// ------------------------------------------------------------------------------
 
 /** @type {import('eslint').Rule.RuleModule} */
 module.exports = {
@@ -23,55 +23,55 @@ module.exports = {
   create(context) {
     const isRootCypress = (node) => {
       if (
-        node.type !== 'CallExpression' ||
-        node.callee.type !== 'MemberExpression'
+        node.type !== 'CallExpression'
+        || node.callee.type !== 'MemberExpression'
       ) {
-        return false;
+        return false
       }
 
       if (
-        node.callee.object.type === 'Identifier' &&
-        node.callee.object.name === 'cy'
+        node.callee.object.type === 'Identifier'
+        && node.callee.object.name === 'cy'
       ) {
-        return true;
+        return true
       }
 
-      return isRootCypress(node.callee.object);
-    };
+      return isRootCypress(node.callee.object)
+    }
 
     const hasChainedGet = (node) => {
       // Check if this node is a get() call
-      const isGetCall =
-        node.callee &&
-        node.callee.type === 'MemberExpression' &&
-        node.callee.property &&
-        node.callee.property.type === 'Identifier' &&
-        node.callee.property.name === 'get';
+      const isGetCall
+        = node.callee
+          && node.callee.type === 'MemberExpression'
+          && node.callee.property
+          && node.callee.property.type === 'Identifier'
+          && node.callee.property.name === 'get'
 
       if (!isGetCall) {
-        return false;
+        return false
       }
 
-      const obj = node.callee.object;
+      const obj = node.callee.object
 
       if (obj.type === 'CallExpression') {
-        const objCallee = obj.callee;
+        const objCallee = obj.callee
 
         if (
-          objCallee &&
-          objCallee.type === 'MemberExpression' &&
-          objCallee.property &&
-          objCallee.property.type === 'Identifier' &&
-          objCallee.property.name === 'get'
+          objCallee
+          && objCallee.type === 'MemberExpression'
+          && objCallee.property
+          && objCallee.property.type === 'Identifier'
+          && objCallee.property.name === 'get'
         ) {
-          return true;
+          return true
         }
 
-        return hasChainedGet(obj);
+        return hasChainedGet(obj)
       }
 
-      return false;
-    };
+      return false
+    }
 
     return {
       CallExpression(node) {
@@ -79,9 +79,9 @@ module.exports = {
           context.report({
             node,
             messageId: 'unexpected',
-          });
+          })
         }
       },
-    };
+    }
   },
-};
+}

package/lib/rules/no-debug.js

@@ -1,8 +1,8 @@
 'use strict'
 
-//------------------------------------------------------------------------------
+// ------------------------------------------------------------------------------
 // Rule Definition
-//------------------------------------------------------------------------------
+// ------------------------------------------------------------------------------
 
 module.exports = {
   meta: {
@@ -20,37 +20,36 @@ module.exports = {
     },
   },
 
-  create (context) {
-
+  create(context) {
     // variables should be defined here
 
-    //----------------------------------------------------------------------
+    // ----------------------------------------------------------------------
     // Helpers
-    //----------------------------------------------------------------------
-    function isCallingDebug (node) {
-      return node.callee &&
-        node.callee.property &&
-        node.callee.property.type === 'Identifier' &&
-        node.callee.property.name === 'debug'
+    // ----------------------------------------------------------------------
+    function isCallingDebug(node) {
+      return node.callee
+        && node.callee.property
+        && node.callee.property.type === 'Identifier'
+        && node.callee.property.name === 'debug'
     }
 
-    function isCypressCall (node) {
+    function isCypressCall(node) {
       if (!node.callee || node.callee.type !== 'MemberExpression') {
-        return false;
+        return false
       }
       if (node.callee.object.type === 'Identifier' && node.callee.object.name === 'cy') {
-        return true;
+        return true
       }
-      return isCypressCall(node.callee.object);
+      return isCypressCall(node.callee.object)
     }
-    
-    //----------------------------------------------------------------------
+
+    // ----------------------------------------------------------------------
     // Public
-    //----------------------------------------------------------------------
+    // ----------------------------------------------------------------------
 
     return {
 
-      CallExpression (node) {
+      CallExpression(node) {
         if (isCypressCall(node) && isCallingDebug(node)) {
           context.report({ node, messageId: 'unexpected' })
         }

package/lib/rules/no-force.js

@@ -1,8 +1,8 @@
 'use strict'
 
-//------------------------------------------------------------------------------
+// ------------------------------------------------------------------------------
 // Rule Definition
-//------------------------------------------------------------------------------
+// ------------------------------------------------------------------------------
 
 module.exports = {
   meta: {
@@ -20,39 +20,36 @@ module.exports = {
     },
   },
 
-  create (context) {
-
+  create(context) {
     // variables should be defined here
 
-    //----------------------------------------------------------------------
+    // ----------------------------------------------------------------------
     // Helpers
-    //----------------------------------------------------------------------
-    function isCallingClickOrType (node) {
+    // ----------------------------------------------------------------------
+    function isCallingClickOrType(node) {
       const allowedMethods = ['click', 'dblclick', 'type', 'trigger', 'check', 'rightclick', 'focus', 'select']
 
-      return node.property && node.property.type === 'Identifier' &&
-             allowedMethods.includes(node.property.name)
+      return node.property && node.property.type === 'Identifier'
+        && allowedMethods.includes(node.property.name)
     }
 
-    function isCypressCall (node) {
-      return node.callee.type === 'MemberExpression' &&
-        node.callee.object.type === 'Identifier' &&
-        node.callee.object.name === 'cy'
+    function isCypressCall(node) {
+      return node.callee.type === 'MemberExpression'
+        && node.callee.object.type === 'Identifier'
+        && node.callee.object.name === 'cy'
     }
 
-    function hasOptionForce (node) {
-
-      return node.arguments && node.arguments.length &&
-      node.arguments.some((arg) => {
-        return arg.type === 'ObjectExpression' && arg.properties.some((propNode) => propNode.key && propNode.key.name === 'force')
-      })
+    function hasOptionForce(node) {
+      return node.arguments && node.arguments.length
+        && node.arguments.some((arg) => {
+          return arg.type === 'ObjectExpression' && arg.properties.some((propNode) => propNode.key && propNode.key.name === 'force')
+        })
     }
 
-    function deepCheck (node, checkFunc) {
+    function deepCheck(node, checkFunc) {
       let currentNode = node
 
       while (currentNode.parent) {
-
         if (checkFunc(currentNode.parent)) {
           return true
         }
@@ -63,13 +60,13 @@ module.exports = {
       return false
     }
 
-    //----------------------------------------------------------------------
+    // ----------------------------------------------------------------------
     // Public
-    //----------------------------------------------------------------------
+    // ----------------------------------------------------------------------
 
     return {
 
-      CallExpression (node) {
+      CallExpression(node) {
         if (isCypressCall(node) && deepCheck(node, isCallingClickOrType) && deepCheck(node, hasOptionForce)) {
           context.report({ node, messageId: 'unexpected' })
         }

package/lib/rules/no-pause.js

@@ -1,8 +1,8 @@
 'use strict'
 
-//------------------------------------------------------------------------------
+// ------------------------------------------------------------------------------
 // Rule Definition
-//------------------------------------------------------------------------------
+// ------------------------------------------------------------------------------
 
 module.exports = {
   meta: {
@@ -20,37 +20,36 @@ module.exports = {
     },
   },
 
-  create (context) {
-
+  create(context) {
     // variables should be defined here
 
-    //----------------------------------------------------------------------
+    // ----------------------------------------------------------------------
     // Helpers
-    //----------------------------------------------------------------------
-    function isCallingPause (node) {
-      return node.callee &&
-        node.callee.property &&
-        node.callee.property.type === 'Identifier' &&
-        node.callee.property.name === 'pause'
+    // ----------------------------------------------------------------------
+    function isCallingPause(node) {
+      return node.callee
+        && node.callee.property
+        && node.callee.property.type === 'Identifier'
+        && node.callee.property.name === 'pause'
     }
 
-    function isCypressCall (node) {
+    function isCypressCall(node) {
       if (!node.callee || node.callee.type !== 'MemberExpression') {
-        return false;
+        return false
       }
       if (node.callee.object.type === 'Identifier' && node.callee.object.name === 'cy') {
-        return true;
+        return true
       }
-      return isCypressCall(node.callee.object);
+      return isCypressCall(node.callee.object)
     }
-    
-    //----------------------------------------------------------------------
+
+    // ----------------------------------------------------------------------
     // Public
-    //----------------------------------------------------------------------
+    // ----------------------------------------------------------------------
 
     return {
 
-      CallExpression (node) {
+      CallExpression(node) {
         if (isCypressCall(node) && isCallingPause(node)) {
           context.report({ node, messageId: 'unexpected' })
         }

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

@@ -14,11 +14,11 @@ module.exports = {
       unexpected: 'Do not wait for arbitrary time periods',
     },
   },
-  create (context) {
+  create(context) {
     const sourceCode = context.sourceCode ?? context.getSourceCode()
 
     return {
-      CallExpression (node) {
+      CallExpression(node) {
         if (isCallingCyWait(node)) {
           const scope = sourceCode.getScope
             ? sourceCode.getScope(node)
@@ -33,7 +33,7 @@ module.exports = {
   },
 }
 
-function nodeIsCalledByCy (node) {
+function nodeIsCalledByCy(node) {
   if (node.type === 'Identifier' && node.name === 'cy') return true
 
   if (typeof node.callee === 'undefined' || typeof node.callee.object === 'undefined') {
@@ -43,20 +43,20 @@ function nodeIsCalledByCy (node) {
   return nodeIsCalledByCy(node.callee.object)
 }
 
-function isCallingCyWait (node) {
-  return node.callee.type === 'MemberExpression' &&
-         nodeIsCalledByCy(node) &&
-         node.callee.property.type === 'Identifier' &&
-         node.callee.property.name === 'wait'
+function isCallingCyWait(node) {
+  return node.callee.type === 'MemberExpression'
+    && nodeIsCalledByCy(node)
+    && node.callee.property.type === 'Identifier'
+    && node.callee.property.name === 'wait'
 }
 
-function isNumberArgument (node) {
-  return node.arguments.length > 0 &&
-         node.arguments[0].type === 'Literal' &&
-         typeof (node.arguments[0].value) === 'number'
+function isNumberArgument(node) {
+  return node.arguments.length > 0
+    && node.arguments[0].type === 'Literal'
+    && typeof (node.arguments[0].value) === 'number'
 }
 
-function isIdentifierNumberConstArgument (node, scope) {
+function isIdentifierNumberConstArgument(node, scope) {
   if (node.arguments.length === 0) return false
 
   if (node.arguments[0].type !== 'Identifier') return false

package/lib/rules/no-xpath.js

@@ -6,7 +6,7 @@ module.exports = {
     docs: {
       description: 'disallow using `cy.xpath()` calls',
       recommended: false,
-      url: 'https://github.com/cypress-io/eslint-plugin-cypress/blob/master/docs/rules/no-xpath.md'
+      url: 'https://github.com/cypress-io/eslint-plugin-cypress/blob/master/docs/rules/no-xpath.md',
     },
     fixable: null, // Or `code` or `whitespace`
     schema: [], // Add a schema if the rule has options
@@ -15,21 +15,21 @@ module.exports = {
     },
   },
 
-  create (context) {
+  create(context) {
     return {
-      CallExpression (node) {
+      CallExpression(node) {
         if (isCallingCyXpath(node)) {
           context.report({ node, messageId: 'unexpected' })
         }
       },
     }
   },
-};
+}
 
-function isCallingCyXpath (node) {
-  return node.callee.type === 'MemberExpression' &&
-         node.callee.object.type === 'Identifier' &&
-         node.callee.object.name === 'cy' &&
-         node.callee.property.type === 'Identifier' &&
-         node.callee.property.name === 'xpath'
+function isCallingCyXpath(node) {
+  return node.callee.type === 'MemberExpression'
+    && node.callee.object.type === 'Identifier'
+    && node.callee.object.name === 'cy'
+    && node.callee.property.type === 'Identifier'
+    && node.callee.property.name === 'xpath'
 }

package/lib/rules/require-data-selectors.js

@@ -15,9 +15,9 @@ module.exports = {
     },
   },
 
-  create (context) {
+  create(context) {
     return {
-      CallExpression (node) {
+      CallExpression(node) {
         if (isCallingCyGet(node) && !isDataArgument(node)) {
           context.report({ node, messageId: 'unexpected' })
         }
@@ -26,23 +26,23 @@ module.exports = {
   },
 }
 
-function isCallingCyGet (node) {
-  return node.callee.type === 'MemberExpression' &&
-         node.callee.object.type === 'Identifier' &&
-         node.callee.object.name === 'cy' &&
-         node.callee.property.type === 'Identifier' &&
-         node.callee.property.name === 'get'
+function isCallingCyGet(node) {
+  return node.callee.type === 'MemberExpression'
+    && node.callee.object.type === 'Identifier'
+    && node.callee.object.name === 'cy'
+    && node.callee.property.type === 'Identifier'
+    && node.callee.property.name === 'get'
 }
 
-function isDataArgument (node) {
-  return node.arguments.length > 0 &&
-    (
-      (node.arguments[0].type === 'Literal' && isAliasOrDataSelector(String(node.arguments[0].value))) ||
-      (node.arguments[0].type === 'TemplateLiteral' && isAliasOrDataSelector(String(node.arguments[0].quasis[0].value.cooked)))
+function isDataArgument(node) {
+  return node.arguments.length > 0
+    && (
+      (node.arguments[0].type === 'Literal' && isAliasOrDataSelector(String(node.arguments[0].value)))
+      || (node.arguments[0].type === 'TemplateLiteral' && isAliasOrDataSelector(String(node.arguments[0].quasis[0].value.cooked)))
     )
 }
 
-function isAliasOrDataSelector (selector) {
+function isAliasOrDataSelector(selector) {
   return ['[data-', '@'].some(function (validValue) {
     return selector.startsWith(validValue)
   })

package/lib/rules/unsafe-to-chain-command.js

@@ -82,15 +82,15 @@ module.exports = {
         'It is unsafe to chain further commands that rely on the subject after this command. It is best to split the chain, chaining again from `cy.` in a next command line.',
     },
   },
-  create (context) {
+  create(context) {
     const { methods } = getDefaultOptions(context)
 
     return {
-      CallExpression (node) {
+      CallExpression(node) {
         if (
-          isRootCypress(node) &&
-          isActionUnsafeToChain(node, methods) &&
-          node.parent.type === 'MemberExpression'
+          isRootCypress(node)
+          && isActionUnsafeToChain(node, methods)
+          && node.parent.type === 'MemberExpression'
         ) {
           context.report({
             node,
@@ -108,15 +108,15 @@ module.exports = {
  */
 const isRootCypress = (node) => {
   if (
-    node.type !== 'CallExpression' ||
-    node.callee.type !== 'MemberExpression'
+    node.type !== 'CallExpression'
+    || node.callee.type !== 'MemberExpression'
   ) {
     return false
   }
 
   if (
-    node.callee.object.type === 'Identifier' &&
-    node.callee.object.name === 'cy'
+    node.callee.object.type === 'Identifier'
+    && node.callee.object.name === 'cy'
   ) {
     return true
   }
@@ -135,9 +135,9 @@ const isActionUnsafeToChain = (node, additionalMethods = []) => {
   ].join('|'))
 
   return (
-    node.callee &&
-    node.callee.property &&
-    node.callee.property.type === 'Identifier' &&
-    unsafeActionsRegex.test(node.callee.property.name)
+    node.callee
+    && node.callee.property
+    && node.callee.property.type === 'Identifier'
+    && unsafeActionsRegex.test(node.callee.property.name)
   )
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "eslint-plugin-cypress",
-  "version": "5.0.0",
+  "version": "5.0.1",
   "description": "An ESLint plugin for projects using Cypress",
   "main": "./lib/flat.js",
   "exports": {
@@ -29,20 +29,19 @@
     "globals": "^16.2.0"
   },
   "devDependencies": {
+    "@stylistic/eslint-plugin": "^4.4.0",
     "eslint": "^9.27.0",
     "eslint-plugin-eslint-plugin": "^6.4.0",
     "eslint-plugin-mocha": "^11.1.0",
     "husky": "^9.1.7",
-    "jest": "^29.7.0",
-    "semantic-release": "24.2.5"
+    "semantic-release": "24.2.5",
+    "vitest": "^3.1.4"
   },
   "scripts": {
     "lint": "eslint",
     "lint-fix": "eslint --fix",
     "semantic-release": "semantic-release",
-    "start": "npm run test-watch",
-    "test": "jest",
-    "test-watch": "jest --watchAll",
+    "test": "vitest run",
     "prepare": "husky"
   }
 }

package/tests/lib/rules/no-chained-get.js

@@ -2,30 +2,30 @@
  * @fileoverview disallow chain of `cy.get()` calls
  * @author benoit
  */
-"use strict";
+'use strict'
 
-//------------------------------------------------------------------------------
+// ------------------------------------------------------------------------------
 // Requirements
-//------------------------------------------------------------------------------
+// ------------------------------------------------------------------------------
 
-const rule = require("../../../lib/rules/no-chained-get"),
-  RuleTester = require("eslint").RuleTester;
+const rule = require('../../../lib/rules/no-chained-get'),
+  RuleTester = require('eslint').RuleTester
 
-//------------------------------------------------------------------------------
+// ------------------------------------------------------------------------------
 // Tests
-//------------------------------------------------------------------------------
+// ------------------------------------------------------------------------------
 
-const ruleTester = new RuleTester();
-ruleTester.run("no-chained-get", rule, {
+const ruleTester = new RuleTester()
+ruleTester.run('no-chained-get', rule, {
   valid: [
-    { code: "cy.get('div')" },
-    { code: "cy.get('.div').find().get()" },
-    { code: "cy.get('input').should('be.disabled')" },
+    { code: 'cy.get(\'div\')' },
+    { code: 'cy.get(\'.div\').find().get()' },
+    { code: 'cy.get(\'input\').should(\'be.disabled\')' },
   ],
   invalid: [
     {
-      code: "cy.get('div').get('div')",
-      errors: [{ messageId: "unexpected" }],
+      code: 'cy.get(\'div\').get(\'div\')',
+      errors: [{ messageId: 'unexpected' }],
     },
   ],
-});
+})

package/tests/lib/rules/no-debug.js

@@ -10,14 +10,14 @@ const errors = [{ messageId: 'unexpected' }]
 ruleTester.run('no-debug', rule, {
 
   valid: [
-    { code: `debug()` },
-    { code: `cy.get('button').dblclick()` },
+    { code: 'debug()' },
+    { code: 'cy.get(\'button\').dblclick()' },
   ],
 
   invalid: [
-    { code: `cy.debug()`, errors },
-    { code: `cy.debug({ log: false })`, errors },
-    { code: `cy.get('button').debug()`, errors },
-    { code: `cy.get('a').should('have.attr', 'href').and('match', /dashboard/).debug()`, errors }
+    { code: 'cy.debug()', errors },
+    { code: 'cy.debug({ log: false })', errors },
+    { code: 'cy.get(\'button\').debug()', errors },
+    { code: 'cy.get(\'a\').should(\'have.attr\', \'href\').and(\'match\', /dashboard/).debug()', errors },
   ],
 })

package/tests/lib/rules/no-force.js

@@ -10,30 +10,30 @@ const errors = [{ messageId: 'unexpected' }]
 ruleTester.run('no-force', rule, {
 
   valid: [
-    { code: `cy.get('button').click()` },
-    { code: `cy.get('button').click({multiple: true})` },
-    { code: `cy.get('button').dblclick()` },
-    { code: `cy.get('input').type('somth')` },
-    { code: `cy.get('input').type('somth', {anyoption: true})` },
-    { code: `cy.get('input').trigger('click', {anyoption: true})` },
-    { code: `cy.get('input').rightclick({anyoption: true})` },
-    { code: `cy.get('input').check()` },
-    { code: `cy.get('input').select()` },
-    { code: `cy.get('input').focus()` },
-    { code: `cy.document().trigger("keydown", { ...event })` },
+    { code: 'cy.get(\'button\').click()' },
+    { code: 'cy.get(\'button\').click({multiple: true})' },
+    { code: 'cy.get(\'button\').dblclick()' },
+    { code: 'cy.get(\'input\').type(\'somth\')' },
+    { code: 'cy.get(\'input\').type(\'somth\', {anyoption: true})' },
+    { code: 'cy.get(\'input\').trigger(\'click\', {anyoption: true})' },
+    { code: 'cy.get(\'input\').rightclick({anyoption: true})' },
+    { code: 'cy.get(\'input\').check()' },
+    { code: 'cy.get(\'input\').select()' },
+    { code: 'cy.get(\'input\').focus()' },
+    { code: 'cy.document().trigger("keydown", { ...event })' },
   ],
 
   invalid: [
-    { code: `cy.get('button').click({force: true})`, errors },
-    { code: `cy.get('button').dblclick({force: true})`, errors },
-    { code: `cy.get('input').type('somth', {force: true})`, errors },
-    { code: `cy.get('div').find('.foo').type('somth', {force: true})`, errors },
-    { code: `cy.get('div').find('.foo').find('.bar').click({force: true})`, errors },
-    { code: `cy.get('div').find('.foo').find('.bar').trigger('change', {force: true})`, errors },
-    { code: `cy.get('input').trigger('click', {force: true})`, errors },
-    { code: `cy.get('input').rightclick({force: true})`, errors },
-    { code: `cy.get('input').check({force: true})`, errors },
-    { code: `cy.get('input').select({force: true})`, errors },
-    { code: `cy.get('input').focus({force: true})`, errors },
+    { code: 'cy.get(\'button\').click({force: true})', errors },
+    { code: 'cy.get(\'button\').dblclick({force: true})', errors },
+    { code: 'cy.get(\'input\').type(\'somth\', {force: true})', errors },
+    { code: 'cy.get(\'div\').find(\'.foo\').type(\'somth\', {force: true})', errors },
+    { code: 'cy.get(\'div\').find(\'.foo\').find(\'.bar\').click({force: true})', errors },
+    { code: 'cy.get(\'div\').find(\'.foo\').find(\'.bar\').trigger(\'change\', {force: true})', errors },
+    { code: 'cy.get(\'input\').trigger(\'click\', {force: true})', errors },
+    { code: 'cy.get(\'input\').rightclick({force: true})', errors },
+    { code: 'cy.get(\'input\').check({force: true})', errors },
+    { code: 'cy.get(\'input\').select({force: true})', errors },
+    { code: 'cy.get(\'input\').focus({force: true})', errors },
   ],
 })

package/tests/lib/rules/no-pause.js

@@ -10,14 +10,14 @@ const errors = [{ messageId: 'unexpected' }]
 ruleTester.run('no-pause', rule, {
 
   valid: [
-    { code: `pause()` },
-    { code: `cy.get('button').dblclick()` },
+    { code: 'pause()' },
+    { code: 'cy.get(\'button\').dblclick()' },
   ],
 
   invalid: [
-    { code: `cy.pause()`, errors },
-    { code: `cy.pause({ log: false })`, errors },
-    { code: `cy.get('button').pause()`, errors },
-    { code: `cy.get('a').should('have.attr', 'href').and('match', /dashboard/).pause()`, errors }
+    { code: 'cy.pause()', errors },
+    { code: 'cy.pause({ log: false })', errors },
+    { code: 'cy.get(\'button\').pause()', errors },
+    { code: 'cy.get(\'a\').should(\'have.attr\', \'href\').and(\'match\', /dashboard/).pause()', errors },
   ],
 })

package/tests/lib/rules/no-xpath.js

@@ -1,30 +1,30 @@
-"use strict";
+'use strict'
 
-//------------------------------------------------------------------------------
+// ------------------------------------------------------------------------------
 // Requirements
-//------------------------------------------------------------------------------
+// ------------------------------------------------------------------------------
 
-const rule = require("../../../lib/rules/no-xpath"),
-  RuleTester = require("eslint").RuleTester;
+const rule = require('../../../lib/rules/no-xpath'),
+  RuleTester = require('eslint').RuleTester
 
-//------------------------------------------------------------------------------
+// ------------------------------------------------------------------------------
 // Tests
-//------------------------------------------------------------------------------
+// ------------------------------------------------------------------------------
 
-const ruleTester = new RuleTester();
-ruleTester.run("no-xpath", rule, {
+const ruleTester = new RuleTester()
+ruleTester.run('no-xpath', rule, {
   valid: [
     { code: 'cy.get("button").click({force: true})' },
   ],
 
   invalid: [
     {
-      code: "cy.xpath('//div[@class=\"container\"]/p[1]').click()",
-      errors: [{ messageId: "unexpected" }],
+      code: 'cy.xpath(\'//div[@class="container"]/p[1]\').click()',
+      errors: [{ messageId: 'unexpected' }],
     },
     {
-      code: "cy.xpath('//p[1]').should('exist')",
-      errors: [{ messageId: "unexpected" }]
-    }
+      code: 'cy.xpath(\'//p[1]\').should(\'exist\')',
+      errors: [{ messageId: 'unexpected' }],
+    },
   ],
-});
+})

package/tests/lib/rules/unsafe-to-chain-command.js

@@ -10,7 +10,7 @@ const errors = [{ messageId: 'unexpected' }]
 ruleTester.run('action-ends-chain', rule, {
   valid: [
     {
-      code: 'cy.get("new-todo").type("todo A{enter}"); cy.get("new-todo").type("todo B{enter}"); cy.get("new-todo").should("have.class", "active");'
+      code: 'cy.get("new-todo").type("todo A{enter}"); cy.get("new-todo").type("todo B{enter}"); cy.get("new-todo").should("have.class", "active");',
     },
     { code: 'cy.focused().should("be.visible");' },
     { code: 'cy.submitBtn().click();' },

package/vitest.config.mts

@@ -0,0 +1,8 @@
+import { defineConfig } from 'vitest/config'
+
+export default defineConfig({
+  test: {
+    include: ['**/tests/**/*.[jt]s?(x)'],
+    globals: true
+  },
+})

package/jest.config.js

@@ -1,6 +0,0 @@
-const { defaults } = require('jest-config')
-
-module.exports = {
-  testMatch: ['**/tests/**/*.[jt]s?(x)'],
-  testPathIgnorePatterns: [...defaults.testPathIgnorePatterns, '.history'],
-}