chore(tests): remove createContainer helper

createContainer function is now implemented by VTU mimicking this behaviour when you pass `attachTo: document.body`
bootstrap-vue · Dec 18, 2021 · d113cc7 · d113cc7
1 parent 7ad43e1
commit d113cc7
Show file tree

Hide file tree

Showing 46 changed files with 376 additions and 389 deletions.
diff --git a/src/components/button-toolbar/button-toolbar.spec.js b/src/components/button-toolbar/button-toolbar.spec.js
@@ -1,5 +1,5 @@
 import { mount } from '@vue/test-utils'
-import { createContainer, waitNT } from '../../../tests/utils'
+import { waitNT } from '../../../tests/utils'
 import { BButton } from '../button/button'
 import { BButtonGroup } from '../button-group/button-group'
 import { BButtonToolbar } from './button-toolbar'
@@ -93,7 +93,7 @@ describe('button-toolbar', () => {
 
     it('has correct structure', async () => {
       const wrapper = mount(App, {
-        attachTo: createContainer()
+        attachTo: document.body
       })
 
       await waitNT(wrapper.vm)
@@ -150,7 +150,7 @@ describe('button-toolbar', () => {
 
     it('focuses first button when tabbed into', async () => {
       const wrapper = mount(App, {
-        attachTo: createContainer()
+        attachTo: document.body
       })
 
       await waitNT(wrapper.vm)
@@ -173,7 +173,7 @@ describe('button-toolbar', () => {
 
     it('keyboard navigation works', async () => {
       const wrapper = mount(App, {
-        attachTo: createContainer()
+        attachTo: document.body
       })
 
       await waitNT(wrapper.vm)

diff --git a/src/components/calendar/calendar.spec.js b/src/components/calendar/calendar.spec.js
@@ -1,5 +1,5 @@
 import { mount } from '@vue/test-utils'
-import { createContainer, waitNT, waitRAF } from '../../../tests/utils'
+import { waitNT, waitRAF } from '../../../tests/utils'
 import { BCalendar } from './calendar'
 import { formatYMD } from '../../utils/date'
 
@@ -8,7 +8,7 @@ import { formatYMD } from '../../utils/date'
 describe('calendar', () => {
   it('has expected base structure', async () => {
     const wrapper = mount(BCalendar, {
-      attachTo: createContainer()
+      attachTo: document.body
     })
 
     expect(wrapper.vm).toBeDefined()
@@ -42,7 +42,7 @@ describe('calendar', () => {
 
   it('has expected structure when value is set', async () => {
     const wrapper = mount(BCalendar, {
-      attachTo: createContainer(),
+      attachTo: document.body,
       propsData: {
         value: '2020-02-15' // Leap year
       }
@@ -62,7 +62,7 @@ describe('calendar', () => {
 
   it('reacts to changes in value', async () => {
     const wrapper = mount(BCalendar, {
-      attachTo: createContainer(),
+      attachTo: document.body,
       propsData: {
         value: '2020-01-01' // Leap year
       }
@@ -88,7 +88,7 @@ describe('calendar', () => {
 
   it('clicking a date selects date', async () => {
     const wrapper = mount(BCalendar, {
-      attachTo: createContainer(),
+      attachTo: document.body,
       propsData: {
         value: '2020-01-01' // Leap year
       }
@@ -122,7 +122,7 @@ describe('calendar', () => {
 
   it('date navigation buttons work', async () => {
     const wrapper = mount(BCalendar, {
-      attachTo: createContainer(),
+      attachTo: document.body,
       propsData: {
         showDecadeNav: true,
         value: '2020-02-15' // Leap year
@@ -178,7 +178,7 @@ describe('calendar', () => {
 
   it('focus and blur methods work', async () => {
     const wrapper = mount(BCalendar, {
-      attachTo: createContainer(),
+      attachTo: document.body,
       propsData: {
         value: '2020-02-15' // Leap year
       }
@@ -211,7 +211,7 @@ describe('calendar', () => {
 
   it('clicking output header focuses grid', async () => {
     const wrapper = mount(BCalendar, {
-      attachTo: createContainer(),
+      attachTo: document.body,
       propsData: {
         value: '2020-02-15' // Leap year
       }
@@ -246,7 +246,7 @@ describe('calendar', () => {
 
   it('has correct header tag when "header-tag" prop is set', async () => {
     const wrapper = mount(BCalendar, {
-      attachTo: createContainer(),
+      attachTo: document.body,
       propsData: {
         value: '2020-02-15', // Leap year,
         headerTag: 'div'
@@ -266,7 +266,7 @@ describe('calendar', () => {
 
   it('keyboard navigation works', async () => {
     const wrapper = mount(BCalendar, {
-      attachTo: createContainer(),
+      attachTo: document.body,
       propsData: {
         value: '2020-02-15' // Leap year
       }
@@ -362,7 +362,7 @@ describe('calendar', () => {
 
   it('should disable key navigation when `no-key-nav` prop set', () => {
     const wrapper = mount(BCalendar, {
-      attachTo: createContainer(),
+      attachTo: document.body,
       propsData: {
         noKeyNav: true,
         navButtonVariant: 'primary'
@@ -381,7 +381,7 @@ describe('calendar', () => {
 
   it('`nav-button-variant` changes nav button class', async () => {
     const wrapper = mount(BCalendar, {
-      attachTo: createContainer(),
+      attachTo: document.body,
       propsData: {
         navButtonVariant: 'primary'
       }
@@ -400,7 +400,7 @@ describe('calendar', () => {
 
   it('disables dates based on `date-disabled-fn` prop', async () => {
     const wrapper = mount(BCalendar, {
-      attachTo: createContainer(),
+      attachTo: document.body,
       propsData: {
         value: '2020-01-01',
         dateDisabledFn(ymd) {
@@ -433,7 +433,7 @@ describe('calendar', () => {
 
   it('applies classes on dates based on `date-info-fn` prop', async () => {
     const wrapper = mount(BCalendar, {
-      attachTo: createContainer(),
+      attachTo: document.body,
       propsData: {
         value: '2020-01-01',
         dateInfoFn(ymd) {

diff --git a/src/components/carousel/carousel.spec.js b/src/components/carousel/carousel.spec.js
@@ -1,5 +1,5 @@
 import { mount } from '@vue/test-utils'
-import { createContainer, waitNT, waitRAF } from '../../../tests/utils'
+import { waitNT, waitRAF } from '../../../tests/utils'
 import { BCarousel } from './carousel'
 import { BCarouselSlide } from './carousel-slide'
 
@@ -34,7 +34,7 @@ const App = {
 describe('carousel', () => {
   it('has expected default structure', async () => {
     const wrapper = mount(BCarousel, {
-      attachTo: createContainer()
+      attachTo: document.body
     })
 
     expect(wrapper.vm).toBeDefined()
@@ -101,7 +101,7 @@ describe('carousel', () => {
 
   it('has prev/next controls when prop controls is set', async () => {
     const wrapper = mount(BCarousel, {
-      attachTo: createContainer(),
+      attachTo: document.body,
       propsData: {
         controls: true
       }
@@ -164,7 +164,7 @@ describe('carousel', () => {
 
   it('has indicators showing when prop indicators is set', async () => {
     const wrapper = mount(BCarousel, {
-      attachTo: createContainer(),
+      attachTo: document.body,
       propsData: {
         indicators: true
       }
@@ -211,7 +211,7 @@ describe('carousel', () => {
 
   it('should have class "carousel-fade" when prop "fade" is "true"', async () => {
     const wrapper = mount(BCarousel, {
-      attachTo: createContainer(),
+      attachTo: document.body,
       propsData: {
         fade: true
       }
@@ -230,7 +230,7 @@ describe('carousel', () => {
 
   it('should not have class "fade" or "slide" when prop "no-animation" is "true"', async () => {
     const wrapper = mount(BCarousel, {
-      attachTo: createContainer(),
+      attachTo: document.body,
       propsData: {
         noAnimation: true
       }
@@ -249,7 +249,7 @@ describe('carousel', () => {
 
   it('should not have class "fade" or "slide" when prop "no-animation" and "fade" are "true"', async () => {
     const wrapper = mount(BCarousel, {
-      attachTo: createContainer(),
+      attachTo: document.body,
       propsData: {
         fade: true,
         noAnimation: true
@@ -269,7 +269,7 @@ describe('carousel', () => {
 
   it('should not automatically scroll to next slide when "interval" is "0"', async () => {
     const wrapper = mount(App, {
-      attachTo: createContainer(),
+      attachTo: document.body,
       propsData: {
         interval: 0
       }
@@ -296,7 +296,7 @@ describe('carousel', () => {
 
   it('should scroll to next/prev slide when next/prev clicked', async () => {
     const wrapper = mount(App, {
-      attachTo: createContainer(),
+      attachTo: document.body,
       propsData: {
         interval: 0,
         controls: true
@@ -358,7 +358,7 @@ describe('carousel', () => {
 
   it('should scroll to next/prev slide when next/prev space keypress', async () => {
     const wrapper = mount(App, {
-      attachTo: createContainer(),
+      attachTo: document.body,
       propsData: {
         interval: 0,
         controls: true
@@ -420,7 +420,7 @@ describe('carousel', () => {
 
   it('should scroll to specified slide when indicator clicked', async () => {
     const wrapper = mount(App, {
-      attachTo: createContainer(),
+      attachTo: document.body,
       propsData: {
         interval: 0,
         controls: true
@@ -482,7 +482,7 @@ describe('carousel', () => {
 
   it('should scroll to specified slide when indicator keypress space/enter', async () => {
     const wrapper = mount(App, {
-      attachTo: createContainer(),
+      attachTo: document.body,
       propsData: {
         interval: 0,
         controls: true
@@ -544,7 +544,7 @@ describe('carousel', () => {
 
   it('should scroll to next/prev slide when key next/prev pressed', async () => {
     const wrapper = mount(App, {
-      attachTo: createContainer(),
+      attachTo: document.body,
       propsData: {
         interval: 0,
         controls: true
@@ -603,7 +603,7 @@ describe('carousel', () => {
 
   it('should emit paused and unpaused events when "interval" changed to 0', async () => {
     const wrapper = mount(App, {
-      attachTo: createContainer(),
+      attachTo: document.body,
       propsData: {
         interval: 0
       }
@@ -682,7 +682,7 @@ describe('carousel', () => {
 
   it('should scroll to specified slide when value (v-model) changed', async () => {
     const wrapper = mount(App, {
-      attachTo: createContainer(),
+      attachTo: document.body,
       propsData: {
         interval: 0,
         value: 0
@@ -761,7 +761,7 @@ describe('carousel', () => {
 
   it('changing slides works when "no-animation" set', async () => {
     const wrapper = mount(App, {
-      attachTo: createContainer(),
+      attachTo: document.body,
       propsData: {
         interval: 0,
         noAnimation: true
@@ -825,7 +825,7 @@ describe('carousel', () => {
 
   it('setting new slide when sliding is active, schedules the new slide to happen after finished', async () => {
     const wrapper = mount(App, {
-      attachTo: createContainer(),
+      attachTo: document.body,
       propsData: {
         interval: 0
       }
@@ -902,7 +902,7 @@ describe('carousel', () => {
 
   it('next/prev slide wraps to end/start when "no-wrap is "false"', async () => {
     const wrapper = mount(App, {
-      attachTo: createContainer(),
+      attachTo: document.body,
       propsData: {
         interval: 0,
         noAnimation: true,
@@ -967,7 +967,7 @@ describe('carousel', () => {
 
   it('next/prev slide does not wrap to end/start when "no-wrap" is "true"', async () => {
     const wrapper = mount(App, {
-      attachTo: createContainer(),
+      attachTo: document.body,
       propsData: {
         interval: 0,
         // Transitions (or fallback timers) are not used when no-animation set