-
Notifications
You must be signed in to change notification settings - Fork 3.2k
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
chore: add mountVersion to props passed to protocol setup #27645
Conversation
35 flaky tests on run #50399 ↗︎
Details:
e2e/origin/config_env.cy.ts • 1 flaky test • 5x-driver-firefox
e2e/origin/commands/assertions.cy.ts • 1 flaky test • 5x-driver-firefox
cypress/cypress.cy.js • 3 flaky tests • 5x-driver-firefox
specs_list_latest_runs.cy.ts • 1 flaky test • app-e2e
commands/net_stubbing.cy.ts • 2 flaky tests • 5x-driver-chrome:beta
The first 5 flaky specs are shown, see all 22 specs in Cypress Cloud. This comment has been generated by cypress-bot as a result of this project's GitHub integration settings. |
@@ -2890,88 +2890,6 @@ exports['e2e record capture-protocol enabled passing retrieves the capture proto | |||
Recorded Run: https://dashboard.cypress.io/projects/cjvoj7/runs/12 | |||
|
|||
|
|||
` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Was this one expected to be removed? I didn't see any changes to record_spec.js.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't either, but it also doesn't match any tests in record_spec. I'm guessing it was an obsolete snapshot that didn't get removed on a previous merge.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sounds good. Good to clean it up then.
Released in This comment thread has been locked. If you are still experiencing this issue after upgrading to |
Additional details
In order to add mountVersion to metadata saved to the protocol db, we're passing it in to the AppCaptureProtocol constructor.
Steps to test
How has the user experience changed?
PR Tasks
cypress-documentation
?type definitions
?