make npm run test-image -- mapbox_* work again

- the latest imagetest image set MAPBOX_ACCESS_TOKEN in
  the image server main which makes the Plotly.setPlotConfig call
  in <script set_plot_config.js> irrelevant.
- replace this by a sed call which injects MAPBOX_ACCESS_TOKEN
  into the image server main

Author: etpinard

tasks/pretest.js

@@ -5,7 +5,6 @@ var common = require('./util/common');
 
 // main
 makeCredentialsFile();
-makeSetPlotConfigFile();
 makeTestImageFolders();
 makeRequireJSFixture();
 
@@ -20,24 +19,6 @@ function makeCredentialsFile() {
     logger('make build/credentials.json');
 }
 
-// Create a 'set plot config' file,
-// to be included in the image test index
-function makeSetPlotConfigFile() {
-    var setPlotConfig = [
-        '\'use strict\';',
-        '',
-        '/* global Plotly:false */',
-        '',
-        'Plotly.setPlotConfig({',
-        '    mapboxAccessToken: \'' + constants.mapboxAccessToken + '\'',
-        '});',
-        ''
-    ].join('\n');
-
-    common.writeFile(constants.pathToSetPlotConfig, setPlotConfig);
-    logger('make build/set_plot_config.js');
-}
-
 // Make artifact folders for image tests
 function makeTestImageFolders() {
 

tasks/util/constants.js

@@ -73,7 +73,6 @@ module.exports = {
     // more info: https://www.mapbox.com/help/define-access-token/
     mapboxAccessToken: 'pk.eyJ1IjoiZXRwaW5hcmQiLCJhIjoiY2luMHIzdHE0MGFxNXVubTRxczZ2YmUxaCJ9.hwWZful0U2CQxit4ItNsiQ',
     pathToCredentials: path.join(pathToBuild, 'credentials.json'),
-    pathToSetPlotConfig: path.join(pathToBuild, 'set_plot_config.js'),
 
     testContainerImage: 'plotly/testbed:latest',
     testContainerName: process.env.PLOTLYJS_TEST_CONTAINER_NAME || 'imagetest',

tasks/util/container_commands.js

@@ -3,7 +3,12 @@ var constants = require('./constants');
 var containerCommands = {
     cdHome: 'cd ' + constants.testContainerHome,
     cpIndex: 'cp -f test/image/index.html ../server_app/index.html',
-    restart: 'supervisorctl restart nw1',
+    injectEnv: [
+        'sed -i',
+        's/process.env.PLOTLY_MAPBOX_DEFAULT_ACCESS_TOKEN/\\\'' + constants.mapboxAccessToken + '\\\'/',
+        '../server_app/main.js'
+    ].join(' '),
+    restart: 'supervisorctl restart nw1'
 };
 
 containerCommands.ping = [
@@ -14,6 +19,7 @@ containerCommands.ping = [
 
 containerCommands.setup = [
     containerCommands.cpIndex,
+    containerCommands.injectEnv,
     containerCommands.restart,
     'sleep 1',
     containerCommands.ping,

test/image/index.html

@@ -5,7 +5,6 @@
 <script type="text/javascript" src="../plotly.js/dist/extras/mathjax/MathJax.js?config=TeX-AMS-MML_SVG"></script>
 <script type="text/javascript" src="../plotly.js/build/plotly.js" charset="utf-8"></script>
 <script type="text/javascript" src="../plotly.js/dist/plotly-geo-assets.js" charset="utf-8"></script>
-<script type="text/javascript" src="../plotly.js/build/set_plot_config.js"></script>
 <script type="text/javascript" src="./main.js"></script>
 </body>
 </html>