use karma-jasmine-spec-tags to skip a few test on CI

- unfortunately, still some test need to be skipped on CI
  e.g. mapbox-gl takes too much resources ant blows up the run
  and similarly for some parcoords tests.

Author: etpinard

package.json

@@ -119,6 +119,7 @@
     "karma-coverage": "^1.0.0",
     "karma-firefox-launcher": "^1.0.0",
     "karma-jasmine": "^1.1.0",
+    "karma-jasmine-spec-tags": "^1.0.1",
     "madge": "^1.6.0",
     "node-sass": "^4.5.0",
     "npm-link-check": "^1.2.0",

test/jasmine/karma.conf.js

@@ -54,7 +54,7 @@ func.defaultConfig = {
 
     // frameworks to use
     // available frameworks: https://npmjs.org/browse/keyword/karma-adapter
-    frameworks: ['jasmine', 'browserify'],
+    frameworks: ['jasmine', 'jasmine-spec-tags', 'browserify'],
 
     // list of files / patterns to load in the browser
     //
@@ -119,6 +119,15 @@ func.defaultConfig = {
         extensions: ['.js'],
         watch: !isCI,
         debug: true
+
+    // unfortunately a few tests don't behave well on CI
+    // using `karma-jasmine-spec-tags`
+    // add @noCI to the spec description to skip a spec on CI
+    client: {
+        tagPrefix: '@',
+        skipTags: isCI ? 'noCI' : null
+    },
+
     }
 };
 

test/jasmine/tests/gl2d_click_test.js

@@ -16,7 +16,7 @@ Plotly.register([
     require('@lib/contourgl')
 ]);
 
-describe('Test hover and click interactions', function() {
+describe('@noCI Test hover and click interactions', function() {
 
     var mock = require('@mocks/gl2d_14.json');
     var mock2 = require('@mocks/gl2d_pointcloud-basic.json');

test/jasmine/tests/mapbox_test.js

@@ -276,7 +276,7 @@ describe('mapbox credentials', function() {
     }, LONG_TIMEOUT_INTERVAL);
 });
 
-describe('mapbox plots', function() {
+describe('@noCI, mapbox plots', function() {
     'use strict';
 
     var mock = require('@mocks/mapbox_0.json'),

test/jasmine/tests/parcoords_test.js

@@ -219,7 +219,7 @@ describe('parcoords initialization tests', function() {
     });
 });
 
-describe('parcoords', function() {
+describe('@noCI parcoords', function() {
 
     beforeAll(function() {
         mock.data[0].dimensions.forEach(function(d) {

test/jasmine/tests/scattermapbox_test.js

@@ -451,7 +451,7 @@ describe('scattermapbox convert', function() {
     }
 });
 
-describe('scattermapbox hover', function() {
+describe('@noCI scattermapbox hover', function() {
     'use strict';
 
     var hoverPoints = ScatterMapbox.hoverPoints;