-
Notifications
You must be signed in to change notification settings - Fork 72
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
Error occurred after updating 3.9 #681
Comments
Currently facing the same issue. The workaround I found is to remove the It seems that the module plugin tries to follow imports, however it does not understand the virtual |
currently started facing the same issue, removing it from nuxt.config helped |
@rafaelmagalhaes But the problem is that when running a test, it fails to load #imports from in @nuxt/test-utils/dist/runtime-utils/index.js. |
Would you be able to provide a reproduction? 🙏 More infoWhy do I need to provide a reproduction?Reproductions make it possible for us to triage and fix issues quickly with a relatively small team. It helps us discover the source of the problem, and also can reveal assumptions you or we might be making. What will happen?If you've provided a reproduction, we'll remove the label and try to reproduce the issue. If we can, we'll mark it as a bug and prioritise it based on its severity and how many people we think it might affect. If How can I create a reproduction?We have a couple of templates for starting with a minimal reproduction: 👉 https://stackblitz.com/github/nuxt/test-utils/tree/main/examples/app-vitest A public GitHub repository is also perfect. 👌 Please ensure that the reproduction is as minimal as possible. See more details in our guide. You might also find these other articles interesting and/or helpful: |
I'll create a reproduction repo tomorrow. I think this has to do with test code in a project importing test-utils code. |
Finally got it reproduced here: https://stackblitz.com/edit/github-3etpwp?file=nuxt.config.ts,pages%2Fapp.nuxt.spec.ts At first I could not get the repro repo to trigger an error on The error:
Could it be that the the Nuxt import mocking messes with the generated vue-router code? Seems like an order thing. |
This should now be resolved via #726. |
Just tested this on the test repo I created. The above error is now gone when running the tests, however a new error pops up when starting the project: https://stackblitz.com/edit/github-3etpwp?file=package.json
The error does not seem to have effect on the app startup itself (it's probably more a warning). Again this error goes away when moving the test out of the @danielroe Should I create a new issue or do you want to reopen this one? |
Do open a new issue 🙏 We can probably resolve this easily. |
@AlbertBrand Same issue here :( |
Created #747 |
I'm still experiencing the issue mentionned on Jan 12, with I tried what's mentionned here but it doesn't do the trick I'll try to create a reproduction |
I guys, I was having the same error, so, I tried a different approach and it worked for me. Well, it's a workaround - I think - but maybe it can help anyone who needs. Try to use pnpm instead. my gitlab-ci
|
Environment
Darwin
v18.18.0
3.9.0
3.10.0
2.8.1
bun@1.0.20
-
devtools
,modules
,primevue
,css
,components
,imports
,i18n
,supabase
,runtimeConfig
,experimental
,vite
,testUtils
@vueuse/nuxt@10.7.1
,@nuxtjs/supabase@1.1.4
,@nuxtjs/eslint-module@4.1.0
,@nuxtjs/i18n@8.0.0-rc.5
,@nuxtjs/stylelint-module@5.1.0
,@pinia/nuxt@0.5.1
,@pinia-plugin-persistedstate/nuxt@1.2.0
,@element-plus/nuxt@1.0.7
,nuxt-icon@0.6.8
,@samk-dev/nuxt-vcalendar@1.0.2
,@nuxt/image@1.1.0
,nuxt-primevue@0.2.2
,dayjs-nuxt@2.1.9
,@nuxt/test-utils/module@3.9.0
-
Reproduction
update the package to 3.9.0. and put '@nuxt/test-utils/module' into modules in nuxt.config.
then in some reasons, it goes error when nuxt modules trying to import explicitly from #imports or #build.
Describe the bug
I just updated test-utils as nuxt-vitest archived, and set config like you mentioned.
and running nuxt, it goes error.
Additional context
No response
Logs
The text was updated successfully, but these errors were encountered: