eslint-plugin-cypress 4.1.0 → 4.2.0

package/.node-version

@@ -1 +1 @@
-20.12.2
+22.11.0

package/.nvmrc

@@ -1 +1 @@
-20.12.2
+22.11.0

package/README.md

@@ -54,6 +54,7 @@ These rules enforce some of the [best practices recommended for using Cypress](h
 | [no-force](docs/rules/no-force.md)                                       | disallow using `force: true` with action commands          |    |
 | [no-pause](docs/rules/no-pause.md)                                       | disallow using `cy.pause()` calls                          |    |
 | [no-unnecessary-waiting](docs/rules/no-unnecessary-waiting.md)           | disallow waiting for arbitrary time periods                | ✅  |
+| [no-xpath](docs/rules/no-xpath.md)                                       | disallow using `cy.xpath()` calls                          |    |
 | [require-data-selectors](docs/rules/require-data-selectors.md)           | require `data-*` attribute selectors                       |    |
 | [unsafe-to-chain-command](docs/rules/unsafe-to-chain-command.md)         | disallow actions within chains                             | ✅  |
 

package/docs/rules/no-xpath.md

@@ -0,0 +1,25 @@
+# Disallow using `cy.xpath()` calls (`cypress/no-xpath`)
+
+<!-- end auto-generated rule header -->
+
+This rule disallows the usage of `cy.xpath()` for selecting elements.
+
+## Rule Details
+
+Examples of **incorrect** code for this rule:
+
+```js
+cy.xpath('//div[@class=\"container\"]').click()
+```
+
+Examples of **correct** code for this rule:
+
+```js
+cy.get('[data-cy="container"]').click();
+```
+
+## Further Reading
+
+Both `@cypress/xpath` and `cypress-xpath` are deprecated.
+
+See [the Cypress Best Practices guide](https://docs.cypress.io/guides/references/best-practices.html#Selecting-Elements).

package/eslint.config.mjs

@@ -1,13 +1,11 @@
 import globals from 'globals'
 import pluginJs from '@eslint/js'
 import eslintPlugin from 'eslint-plugin-eslint-plugin'
-import nodePlugin from 'eslint-plugin-n'
 import mochaPlugin from 'eslint-plugin-mocha'
 
 export default [
   pluginJs.configs.recommended,
   eslintPlugin.configs['flat/recommended'],
-  nodePlugin.configs['flat/recommended-script'],
   mochaPlugin.configs.flat.recommended,
   {
     languageOptions: {
@@ -20,8 +18,6 @@ export default [
       'eslint-plugin/require-meta-docs-description': 'error',
       'eslint-plugin/meta-property-ordering': 'error',
       'eslint-plugin/test-case-property-ordering': 'error',
-      'n/no-extraneous-require':
-        ['error', { 'allowModules': ['jest-config'] }],
       'mocha/no-mocha-arrows': 'off',
       'mocha/no-setup-in-describe': 'off'
     }

package/legacy.js

@@ -12,6 +12,7 @@ module.exports = {
     'no-force': require('./lib/rules/no-force'),
     'no-pause': require('./lib/rules/no-pause'),
     'no-debug': require('./lib/rules/no-debug'),
+    'no-xpath': require('./lib/rules/no-xpath'),
   },
   configs: {
     recommended: require('./lib/config/recommended'),

package/lib/flat.js

@@ -15,6 +15,7 @@ const plugin = {
     'no-force': require('./rules/no-force'),
     'no-pause': require('./rules/no-pause'),
     'no-debug': require('./rules/no-debug'),
+    'no-xpath': require('./rules/no-xpath'),
   },
 }
 

package/lib/rules/no-xpath.js

@@ -0,0 +1,35 @@
+'use strict'
+
+module.exports = {
+  meta: {
+    type: 'suggestion',
+    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'
+    },
+    fixable: null, // Or `code` or `whitespace`
+    schema: [], // Add a schema if the rule has options
+    messages: {
+      unexpected: 'Avoid using cy.xpath command',
+    },
+  },
+
+  create (context) {
+    return {
+      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'
+}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "eslint-plugin-cypress",
-  "version": "4.1.0",
+  "version": "4.2.0",
   "description": "An ESLint plugin for projects using Cypress",
   "main": "legacy.js",
   "exports": {
@@ -32,7 +32,6 @@
     "eslint": "^9.13.0",
     "eslint-plugin-eslint-plugin": "^6.3.1",
     "eslint-plugin-mocha": "^10.5.0",
-    "eslint-plugin-n": "^17.11.1",
     "husky": "^9.1.6",
     "jest": "^29.7.0",
     "semantic-release": "24.2.0"

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

@@ -0,0 +1,30 @@
+"use strict";
+
+//------------------------------------------------------------------------------
+// Requirements
+//------------------------------------------------------------------------------
+
+const rule = require("../../../lib/rules/no-xpath"),
+  RuleTester = require("eslint").RuleTester;
+
+//------------------------------------------------------------------------------
+// Tests
+//------------------------------------------------------------------------------
+
+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('//p[1]').should('exist')",
+      errors: [{ messageId: "unexpected" }]
+    }
+  ],
+});