fix: revert "feat: remove empty div for bot"

This reverts commit 4ffb9b1
Developmint · May 23, 2018 · 6178f68 · 6178f68
1 parent 1b027c8
commit 6178f68
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/src/index.js b/src/index.js
@@ -5,6 +5,6 @@ export default {
     const isBot = process.server ? true : navigator.userAgent && botRegex.test(navigator.userAgent)
     const shouldShow = invert ? isBot : !isBot
 
-    return shouldShow ? h('div', {}, children) : ''
+    return h('div', {}, shouldShow ? children : [h('div')])
   }
 }
diff --git a/test/VueIsBot.spec.js b/test/VueIsBot.spec.js
@@ -10,13 +10,13 @@ const mockUserAgent = userAgent => {
 
 const testCanSeeOrNot = (can, wrapper) => {
   expect(wrapper.isVueInstance()).toBe(true)
-  expect(wrapper.html()).toBe(can ? `<div>${slotContent}</div>` : undefined)
+  expect(wrapper.html()).toBe(`<div>${can ? slotContent : '<div></div>'}</div>`)
 }
 
 const testCanSee = curry(w => testCanSeeOrNot(true, w))
 const testCanNotSee = curry(w => testCanSeeOrNot(false, w))
 
-const testSsr = html => { expect(html).toBe(`<!---->`) }
+const testSsr = html => { expect(html).toBe(`<div data-server-rendered="true"><div></div></div>`) }
 
 describe('VueIfBot', () => {
   afterEach(() => { mockUserAgent('Normal user here') })