fix: return true in isEmpty if child isComment (#475)

Author: eddyerburgh

packages/test-utils/src/wrapper.js

@@ -335,6 +335,9 @@ export default class Wrapper implements BaseWrapper {
     if (!this.vnode) {
       return this.element.innerHTML === ''
     }
+    if (this.vnode.children) {
+      return this.vnode.children.every(vnode => vnode.isComment)
+    }
     return this.vnode.children === undefined || this.vnode.children.length === 0
   }
 

test/specs/wrapper/isEmpty.spec.js

@@ -9,6 +9,12 @@ describeWithShallowAndMount('isEmpty', (mountingMethod) => {
     expect(wrapper.isEmpty()).to.equal(true)
   })
 
+  it('returns true if node contains comment', () => {
+    const compiled = compileToFunctions('<div><div v-if="false"></div></div>')
+    const wrapper = mountingMethod(compiled)
+    expect(wrapper.isEmpty()).to.equal(true)
+  })
+
   it('returns true if innerHTML is empty', () => {
     const TestComponent = {
       render (createElement) {