Update grunt-contrib-qunit to latest version (#5530)

We needed to define the `qunit` module in the unit tests because there was a change in grunt-contrib-qunit 0.6.0 that breaks when you define an AMD loader. It expects that the AMD loader is also used to load QUnit, instead of just being used to support the tests, so if you don't define the qunit module it will just hang and do nothing. Luckily we have the helpers file to help us out here, since it allows us to globally define this module.
select2 · Jun 5, 2019 · 5d2fdd7 · 5d2fdd7
1 parent 70ca392
commit 5d2fdd7
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 1 deletion.
diff --git a/package.json b/package.json
@@ -53,7 +53,7 @@
     "grunt-contrib-concat": "^1.0.1",
     "grunt-contrib-connect": "^2.0.0",
     "grunt-contrib-jshint": "^1.1.0",
-    "grunt-contrib-qunit": "^0.5.2",
+    "grunt-contrib-qunit": "^1.3.0",
     "grunt-contrib-requirejs": "^1.0.0",
     "grunt-contrib-uglify": "~4.0.1",
     "grunt-contrib-watch": "~1.1.0",

diff --git a/tests/helpers.js b/tests/helpers.js
@@ -44,3 +44,7 @@ QUnit.testStart(function(testDetails){
     }
   });
 });
+
+define('qunit', function () {
+  return QUnit;
+})