Upgrade Cypress to version 13

iamdharmesh commented 1 year ago

Is your enhancement related to a problem? Please describe.

Cypress 13 was recently released, and Cypress 12 contains some breaking changes as listed below. We need to update our tests to make them work with the latest version and fix any issues that may arise.

[ ] Upgrade cypress to 13.0.0 (or latest version at the time of working on this)
[ ] Upgrade @10up/cypress-wp-utils to 0.2.0
[ ] Replace legacy API usage such as Cookies.defaults with the new replacements.

Breaking changes we need to be aware of

Cypress dropped support for Node.js 12
In Cypress 12, we enforce running tests in a clean browser context through test isolation.
The Cookies.defaults and Cookies.preserveOnce APIs have been removed. Use the new cy.session() command to preserve cookies between tests.

Designs

No response

Describe alternatives you've considered

No response

Code of Conduct

[X] I agree to follow this project's Code of Conduct

dkotter commented 1 year ago

Note that our E2E tests are failing right now due to the need to upgrade Cypress.

Our @10up/cypress-wp-utils package is used here but we don't load a versioned release, we load from the build branch. This loads the latest version which runs on a newer version of Cypress and conflicts with the version of Cypress we are using. So this is an ideal task for someone to pick up as it helps get our tests running properly again.

iamdharmesh commented 1 year ago

Tests are failing due to Cypress 13 upgrade in https://github.com/10up/safe-redirect-manager/pull/343, as e2e tests still using removed cookies API. Raised #349 to fix this.

10up / safe-redirect-manager