tyler36 / ddev-cypress

Cypress E2E testing for use with DDEV
Apache License 2.0
11 stars 7 forks source link

⏫ bump Cypress to `12.6.0` #20

Closed tyler36 closed 1 year ago

tyler36 commented 1 year ago

This bumps the version os Cypress to 12.3.0.

The latest version is currently 12.5.1 however, this seems to generate an error:

EACCES: permission denied, open '/root/.cache/Cypress/12.5.1/binary_state.json

This also occurs in 12.4.0 flavors, and 12.5.0.

This PR may help resolve #19.

Further testing should be done before merging.

rfay commented 1 year ago

Don't forget to update the README (which says mac not supported and arm64 not supported). I assume this will work fine on macOS amd64 already, but with this change should work on Apple Silicon as well.

tyler36 commented 1 year ago

Good catch

laetus007 commented 1 year ago

If this does support Silicon - doesn't the preinstall _actions on https://github.com/tyler36/ddev-cypress/blob/main/install.yaml also need to be updated?

rfay commented 1 year ago

@laetus007 you're certainly right... are you able to test on Apple Silicon?

laetus007 commented 1 year ago

For sure - I've forked here - https://github.com/laetus007/ddev-cypress. I am able to get the add-on installed. I have tried following all the instructions for running; however, I can't get past the verification step in cypress. It stalls out. I've added additional information in the issue - https://github.com/tyler36/ddev-cypress/issues/19

tyler36 commented 1 year ago

Just a found a thread that seems to suggest ongoing issues with Mac and timeouts:

https://github.com/cypress-io/cypress/issues/7497