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: find component by its definition when using stub without name #704

Merged

Conversation

xanf
Copy link
Collaborator

@xanf xanf commented Jun 28, 2021

Follow-up of #696

When we're stubbing component (no matter with mount or shallow: true) we still want to find a component by searching original component definition

Copy link
Member

@afontcu afontcu left a comment

Choose a reason for hiding this comment

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

LGTM!

I was concerned with the situation where a not-stubbed component that doesn't render (by using a v-if="false" at root, for instance) might be found by using a stub. I checked it out and turns out we can already find() a hidden component, so nothing to worry about!

@xanf xanf force-pushed the xanf-find-component-without-name-by-stub branch from 9c9e8bf to d767435 Compare June 29, 2021 04:14
@xanf
Copy link
Collaborator Author

xanf commented Jun 29, 2021

Rebased, should be ready to go

@lmiller1990 lmiller1990 merged commit f44eb33 into vuejs:master Jun 30, 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.

3 participants