tests: tweak Jasmine usage (#32046)

* jasmine/expect-matcher
* jasmine/prefer-jasmine-matcher

Found with `eslint-plugin-jasmine`

diff --git a/js/tests/unit/dom/event-handler.spec.js b/js/tests/unit/dom/event-handler.spec.js
index 5551ddaa30f78cbb725838a8986afa7b0c5cd82d..e596a49b59b93b3fb9f8db07bafa0486b5c48b59 100644 (file)
--- a/js/tests/unit/dom/event-handler.spec.js
+++ b/js/tests/unit/dom/event-handler.spec.js
@@ -297,31 +297,31 @@ describe('EventHandler', () => {
       EventHandler.trigger(subelement, 'click')
 
       // first listeners called
-      expect(i === 2).toEqual(true)
+      expect(i).toEqual(2)
 
       EventHandler.off(element, 'click', 'span', handler)
       EventHandler.trigger(subelement, 'click')
 
       // removed listener not called
-      expect(i === 2).toEqual(true)
+      expect(i).toEqual(2)
 
       EventHandler.trigger(anchor, 'click')
 
       // not removed listener called
-      expect(i === 3).toEqual(true)
+      expect(i).toEqual(3)
 
       EventHandler.on(element, 'click', 'span', handler)
       EventHandler.trigger(anchor, 'click')
       EventHandler.trigger(subelement, 'click')
 
       // listener re-registered
-      expect(i === 5).toEqual(true)
+      expect(i).toEqual(5)
 
       EventHandler.off(element, 'click', 'span')
       EventHandler.trigger(subelement, 'click')
 
       // listener removed again
-      expect(i === 5).toEqual(true)
+      expect(i).toEqual(5)
     })
   })
 })

diff --git a/js/tests/unit/dom/manipulator.spec.js b/js/tests/unit/dom/manipulator.spec.js
index 16750fd7415dc5f02ad4f2fbcb8910ea3767d2c5..8d8118f2ba4f1588253bb0ec21aafc1972a4068e 100644 (file)
--- a/js/tests/unit/dom/manipulator.spec.js
+++ b/js/tests/unit/dom/manipulator.spec.js
@@ -56,7 +56,7 @@ describe('Manipulator', () => {
 
   describe('getDataAttributes', () => {
     it('should return empty object for null', () => {
-      expect(Manipulator.getDataAttributes(null), {})
+      expect(Manipulator.getDataAttributes(null)).toEqual({})
       expect().nothing()
     })
 

diff --git a/js/tests/unit/dropdown.spec.js b/js/tests/unit/dropdown.spec.js
index c7f05ed22acb13381c49f76bdd383b6606df5814..17a5b41cf01861f8056c11f41430468420e3ca33 100644 (file)
--- a/js/tests/unit/dropdown.spec.js
+++ b/js/tests/unit/dropdown.spec.js
@@ -1320,8 +1320,8 @@ describe('Dropdown', () => {
         triggerDropdown.dispatchEvent(keydown)
 
         expect(document.activeElement.classList.contains('d-none')).toEqual(false, '.d-none not focused')
-        expect(document.activeElement.style.display === 'none').toEqual(false, '"display: none" not focused')
-        expect(document.activeElement.style.visibility === 'hidden').toEqual(false, '"visibility: hidden" not focused')
+        expect(document.activeElement.style.display).not.toBe('none', '"display: none" not focused')
+        expect(document.activeElement.style.visibility).not.toBe('hidden', '"visibility: hidden" not focused')
 
         done()
       })

diff --git a/js/tests/unit/modal.spec.js b/js/tests/unit/modal.spec.js
index 253f9351359727f27f2a6156ba17fc2294695e99..9ed1fc4348b25f799f81362177f7a80dc1dd9533 100644 (file)
--- a/js/tests/unit/modal.spec.js
+++ b/js/tests/unit/modal.spec.js
@@ -637,7 +637,7 @@ describe('Modal', () => {
       modalEl.addEventListener('shown.bs.modal', () => {
         modalEl.click()
         setTimeout(() => {
-          expect(modalEl.clientHeight === modalEl.scrollHeight).toEqual(true)
+          expect(modalEl.clientHeight).toEqual(modalEl.scrollHeight)
           done()
         }, 20)
       })