-
Notifications
You must be signed in to change notification settings - Fork 244
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
Bug: spying on wrapper.vm for script setup methods using Vue 3.2.45 and VTU 2.2.2 stop working #1859
Comments
Fixes vuejs#1859 The proxy needs to implement `defineProperty` to allow spying.
Fixes vuejs#1859 The proxy needs to implement `defineProperty` to allow spying.
👍 Thanks for the repro This should be fixed by #1860 Vue changed its internal, and we now have to do a bit more work in VTU to allow the same feature than previously. You can stay on Vue v3.2.44 until this is released |
ok, just to confirm, Vue 3.2.44 and VTU 2.2.1, correct? |
Exactly |
Fixes #1859 The proxy needs to implement `defineProperty` to allow spying.
it works now, thanks. |
Describe the bug
All the tests we have using explicit casting for wrapper.vm in order to spy methods stop working after updating to Vue 3.2.45 and VTU 2.2.2.
Related bugs:
wrapper.vm
when the component is closed #972To Reproduce
This is the minimal sample based on one of the failing tests. With the previous version, this test passes.
https://stackblitz.com/edit/vitest-dev-vitest-pcp793?file=components%2FHello.vue
Expected behavior
The test must be passed.
Related information:
Additional context
NA
The text was updated successfully, but these errors were encountered: