Several advanced examples fail cypress/unsafe-to-chain-command lint rule

[MikeMcC399]

If the eslint rule cypress/unsafe-to-chain-command from eslint-plugin-cypress: Rules is enabled, then the following 6 examples in cypress/e2e/2-advanced-examples fail linting with the message:

"error 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 cypress/unsafe-to-chain-command"

"✖ 57 problems (51 errors, 0 warnings)"

• actions.cy.js
• cookies.cy.js
• misc.cy.js
• spies_stubs_clocks.cy.js
• storage.cy.js
• utilities.cy.js

Here is a portion of actions.cy.js which fails linting:

    cy.get('.action-email')
      .type('[email protected]').should('have.value', '[email protected]')

      // .type() with special character sequences
      .type('{leftarrow}{rightarrow}{uparrow}{downarrow}')
      .type('{del}{selectall}{backspace}')

      // .type() with key modifiers
      .type('{alt}{option}') //these are equivalent
      .type('{ctrl}{control}') //these are equivalent
      .type('{meta}{command}{cmd}') //these are equivalent
      .type('{shift}')

      // Delay each keypress by 0.1 sec
      .type('[email protected]', { delay: 100 })
      .should('have.value', '[email protected]') 

Expectation

Since the rule cypress/unsafe-to-chain-command is part of plugin:cypress/recommended rules according to eslint-plugin-cypress: Rules I would expect standard examples to pass this lint test.

Steps to reproduce

Add "cypress/unsafe-to-chain-command": "error" to rules of .eslintrc, similar to the following

  "rules": {
    "cypress/unsafe-to-chain-command": "error"
  }

Execute:

npm run lint

Reproducible example on https://github.com/MikeMcC399/cypress-example-kitchensink/tree/test/unsafe-to-chain

Environment

• [email protected]
• [email protected] (current latest)

Next steps

• Correct the examples

[mschile]

Hi @MikeMcC399 👋, thanks for logging this issue. I took a cursory look at the reported errors and the rule appears to be correct so we should do #2 and update the examples.

[MikeMcC399]

Hi @mschile

Thank you very much for qualifying this issue!

I can't provide a solution for this issue, so I would be happy for the core team to make the changes in the examples.

[MikeMcC399]

Unfortunately I'm unable to provide a PR to resolve this issue, which remains unaddressed.

Steps to reproduce

Comment out

cypress-example-kitchensink/.eslintrc

Line 18 in b49f532

"cypress/unsafe-to-chain-command": "off",

and then execute

npm run lint

shows

✖ 57 problems (57 errors, 0 warnings)

[enchorb]

+1

[jennifer-shehane]

@MikeMcC399 I'd be fine with an update to the example recipes to reflect our new conventions. We just never revisited this.

[MikeMcC399]

@jennifer-shehane

@MikeMcC399 I'd be fine with an update to the example recipes to reflect our new conventions. We just never revisited this.

This issue is still open because as I said previously I don't have the skills for this and nobody else has picked it up.

[MikeMcC399]

• Thanks to @rubysolo for addressing this issue in fix: lint errors #802 !

[cypress-app-bot]

🎉 This issue has been resolved in version 2.0.3 🎉

The release is available on:

• npm package (@latest dist-tag)
• GitHub release

Your semantic-release bot 📦🚀