Merge pull request #10099 from alalek:fix_js_tests

modules/js/test/package.json

@@ -3,7 +3,7 @@
   "description": "Tests for opencv js bindings",
   "version": "1.0.0",
   "dependencies" : {
-    "qunit" : "latest"
+    "node-qunit" : "latest"
   },
   "devDependencies": {
     "eslint" : "latest",
@@ -18,7 +18,7 @@
   },
   "keywords": [],
   "author": "",
-  "license": "BSD-4-Clause",
+  "license": "BSD-3-Clause",
   "bugs": {
     "url": "https://github.com/opencv/opencv/issues"
   },

modules/js/test/tests.js

@@ -38,7 +38,7 @@
 // the use of this software, even if advised of the possibility of such damage.
 //
 
-let testrunner = require('qunit');
+let testrunner = require('node-qunit');
 testrunner.options.maxBlockDuration = 20000; // cause opencv_js.js need time to load
 
 testrunner.run(
@@ -49,5 +49,10 @@ testrunner.run(
     },
     function(err, report) {
         console.log(report.failed + ' failed, ' + report.passed + ' passed');
+        if (report.failed) {
+            process.on('exit', function() {
+                process.exit(1);
+            });
+        }
     }
 );