Skip to content
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.

Sign up for GitHub

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

Jump to bottom

renderSuspended does not provide the values returned from setup to the template #722

Closed
obulat opened this issue Jan 18, 2024 · 1 comment · Fixed by #852
Closed

renderSuspended does not provide the values returned from setup to the template #722

obulat opened this issue Jan 18, 2024 · 1 comment · Fixed by #852
Labels
pending triage

Comments

@obulat
Copy link

obulat commented Jan 18, 2024

Environment

Nuxt project info:

  • Operating System: Linux
  • Node Version: v18.18.0
  • Nuxt Version: 3.9.3
  • CLI Version: 3.10.0
  • Nitro Version: 2.8.1
  • Package Manager: [email protected]
  • Builder: -
  • User Config: devtools, modules
  • Runtime Modules: @nuxt/test-utils/[email protected]
  • Build Modules: -

Reproduction

https://stackblitz.com/edit/github-7wbhms?file=test%2Ftest-component.spec.ts

Describe the bug

renderSuspended does not pass the values from setup function (not the <script setup>) to the template. The mountSuspended works fine.

Additional context

In the Openverse Nuxt 3 migration PR, I replaced render(renderContext, ...args) with render(_ctx, ...args) on this line:

test-utils/src/runtime-utils/render.ts

Line 113 in bc86d23

render(renderContext, ...args)

which seems to fix the issue.

Logs

No response
@dosubot dosubot bot added the stale Issue has not had recent activity or appears to be solved. Stale issues will be automatically closed label Apr 24, 2024
@dosubot dosubot bot closed this as not planned Won't fix, can't repro, duplicate, stale May 1, 2024
@dosubot dosubot bot removed the stale Issue has not had recent activity or appears to be solved. Stale issues will be automatically closed label May 1, 2024
@danielroe danielroe reopened this May 1, 2024
@joaovitoras joaovitoras mentioned this issue May 20, 2024
Merged
@joaovitoras
Copy link
Contributor

joaovitoras commented May 20, 2024 •
edited

This is because of defineComponent wrapper
I opened a PR to fix this #852

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
pending triage
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

feat(runtime-utils): expose setup context from renderSuspended result joaovitoras/test-utils
3 participants
@joaovitoras @obulat @danielroe