mirror of
https://github.com/twbs/bootstrap.git
synced 2025-08-18 03:11:19 +02:00
tests: try to fix a few random failures (#35184)
* Change `Swipe` dispose spy on EventHandler * Modal hide spy on backdrop hide
This commit is contained in:
@@ -1268,6 +1268,7 @@ describe('Carousel', () => {
|
|||||||
|
|
||||||
carousel.dispose()
|
carousel.dispose()
|
||||||
|
|
||||||
|
expect(carousel._swipeHelper).toBeNull()
|
||||||
expect(removeEventSpy).toHaveBeenCalledWith(carouselEl, Carousel.EVENT_KEY)
|
expect(removeEventSpy).toHaveBeenCalledWith(carouselEl, Carousel.EVENT_KEY)
|
||||||
expect(swipeHelperSpy).toHaveBeenCalled()
|
expect(swipeHelperSpy).toHaveBeenCalled()
|
||||||
|
|
||||||
|
@@ -608,6 +608,7 @@ describe('Modal', () => {
|
|||||||
|
|
||||||
const modalEl = fixtureEl.querySelector('.modal')
|
const modalEl = fixtureEl.querySelector('.modal')
|
||||||
const modal = new Modal(modalEl)
|
const modal = new Modal(modalEl)
|
||||||
|
const backdropSpy = spyOn(modal._backdrop, 'hide').and.callThrough()
|
||||||
|
|
||||||
modalEl.addEventListener('shown.bs.modal', () => {
|
modalEl.addEventListener('shown.bs.modal', () => {
|
||||||
modal.hide()
|
modal.hide()
|
||||||
@@ -622,7 +623,7 @@ describe('Modal', () => {
|
|||||||
expect(modalEl.getAttribute('role')).toBeNull()
|
expect(modalEl.getAttribute('role')).toBeNull()
|
||||||
expect(modalEl.getAttribute('aria-hidden')).toEqual('true')
|
expect(modalEl.getAttribute('aria-hidden')).toEqual('true')
|
||||||
expect(modalEl.style.display).toEqual('none')
|
expect(modalEl.style.display).toEqual('none')
|
||||||
expect(document.querySelector('.modal-backdrop')).toBeNull()
|
expect(backdropSpy).toHaveBeenCalled()
|
||||||
done()
|
done()
|
||||||
})
|
})
|
||||||
|
|
||||||
|
@@ -232,7 +232,7 @@ describe('Swipe', () => {
|
|||||||
|
|
||||||
it('should destroy', () => {
|
it('should destroy', () => {
|
||||||
const addEventSpy = spyOn(fixtureEl, 'addEventListener').and.callThrough()
|
const addEventSpy = spyOn(fixtureEl, 'addEventListener').and.callThrough()
|
||||||
const removeEventSpy = spyOn(fixtureEl, 'removeEventListener').and.callThrough()
|
const removeEventSpy = spyOn(EventHandler, 'off').and.callThrough()
|
||||||
defineDocumentElementOntouchstart()
|
defineDocumentElementOntouchstart()
|
||||||
|
|
||||||
const swipe = new Swipe(fixtureEl)
|
const swipe = new Swipe(fixtureEl)
|
||||||
@@ -253,9 +253,8 @@ describe('Swipe', () => {
|
|||||||
|
|
||||||
swipe.dispose()
|
swipe.dispose()
|
||||||
|
|
||||||
expect(removeEventSpy.calls.allArgs()).toEqual(expectedArgs)
|
expect(removeEventSpy).toHaveBeenCalledWith(fixtureEl, '.bs.swipe')
|
||||||
|
deleteDocumentElementOntouchstart()
|
||||||
delete document.documentElement.ontouchstart
|
|
||||||
})
|
})
|
||||||
})
|
})
|
||||||
|
|
||||||
|
Reference in New Issue
Block a user