Merge pull request #829 from OpenGeoscience/use-karma-spec-reporter

Switch test progress to use the karma-spec-reporter.

.travis.yml

@@ -39,8 +39,13 @@ before_install:
   - "/sbin/start-stop-daemon --start --quiet --pidfile /tmp/custom_xvfb_99.pid --make-pidfile --background --exec /usr/bin/Xvfb -- :99 -ac -screen 0 1280x1024x24"
   - CACHE="${HOME}/cache" CMAKE_VERSION=3.5.0 CMAKE_SHORT_VERSION=3.5 source ./scripts/install_cmake.sh
   - npm install -g npm@latest
-  - npm install -g node-pre-gyp
-  - npm prune
+  # Prune the npm packages.  If this fails for any reason, just remove them all
+  - npm prune || rm -r node_modules
+
+install:
+  # travis moved to "npm ci", but that fails with canvas-prebuilt as it tries
+  # to install it before node-pre-gyp
+  - npm install
 
 before_script:
   - export DISPLAY=:99.0

karma-base.js

@@ -323,9 +323,11 @@ module.exports = function (config) {
     },
     browserNoActivityTimeout: 30000,
     reporters: [
-      'progress',
+      'spec',  // we had used the 'progress' reporter in the past.
       'kjhtml'
     ],
+    // We could suppress passing results
+    // specReporter = {suppressPassed: true, suppressSkipped: true},
     middleware: [
       'notes',
       'osmtiles'

karma-cov.conf.js

@@ -18,7 +18,7 @@ function subdir_name(b) {
     /* Use thes test case and as part of the name so that different tests end
      * up in different coverage directories.  Get the last element of the path
      * without the extension from the test case for this purpose. */
-    var parts = /^(.+\/)*(([^\/]+)\.[^\/.]*|[^\/.]+)$/.exec(process.env.GEOJS_TEST_CASE);
+    var parts = /^(.+\/)*(([^/]+)\.[^/.]*|[^/.]+)$/.exec(process.env.GEOJS_TEST_CASE);
     if (parts) {
       subdir += '_' + (parts[3] || parts[2]);
     }
@@ -29,7 +29,9 @@ function subdir_name(b) {
 module.exports = function (config) {
   var karma_config = require('./karma-base')(config);
 
-  karma_config.reporters = ['progress', 'coverage'];
+  karma_config.reporters = ['spec', 'coverage'];
+  /* Suppress listing passing and skipped tests when getting coverage. */
+  karma_config.specReporter = {suppressPassed: true, suppressSkipped: true};
   karma_config.coverageReporter = {
     reporters: [
       {type: 'html', dir: 'dist/coverage/', subdir: subdir_name},