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: add remote.active-protocols argument and set it to 2 to enable CDP #29720

Merged
merged 1 commit into from
Jun 21, 2024

Conversation

AtofStryker
Copy link
Contributor

@AtofStryker AtofStryker commented Jun 20, 2024

Additional details

This PR allows for Firefox 129 and up (Currently Firefox Nightly) to run with Cypress. CDP has been deprecated with Firefox 129, which is scheduled to release around early August. To handle this, we need to enable CDP via the 'remote.active-protocols': 2, browser argument

Steps to test

Download Firefox Nightly and make sure cypress can run (via open or run mode) with firefox as the target browser.

How has the user experience changed?

PR Tasks

@AtofStryker AtofStryker force-pushed the fix/firefox_nightly_129_launch branch from f358e10 to 5b0dc25 Compare June 20, 2024 16:54
@AtofStryker AtofStryker force-pushed the fix/firefox_nightly_129_launch branch from 5b0dc25 to 247403b Compare June 20, 2024 16:57
Copy link

cypress bot commented Jun 20, 2024

11 flaky tests on run #55858 ↗︎

0 29292 1328 0 Flakiness 11

Details:

fix: add remote.active-protocols argument and set it to 2 to enable CDP
Project: cypress Commit: 247403b0cf
Status: Passed Duration: 21:34 💡
Started: Jun 20, 2024 5:07 PM Ended: Jun 20, 2024 5:29 PM
Flakiness  open-mode.cy.ts • 1 flaky test • launchpad-e2e

View Output

Test Artifacts
Launchpad: Open Mode > opens an e2e project without a supportFile Test Replay Screenshots
Flakiness  commands/net_stubbing.cy.ts • 1 flaky test • 5x-driver-electron

View Output

Test Artifacts
... > can timeout when retrieving upstream response Test Replay
Flakiness  commands/querying/querying.cy.js • 1 flaky test • 5x-driver-chrome

View Output

Test Artifacts
... > throws when alias property isnt just a digit Test Replay
Flakiness  e2e/origin/commands/waiting.cy.ts • 1 flaky test • 5x-driver-chrome

View Output

Test Artifacts
... > throws when foo cannot resolve Test Replay
Flakiness  commands/querying/querying.cy.js • 1 flaky test • 5x-driver-chrome:beta

View Output

Test Artifacts
... > throws after timing out after a .wait() alias reference Test Replay

The first 5 flaky specs are shown, see all 9 specs in Cypress Cloud.

Review all test suite changes for PR #29720 ↗︎

Copy link
Member

@jennifer-shehane jennifer-shehane left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I tested in Firefox Nightly 129 and it now loads in Cypress.

@jennifer-shehane jennifer-shehane merged commit 8217fdc into develop Jun 21, 2024
82 of 84 checks passed
@jennifer-shehane jennifer-shehane deleted the fix/firefox_nightly_129_launch branch June 21, 2024 15:37
@cypress-bot
Copy link
Contributor

cypress-bot bot commented Jul 1, 2024

Released in 13.13.0.

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

@cypress-bot cypress-bot bot locked as resolved and limited conversation to collaborators Jul 1, 2024
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.

Handle deprecation of CDP in Firefox 129
3 participants