From 27bbc7dbe93799730112cb786a859331ca193130 Mon Sep 17 00:00:00 2001 From: Chris Rebert Date: Tue, 3 Mar 2015 13:32:24 -0800 Subject: Hiding an uninitialized tooltip/popover no longer initializes it; fixes #15874 --- js/tests/unit/popover.js | 12 ++++++++++++ js/tests/unit/tooltip.js | 12 ++++++++++++ 2 files changed, 24 insertions(+) (limited to 'js/tests') diff --git a/js/tests/unit/popover.js b/js/tests/unit/popover.js index ce89c8dbd..73a67c984 100644 --- a/js/tests/unit/popover.js +++ b/js/tests/unit/popover.js @@ -247,4 +247,16 @@ $(function () { }, new Error('`selector` option must be specified when initializing popover on the window.document object!')) }) + QUnit.test('should do nothing when an attempt is made to hide an uninitialized popover', function (assert) { + assert.expect(1) + + var $popover = $('some text') + .appendTo('#qunit-fixture') + .on('hidden.bs.popover shown.bs.popover', function () { + assert.ok(false, 'should not fire any popover events') + }) + .bootstrapPopover('hide') + assert.strictEqual($popover.data('bs.popover'), undefined, 'should not initialize the popover') + }) + }) diff --git a/js/tests/unit/tooltip.js b/js/tests/unit/tooltip.js index c2e2b937b..2875eff85 100644 --- a/js/tests/unit/tooltip.js +++ b/js/tests/unit/tooltip.js @@ -1168,4 +1168,16 @@ $(function () { }, new Error('`selector` option must be specified when initializing tooltip on the window.document object!')) }) + QUnit.test('should do nothing when an attempt is made to hide an uninitialized tooltip', function (assert) { + assert.expect(1) + + var $tooltip = $('some text') + .appendTo('#qunit-fixture') + .on('hidden.bs.tooltip shown.bs.tooltip', function () { + assert.ok(false, 'should not fire any tooltip events') + }) + .bootstrapTooltip('hide') + assert.strictEqual($tooltip.data('bs.tooltip'), undefined, 'should not initialize the tooltip') + }) + }) -- cgit v1.2.3