cypress-documentation icon indicating copy to clipboard operation
cypress-documentation copied to clipboard

Remove mentions to outdated plugin: cypress-plugin-snapshots

Open dwilches opened this issue 2 years ago • 3 comments

In this and this pages there are mentions and instructions to install a plugin that has not worked with Cypress since version 4.5.0: cypress-plugin-snapshots.

The first page I mentioned even has instructions to install this plugin:

  // run 'npm i cypress-plugin-snapshots -S'
  // capture the element screenshot and
  // compare to the baseline image

Which if attempted with any recent version of Cypress (> 2020) will give this error:

npm ERR! code ERESOLVE
npm ERR! ERESOLVE unable to resolve dependency tree
npm ERR! 
npm ERR! While resolving: [email protected]
npm ERR! Found: [email protected]
npm ERR! node_modules/cypress
npm ERR!   dev cypress@"^12.6.0" from the root project
npm ERR! 
npm ERR! Could not resolve dependency:
npm ERR! peer cypress@"^4.5.0" from [email protected]
npm ERR! node_modules/cypress-plugin-snapshots
npm ERR!   cypress-plugin-snapshots@"*" from the root project
npm ERR! 
npm ERR! Fix the upstream dependency conflict, or retry
npm ERR! this command with --force, or --legacy-peer-deps
npm ERR! to accept an incorrect (and potentially broken) dependency resolution.

Because of this line in that plugin's package.json:

  "peerDependencies": {
    "cypress": "^4.5.0"
  },

I think there is no purpose on having instructions for a command that won't work if attempted. This Issue is to ask if it can be removed so the next person can focus on visual testing plugins that are maintained.

dwilches avatar Feb 24 '23 18:02 dwilches

+1 to this.

The same issue is raised here too: https://github.com/cypress-io/cypress-documentation/issues/5073.

rafaelbeckel avatar Sep 22 '23 03:09 rafaelbeckel

  • See also https://github.com/cypress-io/cypress-documentation/issues/5293

MikeMcC399 avatar Sep 22 '23 09:09 MikeMcC399

@dwilches thank you for reporting this bug. If you would like to make a PR we would gladly accept it.

nagash77 avatar Sep 22 '23 12:09 nagash77