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

feat: (origin) handle waiting for aliased intercepts #21579

Merged
merged 8 commits into from
May 24, 2022

Conversation

mschile
Copy link
Contributor

@mschile mschile commented May 20, 2022

User facing changelog

  • Updated cy.origin to support waiting for aliased intercepted requests.

Additional details

  • Aliased intercepted requests are stored in Cypress.state within the primary domain. As such, when waiting on an aliased intercepted request within cy.origin, a message is sent to the primary domain to handle the waiting. The response or error is then sent back to the the secondary domain.

How has the user experience changed?

Before:
Screen Shot 2022-05-19 at 10 45 34 PM

After:

Waiting on intercepted request:
Screen Shot 2022-05-19 at 10 30 42 PM

Error when request not called:
Screen Shot 2022-05-19 at 10 36 48 PM

PR Tasks

  • Have tests been added/updated?
  • Has the original issue (or this PR, if no issue exists) been tagged with a release in ZenHub? (user-facing changes only)
  • [na] Has a PR for user-facing changes been opened in cypress-documentation?
  • [na] Have API changes been updated in the type definitions?
  • [na] Have new configuration options been added to the cypress.schema.json?

@mschile mschile requested a review from a team as a code owner May 20, 2022 04:51
@mschile mschile requested review from jennifer-shehane and removed request for a team May 20, 2022 04:51
@cypress-bot
Copy link
Contributor

cypress-bot bot commented May 20, 2022

Thanks for taking the time to open a PR!

@mschile mschile changed the title feat (origin): handle waiting for aliased intercepts feat: (origin) handle waiting for aliased intercepts May 20, 2022
@cypress
Copy link

cypress bot commented May 20, 2022



Test summary

4437 0 54 0Flakiness 0


Run details

Project cypress
Status Passed
Commit 22dd678
Started May 23, 2022 9:33 PM
Ended May 23, 2022 9:45 PM
Duration 11:58 💡
OS Linux Debian - 10.11
Browser Firefox 98

View run in Cypress Dashboard ➡️


This comment has been generated by cypress-bot as a result of this project's GitHub integration settings. You can manage this integration in this project's settings in the Cypress Dashboard

packages/driver/src/cy/commands/waiting.ts Outdated Show resolved Hide resolved
packages/driver/src/cy/commands/waiting.ts Outdated Show resolved Hide resolved
packages/driver/src/cy/commands/waiting.ts Outdated Show resolved Hide resolved
@mschile mschile requested a review from AtofStryker May 23, 2022 20:56
@mschile mschile merged commit 53030b7 into develop May 24, 2022
@mschile mschile deleted the issue-21317_origin_alias_waiting branch May 24, 2022 15:02
tgriesser added a commit that referenced this pull request May 25, 2022
…pack

* develop:
  test(launchpad): skip failure due to recaptcha on windows, enable more windows jobs (#21620)
  docs: add "Steps to test" to PR template (#21606)
  feat: (origin) handle waiting for aliased intercepts (#21579)
  fix: handle git watch error (#21600)
  fix: Successfully create new specs files that do not have a known extension (#21593)
  chore: release 9.7.0
  fix: do not allow experimentalSessionAndOrigin to be available in CT … (#21588)
  chore: sort the frameworks select dropdown in a more logical way (#21553)
  Trigger Build
  fix: disable ct testing when run via binary (#21540)
  chore: Update Chrome (beta) to 102.0.5005.61 (#21556)
  chore: release @cypress/react-v5.12.5
  fix: add support for Next.js v12.1.6 (#21516)
  chore: release @cypress/schematic-v1.7.0
  feat(cypress/schematic): add headed option and other fixes (#21415)
  chore: release @cypress/vite-dev-server-v2.2.3
  fix: handle specs with white space in vite-dev-server (#21386)
  Empty commit
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Handle Aliases in cy.origin callback
4 participants