Merge pull request #816 from plotly/scattermapbox-in-main-bundle

Include `scattermapbox` in main bundle

Author: etpinard

lib/index.js

@@ -25,7 +25,8 @@ Plotly.register([
     require('./scattergeo'),
     require('./choropleth'),
     require('./scattergl'),
-    require('./scatterternary')
+    require('./scatterternary'),
+    require('./scattermapbox')
 ]);
 
 module.exports = Plotly;

test/image/baselines/mapbox_0.png

@@ -55,10 +55,10 @@ if(mockList.length === 0) {
     throw new Error('No mocks found with pattern ' + pattern);
 }
 
-mockList = mockList.filter(untestableFilter);
-
-if(mockList.length === 0) {
-    throw new Error('All mocks found with pattern ' + pattern + ' are currently untestable');
+// filter out untestable mocks if no pattern is specified
+if(!pattern) {
+    console.log('Filtering out untestable mocks\n');
+    mockList = mockList.filter(untestableFilter);
 }
 
 // main

test/image/baselines/mapbox_angles.png

@@ -17,11 +17,6 @@ var MOUSE_DELAY = 100;
 
 var noop = function() {};
 
-// until it is part of the main plotly.js bundle
-Plotly.register(
-    require('@lib/scattermapbox')
-);
-
 Plotly.setPlotConfig({
     mapboxAccessToken: MAPBOX_ACCESS_TOKEN
 });

test/image/baselines/mapbox_connectgaps.png

@@ -2,9 +2,6 @@ var Plotly = require('@lib/index');
 var Lib = require('@src/lib');
 
 Plotly.register([
-    // until it is part of the main plotly.js bundle
-    require('@lib/scattermapbox'),
-
     // until they become official
     require('../assets/transforms/filter')
 ]);

test/image/baselines/mapbox_custom-style.png

@@ -10,11 +10,6 @@ var destroyGraphDiv = require('../assets/destroy_graph_div');
 var hasWebGLSupport = require('../assets/has_webgl_support');
 var customMatchers = require('../assets/custom_matchers');
 
-// until it is part of the main plotly.js bundle
-Plotly.register(
-    require('@lib/scattermapbox')
-);
-
 Plotly.setPlotConfig({
     mapboxAccessToken: require('@build/credentials.json').MAPBOX_ACCESS_TOKEN
 });