tasks: centralise all required docker-related tasks in 1 script

- npm run docker -- <pull | run | setup | stop | remove>
  will now handle all docker-related tasks

Author: etpinard

circle.yml

@@ -11,12 +11,12 @@ machine:
 
 dependencies:
   pre:
-    - docker pull plotly/testbed:latest
+    - eval $(node tasks/docker.js pull)
   post:
-    - eval $(node tasks/run_docker.js)
+    - eval $(node tasks/docker.js run)
     - npm run cibuild
     - npm run pretest
-    - eval $(node tasks/setup_docker.js)
+    - eval $(node tasks/docker.js setup)
 
 test:
   override:

package.json

@@ -30,6 +30,7 @@
     "watch": "node tasks/watch.js",
     "lint": "eslint . || true",
     "lint-fix": "eslint . --fix",
+    "docker": "node tasks/docker.js",
     "pretest": "node tasks/pretest.js",
     "test-jasmine": "karma start test/jasmine/karma.conf.js",
     "citest-jasmine": "karma start test/jasmine/karma.ciconf.js",

tasks/docker.js

@@ -0,0 +1,57 @@
+var constants = require('./util/constants');
+var common = require('./util/common');
+var containerCommands = require('./util/container_commands');
+
+var isCI = process.env.CIRCLECI;
+var arg = process.argv[2];
+
+var msg, cmd, cb, errorCb;
+
+switch(arg) {
+
+    case 'pull':
+        msg = 'Pulling latest docker image';
+        cmd = 'docker pull ' + constants.testContainerImage;
+        break;
+
+    case 'run':
+        msg = 'Booting up ' + constants.testContainerName + ' docker container';
+        cmd = containerCommands.dockerRun;
+
+        // if docker-run fails, try docker-start.
+        errorCb = function(err) {
+            if(err) common.execCmd('docker start ' + constants.testContainerName);
+        };
+
+        break;
+
+    case 'setup':
+        msg = 'Setting up ' + constants.testContainerName + ' docker container for testing';
+        cmd = containerCommands.getRunCmd(isCI, containerCommands.setup);
+        break;
+
+    case 'stop':
+        msg = 'Stopping ' + constants.testContainerName + ' docker container';
+        cmd = 'docker stop ' + constants.testContainerName;
+        break;
+
+    case 'remove':
+        msg = 'Removing ' + constants.testContainerName + ' docker container';
+        cmd = 'docker rm ' + constants.testContainerName;
+        break;
+
+    default:
+        console.log('Usage: pull, run, setup, stop, remove');
+        process.exit(0);
+        break;
+}
+
+// Log command string on CircleCI, to then `eval` them,
+// which appears to be more reliable then calling `child_process.exec()`
+if(isCI) {
+    console.log(cmd);
+}
+else {
+    console.log(msg);
+    common.execCmd(cmd, cb, errorCb);
+}

tasks/run_docker.js

@@ -75,6 +75,7 @@ module.exports = {
     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',
     testContainerPort: '9010',
     testContainerUrl: 'http://localhost:9010/',