Skip to content

feat(runtime-utils): refactor mount + render helpers#1483

Merged
danielroe merged 1 commit intonuxt:mainfrom
yamachi4416:feat/refactor-mount-render-helpers
Dec 4, 2025
Merged

feat(runtime-utils): refactor mount + render helpers#1483
danielroe merged 1 commit intonuxt:mainfrom
yamachi4416:feat/refactor-mount-render-helpers

Conversation

@yamachi4416
Copy link
Copy Markdown
Collaborator

@yamachi4416 yamachi4416 commented Nov 5, 2025

🔗 Linked issue

resolves #684, resolves #731, resolves #735, resolves #794, resolves #773, resolves #1115, resolves #1199, resolves #1205

❓ Type of change

  • 📖 Documentation (updates to the documentation or readme)
  • 🐞 Bug fix (a non-breaking change that fixes an issue)
  • 👌 Enhancement (improving an existing functionality)
  • ✨ New feature (a non-breaking change that adds functionality)
  • 🧹 Chore (updates to the build process or auxiliary tools and libraries)
  • ⚠️ Breaking change (fix or feature that would cause existing functionality to change)

📚 Description

@pkg-pr-new
Copy link
Copy Markdown

pkg-pr-new bot commented Nov 5, 2025

Open in StackBlitz

npm i https://pkg.pr.new/@nuxt/test-utils@1483

commit: d0fa824

@yamachi4416
Copy link
Copy Markdown
Collaborator Author

yamachi4416 commented Nov 5, 2025

I've incorporated into my repo and checked it out.
https://github.com/yamachi4416/aon-typing/actions/runs/19104148782

@yamachi4416 yamachi4416 marked this pull request as ready for review November 5, 2025 13:58
@yamachi4416 yamachi4416 marked this pull request as draft November 6, 2025 05:55
@yamachi4416
Copy link
Copy Markdown
Collaborator Author

I've reverted the status to draft because I want to conduct more testing and operational verification.
Thank you❤

@yamachi4416 yamachi4416 force-pushed the feat/refactor-mount-render-helpers branch 3 times, most recently from bb5c5b0 to 8b8d891 Compare November 8, 2025 05:11
@yamachi4416 yamachi4416 marked this pull request as ready for review November 8, 2025 05:53
@yamachi4416
Copy link
Copy Markdown
Collaborator Author

I have completed the intended testing and operational checks, so I have removed the draft status from the PR.
Please review it.

@yamachi4416 yamachi4416 force-pushed the feat/refactor-mount-render-helpers branch from 8b8d891 to 8523c1b Compare November 8, 2025 06:08
@yamachi4416 yamachi4416 marked this pull request as draft November 8, 2025 10:20
@yamachi4416
Copy link
Copy Markdown
Collaborator Author

I've reverted the status to draft because I want to conduct more testing and operational verification.
sorry...

@yamachi4416 yamachi4416 force-pushed the feat/refactor-mount-render-helpers branch from 8523c1b to 75efe57 Compare November 8, 2025 14:09
@yamachi4416
Copy link
Copy Markdown
Collaborator Author

I've incorporated into nuxt/ui and checked it out.
https://github.com/yamachi4416/nuxt-ui/actions/runs/19194872201

One snapshot failed, but I checked it with the same settings in the playground.
yamachi4416/nuxt-ui@043d453#r170098451

@yamachi4416 yamachi4416 force-pushed the feat/refactor-mount-render-helpers branch from 75efe57 to 3a258b9 Compare November 9, 2025 00:49
@yamachi4416
Copy link
Copy Markdown
Collaborator Author

yamachi4416 commented Nov 9, 2025

I've incorporated into nuxt and checked it out.
https://github.com/yamachi4416/nuxt/actions/runs/19201294069

The test failed, but I think it is correct.
yamachi4416/nuxt@c0981d9#r170104846

@yamachi4416
Copy link
Copy Markdown
Collaborator Author

I have updated the test-utils in the PR and re-ran the tests on the next/ui side as wel
https://github.com/yamachi4416/nuxt-ui/actions/runs/19201476840/job/54890049127

@yamachi4416 yamachi4416 force-pushed the feat/refactor-mount-render-helpers branch from 3a258b9 to d0fa824 Compare November 9, 2025 07:30
@yamachi4416
Copy link
Copy Markdown
Collaborator Author

I've made an effort to align the behavior with vue/test-utils as much as possible. I apologize for changing the review status multiple times due to the large number of things I needed to check. Please review.

Below are the links to my confirmation using the package from this PR:
https://github.com/yamachi4416/nuxt/actions/runs/19205724525
https://github.com/yamachi4416/nuxt-ui/actions/runs/19205732383

@yamachi4416 yamachi4416 marked this pull request as ready for review November 9, 2025 09:42
Copy link
Copy Markdown
Member

@danielroe danielroe left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thank you very much for this thorough work ❤️

@danielroe danielroe merged commit 72cf241 into nuxt:main Dec 4, 2025
7 checks passed
This was referenced Dec 4, 2025
@yamachi4416 yamachi4416 deleted the feat/refactor-mount-render-helpers branch December 5, 2025 10:24
@danielroe
Copy link
Copy Markdown
Member

@yamachi4416 I'm not sure how I didn't catch it, but it seems that this has caused a number of test failures upstream in nuxt/nuxt.

If you have a chance, I'd welcome your review/thoughts 🙏

@yamachi4416
Copy link
Copy Markdown
Collaborator Author

@danielroe
thank you for letting me know. I'll take a look.

@yamachi4416
Copy link
Copy Markdown
Collaborator Author

@danielroe
thank you again.
I've opened a PR to update nuxt/test-utils and fix the failing tests.
please see the PR description for the reason behind the failures.
I'd appreciate your review.
nuxt/nuxt#33842

@danielroe
Copy link
Copy Markdown
Member

thank you so much ❤️

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment