add test for bundles:

- allow test file glob in karma.conf to be overrode using
  a cli arg
- add test_bundle.js script that boots up one karma instance per
  test file found in jasmine/bundle_test/
- add two bundle tests

Author: etpinard

tasks/test_bundle.js

@@ -0,0 +1,20 @@
+var path = require('path');
+var exec = require('child_process').exec;
+
+var glob = require('glob');
+var constants = require('./util/constants');
+var pathToBundleTests = path.join(constants.pathToJasmineTests, '..', 'bundle_tests');
+
+
+glob(pathToBundleTests + '/*.js', function(err, files) {
+    files.forEach(function(file) {
+        var baseName = path.basename(file);
+        var cmd = 'npm run citest-jasmine -- bundle_tests/' + baseName;
+
+        exec(cmd, function(err, stdout) {
+            console.log(stdout);
+
+            if(err) throw err;
+        });
+    });
+});

test/jasmine/bundle_tests/bar_test.js

@@ -0,0 +1,34 @@
+var d3 = require('d3');
+
+var Plotly = require('@lib/core');
+var PlotlyBar = require('@lib/bar');
+
+var createGraphDiv = require('../assets/create_graph_div');
+var destroyGraphDiv = require('../assets/destroy_graph_div');
+
+
+describe('Bundle with bar', function() {
+    'use strict';
+
+    Plotly.register(PlotlyBar);
+
+    var mock = require('@mocks/bar_line.json');
+
+    beforeEach(function(done) {
+        Plotly.plot(createGraphDiv(), mock.data, mock.layout).then(done);
+    });
+
+    afterEach(destroyGraphDiv);
+
+    it('should graph scatter traces', function() {
+        var nodes = d3.selectAll('g.trace.scatter');
+
+        expect(nodes.size()).toEqual(1);
+    });
+
+    it('should graph bar traces', function() {
+        var nodes = d3.selectAll('g.trace.bars');
+
+        expect(nodes.size()).toEqual(1);
+    });
+});

test/jasmine/bundle_tests/core_test.js

@@ -0,0 +1,37 @@
+var d3 = require('d3');
+
+var Plotly = require('@lib/core');
+
+var createGraphDiv = require('../assets/create_graph_div');
+var destroyGraphDiv = require('../assets/destroy_graph_div');
+
+
+describe('Bundle with core only', function() {
+    'use strict';
+
+    var mock = require('@mocks/bar_line.json');
+
+    beforeEach(function(done) {
+        spyOn(console, 'warn');
+
+        Plotly.plot(createGraphDiv(), mock.data, mock.layout).then(done);
+    });
+
+    afterEach(destroyGraphDiv);
+
+    it('should graph scatter traces', function() {
+        var nodes = d3.selectAll('g.trace.scatter');
+
+        expect(nodes.size()).toEqual(mock.data.length);
+    });
+
+    it('should not graph bar traces', function() {
+        var nodes = d3.selectAll('g.trace.bars');
+
+        expect(nodes.size()).toEqual(0);
+    });
+
+    it('should warn users about unregistered bar trace type', function() {
+        expect(console.warn).toHaveBeenCalled();
+    });
+});

test/jasmine/karma.conf.js

@@ -1,5 +1,20 @@
+/*eslint-env node*/
+
 // Karma configuration
 
+/*
+ * Test file globs can be passed with an argument.
+ *
+ * Example:
+ *
+ *  $ npm run test-jasmine -- tests/axes_test.js
+ *
+ * will only run the tests in axes_test.js
+ *
+ */
+var testFileGlob = process.argv[4] ? process.argv[4] : 'tests/*_test.js';
+
+
 function func(config) {
 
     // level of logging
@@ -21,7 +36,7 @@ func.defaultConfig = {
     // list of files / patterns to load in the browser
     files: [
         'assets/jquery-1.8.3.min.js',
-        'tests/*_test.js'
+        testFileGlob
     ],
 
     // list of files to exclude
@@ -30,9 +45,9 @@ func.defaultConfig = {
 
     // preprocess matching files before serving them to the browser
     // available preprocessors: https://npmjs.org/browse/keyword/karma-preprocessor
-    preprocessors: {
-        'tests/*_test.js': ['browserify']
-    },
+    //
+    // N.B. this field is filled below
+    preprocessors: {},
 
     // test results reporter to use
     // possible values: 'dots', 'progress'
@@ -64,5 +79,7 @@ func.defaultConfig = {
     }
 };
 
+// browserify the test files
+func.defaultConfig.preprocessors[testFileGlob] = ['browserify'];
 
 module.exports = func;