-
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
Deprecation warnings from electron / node during cypress open
#6750
Comments
Bleh, the I can look into the IPC warnings, most likely just need to filter out non-clonable objects in the desktop-gui's IPC code. |
Found some info to help clean up objects so they can be cleanly cloned and avoid the second deprecation warning: https://developer.mozilla.org/en-US/docs/Web/API/Web_Workers_API/Structured_clone_algorithm |
The code for this is done in cypress-io/cypress#6753, but has yet to be released. |
Released in This comment thread has been locked. If you are still experiencing this issue after upgrading to |
Current behavior:
Since updating to 4.2.0, I'm seeing some deprecation warnings in the stdout when running
cypress open
Desired behavior:
Would like for users of cypress to either not have these deprecation warnings shown to them or fix the usecases so the deprecation warnings do not need to be shown at all.
Deprecation warnings
I believe all of these originate from #6555
Electron [Discussion] Requiring Native Modules in the Renderer Process to be NAPI or Context Aware electron/electron#18397
This is being used here, looks like this issue is already noted https://github.com/cypress-io/cypress/blob/develop/packages/server/lib/util/electron-app.js#L18:L18
Node - also coming from Electron. Explanation of this one here: https://www.electronjs.org/docs/breaking-changes#sending-non-js-objects-over-ipc-now-throws-an-exception
Versions
4.2.0
The text was updated successfully, but these errors were encountered: