From 4b6a691449fb769270f953f85cf3d96f8b8cf805 Mon Sep 17 00:00:00 2001
From: Joeri Hendrickx <joeri.hendrickx@unleashed.be>
Date: Thu, 22 Oct 2020 14:58:57 +0200
Subject: [PATCH] fix: ignore non-vue wrapper for auto-destroy

---
 packages/test-utils/src/auto-destroy.js |  6 +++---
 test/specs/wrapper.spec.js              | 14 +++++++++++++-
 2 files changed, 16 insertions(+), 4 deletions(-)

diff --git a/packages/test-utils/src/auto-destroy.js b/packages/test-utils/src/auto-destroy.js
index 8f9073077..15f27c500 100644
--- a/packages/test-utils/src/auto-destroy.js
+++ b/packages/test-utils/src/auto-destroy.js
@@ -21,9 +21,9 @@ export function enableAutoDestroy(hook: (() => void) => void) {
   hook(() => {
     wrapperInstances.forEach((wrapper: Wrapper) => {
       // skip child wrappers created by wrapper.find()
-      if (wrapper.selector) return
-
-      wrapper.destroy()
+      if (wrapper.vm || wrapper.isFunctionalComponent) {
+        wrapper.destroy()
+      }
     })
 
     wrapperInstances.length = 0
diff --git a/test/specs/wrapper.spec.js b/test/specs/wrapper.spec.js
index 95934dac3..b1855b196 100644
--- a/test/specs/wrapper.spec.js
+++ b/test/specs/wrapper.spec.js
@@ -1,7 +1,8 @@
 import { describeWithShallowAndMount } from '~resources/utils'
 import {
   enableAutoDestroy,
-  resetAutoDestroyState
+  resetAutoDestroyState,
+  createWrapper
 } from 'packages/test-utils/src'
 
 describeWithShallowAndMount('Wrapper', mountingMethod => {
@@ -51,5 +52,16 @@ describeWithShallowAndMount('Wrapper', mountingMethod => {
 
       expect(() => enableAutoDestroy(noop)).toThrow()
     })
+
+    it('does not fail when non-Vue wrappers exist', async () => {
+      let hookCallback
+      enableAutoDestroy(callback => {
+        hookCallback = callback
+      })
+
+      createWrapper(document.createElement('div'))
+
+      expect(hookCallback).not.toThrow()
+    })
   })
 })