fix: do not match component name on getter that returns component (#1312

)

* test: Add failing tests for shallowMount issue with dynamic components

* fix: Do not match component name on getter that returns component obj

* test: Removing extraneous/illustrative test code

src/utils/componentName.ts

@@ -11,7 +11,8 @@ const getComponentNameInSetup = (
   type: VNodeTypes
 ): string | undefined =>
   Object.keys(instance?.setupState || {}).find(
-    (key) => instance.setupState[key] === type
+    (key) =>
+      Object.getOwnPropertyDescriptor(instance.setupState, key)?.value === type
   )
 
 export const getComponentRegisteredName = (

tests/components/DynamicComponentWithComputedProperty.vue

@@ -0,0 +1,10 @@
+<template>
+  <component :is="computedProperty" />
+</template>
+
+<script setup lang="ts">
+import { computed } from 'vue'
+import Hello from './Hello.vue'
+
+const computedProperty = computed(() => Hello)
+</script>

tests/shallowMount.spec.ts

@@ -2,6 +2,7 @@ import { defineAsyncComponent, defineComponent } from 'vue'
 import { mount, shallowMount, VueWrapper } from '../src'
 import ComponentWithChildren from './components/ComponentWithChildren.vue'
 import ScriptSetupWithChildren from './components/ScriptSetupWithChildren.vue'
+import DynamicComponentWithComputedProperty from './components/DynamicComponentWithComputedProperty.vue'
 
 describe('shallowMount', () => {
   it('renders props for stubbed component in a snapshot', () => {
@@ -196,4 +197,16 @@ describe('shallowMount', () => {
         '</div>'
     )
   })
+
+  it('stubs a given component that is also returned by a computed property', () => {
+    const wrapper = shallowMount(DynamicComponentWithComputedProperty)
+
+    expect(wrapper.find('hello-stub').exists()).toBe(true)
+  })
+
+  it('does not attempt to stub a dynamic component based on the name of a computed property', () => {
+    const wrapper = shallowMount(DynamicComponentWithComputedProperty)
+
+    expect(wrapper.find('computed-property-stub').exists()).toBe(false)
+  })
 })