eslint-plugin-cypress 3.3.0 → 3.5.0

package/README.md

@@ -17,6 +17,10 @@ or
 yarn add eslint-plugin-cypress --dev
 ```
 
+## Deprecations
+
+The use of ESLint `v7` with `eslint-plugin-cypress` is deprecated and support will be removed in a future version of this plugin. ESLint `v7` reached end-of-life on Apr 9, 2022 and users are encouraged to migrate to ESLint `v9`. See [ESLint Version Support](https://eslint.org/version-support/) for ESLint's current release lines.
+
 ## Usage
 
 If you are using ESLint `v7` or `v8`, then add an `.eslintrc.json` file to the root directory of your Cypress project with the contents shown below. You can continue to use this format with ESLint `v9` if you set the `ESLINT_USE_FLAT_CONFIG` environment variable to `false` (see [ESLint v9 > Configuration Files (Deprecated)](https://eslint.org/docs/latest/use/configure/configuration-files-deprecated).
@@ -42,7 +46,8 @@ You can add rules:
     "cypress/no-force": "warn",
     "cypress/no-async-tests": "error",
     "cypress/no-async-before": "error",
-    "cypress/no-pause": "error"
+    "cypress/no-pause": "error",
+    "cypress/no-debug": "error"
   }
 }
 ```
@@ -129,6 +134,7 @@ These rules enforce some of the [best practices recommended for using Cypress](h
 | [no-assigning-return-values](docs/rules/no-assigning-return-values.md)   | disallow assigning return values of `cy` calls             | ✅  |
 | [no-async-before](docs/rules/no-async-before.md)                         | disallow using `async`/`await` in Cypress `before` methods |    |
 | [no-async-tests](docs/rules/no-async-tests.md)                           | disallow using `async`/`await` in Cypress test cases       | ✅  |
+| [no-debug](docs/rules/no-debug.md)                                       | disallow using `cy.debug()` calls                          |    |
 | [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                | ✅  |

package/docs/rules/no-debug.md

@@ -0,0 +1,19 @@
+# Disallow using `cy.debug()` calls (`cypress/no-debug`)
+
+<!-- end auto-generated rule header -->
+It is recommended to remove any [cy.debug](https://on.cypress.io/debug) commands before committing specs to avoid other developers getting unexpected results.
+
+## Rule Details
+
+Examples of **incorrect** code for this rule:
+
+```js
+cy.debug();
+cy.get('selector').debug();
+```
+
+Examples of **correct** code for this rule:
+
+```js
+cy.get('selector')
+```

package/legacy.js

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

package/lib/flat.js

@@ -14,6 +14,7 @@ const plugin = {
     'require-data-selectors': require('./rules/require-data-selectors'),
     'no-force': require('./rules/no-force'),
     'no-pause': require('./rules/no-pause'),
+    'no-debug': require('./rules/no-debug'),
   },
 }
 
@@ -33,6 +34,7 @@ const commonLanguageOptions = {
 
 Object.assign(plugin.configs, {
   globals: {
+    name: 'cypress/globals',
     plugins: {
       cypress: plugin
     },
@@ -46,6 +48,7 @@ Object.assign(plugin.configs, {
 
 Object.assign(plugin.configs, {
   recommended: {
+    name: 'cypress/recommended',
     plugins: {
       cypress: plugin
     },

package/lib/rules/no-debug.js

@@ -0,0 +1,61 @@
+'use strict'
+
+//------------------------------------------------------------------------------
+// Rule Definition
+//------------------------------------------------------------------------------
+
+module.exports = {
+  meta: {
+    type: 'suggestion',
+    docs: {
+      description: 'disallow using `cy.debug()` calls',
+      category: 'Possible Errors',
+      recommended: false,
+      url: 'https://github.com/cypress-io/eslint-plugin-cypress/blob/master/docs/rules/no-debug.md',
+    },
+    fixable: null, // or "code" or "whitespace"
+    schema: [],
+    messages: {
+      unexpected: 'Do not use cy.debug command',
+    },
+  },
+
+  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 isCypressCall (node) {
+      if (!node.callee || node.callee.type !== 'MemberExpression') {
+        return false;
+      }
+      if (node.callee.object.type === 'Identifier' && node.callee.object.name === 'cy') {
+        return true;
+      }
+      return isCypressCall(node.callee.object);
+    }
+    
+    //----------------------------------------------------------------------
+    // Public
+    //----------------------------------------------------------------------
+
+    return {
+
+      CallExpression (node) {
+        if (isCypressCall(node) && isCallingDebug(node)) {
+          context.report({ node, messageId: 'unexpected' })
+        }
+      },
+
+    }
+  },
+}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "eslint-plugin-cypress",
-  "version": "3.3.0",
+  "version": "3.5.0",
   "description": "An ESLint plugin for projects using Cypress",
   "main": "legacy.js",
   "exports": {

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

@@ -0,0 +1,23 @@
+'use strict'
+
+const rule = require('../../../lib/rules/no-debug')
+const RuleTester = require('eslint').RuleTester
+
+const ruleTester = new RuleTester()
+
+const errors = [{ messageId: 'unexpected' }]
+
+ruleTester.run('no-debug', rule, {
+
+  valid: [
+    { 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 }
+  ],
+})