]> git.ipfire.org Git - thirdparty/bootstrap.git/commitdiff
fix modal test to be sure we call getTransitionDurationFromElement
authorJohann-S <johann.servoire@gmail.com>
Wed, 6 Feb 2019 13:35:59 +0000 (14:35 +0100)
committerJohann-S <johann.servoire@gmail.com>
Wed, 6 Feb 2019 14:01:13 +0000 (15:01 +0100)
js/tests/unit/modal.js

index 17629048031d19137f99a10d38cf1f394fd29463..9b7fb0aa669a2c6727391095328c4bdfd204e154 100644 (file)
@@ -669,7 +669,7 @@ $(function () {
   })
 
   QUnit.test('transition duration should be the modal-dialog duration before triggering shown event', function (assert) {
-    assert.expect(2)
+    assert.expect(1)
     var done = assert.async()
     var style = [
       '<style>',
@@ -694,22 +694,17 @@ $(function () {
       '</div>'
     ].join('')
 
-    var beginTimestamp = 0
     var $modal = $(modalHTML).appendTo('#qunit-fixture')
-    var $modalDialog = $('.modal-dialog')
-    var transitionDuration  = Util.getTransitionDurationFromElement($modalDialog[0])
-
-    assert.strictEqual(transitionDuration, 300)
+    var expectedTransitionDuration = 300
+    var spy = sinon.spy(Util, 'getTransitionDurationFromElement')
 
     $modal.on('shown.bs.modal', function () {
-      var diff = Date.now() - beginTimestamp
-      assert.ok(diff < 400)
+      assert.ok(spy.returned(expectedTransitionDuration))
       $style.remove()
+      spy.restore()
       done()
     })
       .bootstrapModal('show')
-
-    beginTimestamp = Date.now()
   })
 
   QUnit.test('should dispose modal', function (assert) {