-
Notifications
You must be signed in to change notification settings - Fork 3.1k
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 deprecation warnings from Electron 8 upgrade #6753
Conversation
The previous code would only fix it in run mode.
Thanks for the contribution! Below are some guidelines Cypress uses when doing PR reviews.
PR Review ChecklistIf any of the following requirements can't be met, leave a comment in the review selecting 'Request changes', otherwise 'Approve'. User Experience
Functionality
Maintainability
Quality
Internal
|
Test summaryRun details
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 |
8bd8dd3
to
a362758
Compare
@@ -62,6 +62,18 @@ const ipcBus = (...args) => { | |||
// get the last argument | |||
const lastArg = args.pop() | |||
|
|||
// remove nodes that cannot be cloned | |||
// https://github.com/cypress-io/cypress/issues/6750 | |||
args = args.map((arg) => { |
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.
is there a test for this?
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.
no, i can add tests, just wasn't sure where to add them... probably just need to add tests for the ipc-bus
file
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.
it would be nice to send an object from the browser with functions and elements and ping it back from the server - and the receiver would check if the returned object has invalid fields stripped
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.
85abe2c
to
e34663c
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.
so beautiful
cypress open
#6750User facing changelog
stderr
inopen
mode.Additional details
open
mode are cloned to exclude functions and elements, to make the Structured Clone Algorithm usable: https://developer.mozilla.org/en-US/docs/Web/API/Web_Workers_API/Structured_clone_algorithmallowRendererProcessReuse
was only set inrun
mode, now it's set inopen
mode too to get rid of the warningnoDeprecation
when the env isproduction
, so no deprecation warnings will be printed in built binariesPR Tasks
cypress-documentation
?type definitions
?cypress.schema.json
?