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

fix: prerequest correlation for various retried and cached requests #27771

Merged

Conversation

ryanthemanuel
Copy link
Collaborator

Additional details

Fix prerequest correlation and response body capture for various retried and cached requests.

Steps to test

These scenarios are somewhat hard to reproduce, but I was able to stumble upon them through running through the suite of tests in the cypress dashboard.

How has the user experience changed?

Intercepts will now work more consistently.

PR Tasks

@cypress
Copy link

cypress bot commented Sep 8, 2023

5 flaky tests on run #50914 ↗︎

0 4475 942 0 Flakiness 5

Details:

PR comments
Project: cypress Commit: c7f795fd98
Status: Passed Duration: 13:34 💡
Started: Sep 11, 2023 1:49 AM Ended: Sep 11, 2023 2:02 AM
Flakiness  commands/net_stubbing.cy.ts • 1 flaky test • 5x-driver-webkit

View Output Video

Test Artifacts
... > with `times` > only uses each handler N times Output
Flakiness  commands/waiting.cy.js • 1 flaky test • 5x-driver-webkit

View Output Video

Test Artifacts
... > errors > throws waiting for the 3rd response Output
Flakiness  cypress/cypress.cy.js • 3 flaky tests • 5x-driver-webkit

View Output Video

Test Artifacts
... > correctly returns currentRetry Output
... > correctly returns currentRetry Output
... > correctly returns currentRetry Output

Review all test suite changes for PR #27771 ↗︎

@ryanthemanuel ryanthemanuel merged commit 70248ab into develop Sep 11, 2023
@ryanthemanuel ryanthemanuel deleted the ryanm/fix/various-fixes-for-prerequest-correlation branch September 11, 2023 02:41
@cypress-bot
Copy link
Contributor

cypress-bot bot commented Sep 12, 2023

Released in 13.2.0.

This comment thread has been locked. If you are still experiencing this issue after upgrading to
Cypress v13.2.0, please open a new issue.

@cypress-bot cypress-bot bot locked as resolved and limited conversation to collaborators Sep 12, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants