eslint-plugin-cypress 4.2.1 → 4.3.0

package/CONTRIBUTING.md

@@ -48,3 +48,29 @@ This plugin uses the ESLint [eslint-doc-generator](https://www.npmjs.com/package
 
 * The directory [tests-legacy](https://github.com/cypress-io/eslint-plugin-cypress/tree/master/tests-legacy) contains tests which are compatible with the legacy [ESLint v8 RuleTester](https://eslint.org/docs/v8.x/integrate/nodejs-api#ruletester) utility. It is not expected to add new rules to this set of tests.
 * The directory [tests](https://github.com/cypress-io/eslint-plugin-cypress/tree/master/tests) is for tests compatible with the current [ESLint RuleTester](https://eslint.org/docs/latest/integrate/nodejs-api#ruletester).
+
+## Merging pull requests
+
+This information is for Cypress.io Members or Collaborators who merge pull requests:
+
+1. When merging a pull request, use the [Squash and merge](https://docs.github.com/en/repositories/configuring-branches-and-merges-in-your-repository/configuring-pull-request-merges/about-merge-methods-on-github#squashing-your-merge-commits) option to squash all commits into one.
+1. Make sure the commit subject and body follow [semantic commit convention](https://semantic-release.gitbook.io/semantic-release/#commit-message-format), for instance:
+
+   ```text
+   feat: added new parameter
+   fix: fixed a bug
+   ```
+
+   If you need to bump the major version, mark it as breaking change in the body of the commit's message like:
+
+   ```text
+   fix: upgrade dependency X
+
+   BREAKING CHANGE: requires minimum Node.js 20 to run
+   ```
+
+1. New versions of this module will be released automatically by the CI pipeline when any PR with a triggering commit message is merged to the `master` branch: see the `release` job of [circle.yml](circle.yml).
+This will create a new [GitHub release](https://github.com/cypress-io/eslint-plugin-cypress/releases) and publish it to [eslint-plugin-cypress](https://www.npmjs.com/package/eslint-plugin-cypress) on the [npm registry](https://docs.npmjs.com/about-the-public-npm-registry).
+1. The module's CI is configured to use the [default Angular release rules](https://github.com/semantic-release/commit-analyzer/blob/master/lib/default-release-rules.js).
+This means that only `feat:`, `fix:` and `perf:` trigger a new release.
+Other Angular commit types listed in the [Angular commit message guidelines](https://github.com/angular/angular/blob/main/contributing-docs/commit-message-guidelines.md) can be used for documentation purposes, however they are ignored by the currently configured release process.

package/README.md

@@ -50,6 +50,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-chained-get](docs/rules/no-chained-get.md)                           | disallow chain of `cy.get()` calls                         |    |
 | [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                          |    |

package/docs/rules/no-chained-get.md

@@ -0,0 +1,19 @@
+# Disallow chain of `cy.get()` calls (`cypress/no-chained-get`)
+
+<!-- end auto-generated rule header -->
+This rule disallows the usage of chained `.get()` calls as `cy.get()` always starts its search from the cy.root element.
+
+## Rule Details
+
+Examples of **incorrect** code for this rule:
+
+```js
+cy.get('parent').get('child')
+```
+
+Examples of **correct** code for this rule:
+
+```js
+cy.get('parent')
+  .find('child')
+```

package/legacy.js

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

package/lib/flat.js

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

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

@@ -0,0 +1,87 @@
+'use strict';
+
+//------------------------------------------------------------------------------
+// Rule Definition
+//------------------------------------------------------------------------------
+
+/** @type {import('eslint').Rule.RuleModule} */
+module.exports = {
+  meta: {
+    type: 'problem',
+    docs: {
+      description: 'disallow chain of `cy.get()` calls',
+      recommended: false,
+      url: 'https://github.com/cypress-io/eslint-plugin-cypress/blob/master/docs/rules/no-chained-get.md',
+    },
+    fixable: null,
+    schema: [],
+    messages: {
+      unexpected: 'Avoid chaining multiple cy.get() calls',
+    },
+  },
+
+  create(context) {
+    const isRootCypress = (node) => {
+      if (
+        node.type !== 'CallExpression' ||
+        node.callee.type !== 'MemberExpression'
+      ) {
+        return false;
+      }
+
+      if (
+        node.callee.object.type === 'Identifier' &&
+        node.callee.object.name === 'cy'
+      ) {
+        return true;
+      }
+
+      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';
+
+      if (!isGetCall) {
+        return false;
+      }
+
+      const obj = node.callee.object;
+
+      if (obj.type === 'CallExpression') {
+        const objCallee = obj.callee;
+
+        if (
+          objCallee &&
+          objCallee.type === 'MemberExpression' &&
+          objCallee.property &&
+          objCallee.property.type === 'Identifier' &&
+          objCallee.property.name === 'get'
+        ) {
+          return true;
+        }
+
+        return hasChainedGet(obj);
+      }
+
+      return false;
+    };
+
+    return {
+      CallExpression(node) {
+        if (isRootCypress(node) && hasChainedGet(node)) {
+          context.report({
+            node,
+            messageId: 'unexpected',
+          });
+        }
+      },
+    };
+  },
+};

package/package.json

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

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

@@ -0,0 +1,31 @@
+/**
+ * @fileoverview disallow chain of `cy.get()` calls
+ * @author benoit
+ */
+"use strict";
+
+//------------------------------------------------------------------------------
+// Requirements
+//------------------------------------------------------------------------------
+
+const rule = require("../../../lib/rules/no-chained-get"),
+  RuleTester = require("eslint").RuleTester;
+
+//------------------------------------------------------------------------------
+// Tests
+//------------------------------------------------------------------------------
+
+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')" },
+  ],
+  invalid: [
+    {
+      code: "cy.get('div').get('div')",
+      errors: [{ messageId: "unexpected" }],
+    },
+  ],
+});