-
Notifications
You must be signed in to change notification settings - Fork 3.3k
misc: throw error when chrome 137 branded is used with @cypress/puppeteer
or --load-extension
#31709
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
Conversation
cypress
|
Project |
cypress
|
Branch Review |
misc/throw_error_on_extension_chrome_137
|
Run status |
|
Run duration | 09m 49s |
Commit |
|
Committer | Bill Glesias |
View all properties for this run ↗︎ |
Test results | |
---|---|
|
0
|
|
0
|
|
694
|
|
0
|
|
130
|
View all changes introduced in this branch ↗︎ |
748fcdd
to
17d43a6
Compare
So, if I'm understanding this properly..... Cypress will no longer support Google Chrome (in headed mode) and just error out and not actually run the tests? |
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.
Looks good outside of a suggestion on making the changelog more terse and actionable.
npm/puppeteer/README.md
Outdated
@@ -46,6 +46,8 @@ Add the following in `tsconfig.json`: | |||
|
|||
Only Chromium-based browsers (e.g. Chrome, Chromium, Electron) are supported. | |||
|
|||
As of Chrome 137, the `@cypress/puppeteer` plugin will not work in `cypress open` mode (or headed `run` mode) with official branded versions of Chrome (`stable`, `beta`, `dev`, and `canary`). If you need to run the `@cypress/puppeteer` in `cypress open` mode, you will need to use [Chrome for Testing](https://developer.chrome.com/blog/chrome-for-testing), Chromium, or another Chrome variant that supports loading web extensions. The plugin will work as expected in `cypress run` mode in any version of Chrome. |
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 already merged a PR for this doc update, so you need to pull in develop.
@alexsch01 We will only error if you are using the |
75f6190
to
d98bad5
Compare
@cypress/puppeteer
or --load-extension
@cypress/puppeteer
or --load-extension
d98bad5
to
4864c12
Compare
91abfc2
to
37835e0
Compare
…uppeteeris used in headed mode or if --load-extension is added to launch options via a user testing an extension empty commit to run ci
37835e0
to
ff5fbb8
Compare
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.
Wish I could have gotten the manual testing working, but this code looks good.
Released in This comment thread has been locked. If you are still experiencing this issue after upgrading to |
--load-extension
usage with Chrome 137+ branded builds #31702@cypress/puppeteer
plugin to warn users about Chrome 137+ branded builds #31703Additional details
With the release of Chrome v137, the
--load-extension
launch argument will no longer be supported in branded versions of chrome. This impacts the@cypress/puppeteer
plugin and users leveraging launchOptions.extensions to add extensions to Chrome. If users need to leverage these options, please use Chrome for Testing, Chromium, or another Chrome variant that supports loading extensions.To mitigate this, Cypress will now print a warning if branded chrome 137 and up is detected and the
--load-extension
argument is passed in by a user. Additionally,@cypress/puppeteer
will also error if branded chrome 137 and up is detected.Steps to test
TODO: test the binary and verify functionality. Add unit tests and system tests that leverage the plugin
How has the user experience changed?
PR Tasks
cypress-documentation
? Update limitations for loading extensions within Cypress cypress-documentation#6183type definitions
?