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.

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

fix: opt-out of stubbing by passing false as stub #702

Merged
merged 1 commit into from
Jun 28, 2021

Conversation

xanf
Copy link
Collaborator

@xanf xanf commented Jun 28, 2021

@vue/test-utils 1 allowed to un-stub certain component by passing false as stub configuration

Recover and document this behavior

}
})
expect(wrapper.html()).toEqual(
'<div class="ComponentWithChildren">\n' +
Copy link
Member

Choose a reason for hiding this comment

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

I wonder if we can use the inlineSnapshot feature from Jest here? Or do you think the string assertion is easier to read?

Thanks for all these PRs - let's do a release as soon as they are all merged up.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I prefer to keep this as it is for now - just for consistency with style of other tests here. We could probably address it in separate clean-up MR

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

A bit offtopic, regarding release. I believe I have some more (~5-10 MR coming) somewhere in next two days. I'm working (for fun) on building a prototype migration of gitlab-ui to Vue3 using compat build and it is surprisingly satisfying.

With new plugin system in play I was able to mimic a huge precent of old VTU v1 behavior in pretty clean manner (for now we would like to have tests which run on both versions of VTU and)

Copy link
Member

Choose a reason for hiding this comment

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

Ok, happy to keep this as-is.

If you have 5-10 more PRs coming, we can wait until they are all merged up before doing a release.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

@lmiller1990 Sure, I'll drop a comment in "last PR" for letting you know. thank you for acting super-fast

@lmiller1990 lmiller1990 merged commit 181cd17 into vuejs:master Jun 28, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants