Add Sinon to do better unit test

js/tests/README.md

@@ -1,9 +1,9 @@
 ## How does Bootstrap's test suite work?
 
-Bootstrap uses [QUnit](https://qunitjs.com/), a powerful, easy-to-use JavaScript unit test framework. Each plugin has a file dedicated to its tests in `unit/<plugin-name>.js`.
+Bootstrap uses [QUnit](https://qunitjs.com/) and [Sinon](http://sinonjs.org/). Each plugin has a file dedicated to its tests in `unit/<plugin-name>.js`.
 
 * `unit/` contains the unit test files for each Bootstrap plugin.
-* `vendor/` contains third-party testing-related code (QUnit and jQuery).
+* `vendor/` contains third-party testing-related code (QUnit, jQuery and Sinon).
 * `visual/` contains "visual" tests which are run interactively in real browsers and require manual verification by humans.
 
 To run the unit test suite via [Karma](http://karma-runner.github.io/), run `npm run js-test`.
@@ -51,13 +51,17 @@ QUnit.test('should describe the unit being tested', function (assert) {
 
 // Asynchronous test
 QUnit.test('should describe the unit being tested', function (assert) {
-  assert.expect(1)
+  assert.expect(2)
   var done = assert.async()
 
-  $('<div title="tooltip title"></div>')
+  var $tooltip = $('<div title="tooltip title"></div>').bootstrapTooltip()
-    .appendTo('#qunit-fixture')
+  var tooltipInstance = $tooltip.data('bs.tooltip')
+  var spyShow = sinon.spy(tooltipInstance, 'show')
+
+  $tooltip.appendTo('#qunit-fixture')
     .on('shown.bs.tooltip', function () {
       assert.ok(true, '"shown" event was fired after calling "show"')
+      assert.ok(spyShow.called, 'show called')
       done()
     })
     .bootstrapTooltip('show')

js/tests/index.html

@@ -25,6 +25,9 @@
     <link rel="stylesheet" href="vendor/qunit.css" media="screen">
     <script src="vendor/qunit.js"></script>
 
+    <!-- Sinon -->
+    <script src="vendor/sinon.min.js"></script>
+
     <script>
       // Disable jQuery event aliases to ensure we don't accidentally use any of them
       [

js/tests/karma.conf.js

@@ -8,11 +8,12 @@ module.exports = (config) => {
 
   config.set({
     basePath: '../..',
-    frameworks: ['qunit', 'detectBrowsers'],
+    frameworks: ['qunit', 'sinon', 'detectBrowsers'],
     plugins: [
       'karma-chrome-launcher',
       'karma-firefox-launcher',
       'karma-qunit',
+      'karma-sinon',
       'karma-detect-browsers',
       'karma-coverage-istanbul-reporter'
     ],

js/tests/unit/.eslintrc.json

@@ -5,6 +5,7 @@
     "jquery": true
   },
   "globals": {
+    "sinon": false,
     "Util": false
   },
   "parserOptions": {

package.json

@@ -107,6 +107,7 @@
     "karma-detect-browsers": "^2.2.6",
     "karma-firefox-launcher": "^1.1.0",
     "karma-qunit": "^1.2.1",
+    "karma-sinon": "^1.0.5",
     "node-sass": "^4.7.2",
     "nodemon": "^1.17.1",
     "npm-run-all": "^4.1.2",
@@ -118,6 +119,7 @@
     "rollup-plugin-node-resolve": "^3.0.3",
     "shelljs": "^0.8.1",
     "shx": "^0.2.2",
+    "sinon": "^4.4.3",
     "sri-toolbox": "^0.2.0",
     "stylelint": "^9.1.1",
     "stylelint-config-recommended-scss": "^3.1.0",