Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Cypress] Replace all cy.server() and cy.route() by cy.intercept() #13345

Closed
1 task done
nassimerrahoui opened this issue Dec 21, 2020 · 3 comments · Fixed by #13414
Closed
1 task done

[Cypress] Replace all cy.server() and cy.route() by cy.intercept() #13345

nassimerrahoui opened this issue Dec 21, 2020 · 3 comments · Fixed by #13414
Labels
area: enhancement 🔧 $$ bug-bounty $$ https://www.jhipster.tech/bug-bounties/ theme: tests $100 https://www.jhipster.tech/bug-bounties/
Milestone

Comments

@nassimerrahoui
Copy link
Contributor

nassimerrahoui commented Dec 21, 2020

Overview of the feature request

Replace all cy.server() and cy.route() by cy.intercept().
(see: https://docs.cypress.io/api/commands/intercept.html#Comparison-to-cy-route)

Motivation for or Use Case

In a future release, cypress support for cy.server() and cy.route() will be moved to a plugin.
They encourage to use cy.intercept() instead.
(see: https://docs.cypress.io/guides/references/changelog.html#6-0-0)

Related issues or PR

N/A

  • Checking this box is mandatory (this is just to show you read everything)
@qmonmert
Copy link
Contributor

qmonmert commented Jan 7, 2021

@pascalgrimaud
Copy link
Member

@qmonmert : approved

@qmonmert
Copy link
Contributor

qmonmert commented Jan 7, 2021

thanks @pascalgrimaud

@pascalgrimaud pascalgrimaud added this to the 7.0.0-beta.1 milestone Jan 16, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: enhancement 🔧 $$ bug-bounty $$ https://www.jhipster.tech/bug-bounties/ theme: tests $100 https://www.jhipster.tech/bug-bounties/
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants