chore: get the CI tests working again (#1182)

* chore: get the CI tests working again
 * an updated eslint-plugin-lodash was creating extra linter errors where we
   were using built-in String methods instead of the lodash equivalents
 * an updated codespell package now installs with a different executable-name
   (no .py extension), and now supports multiple dictionaries
 * while I was at it I replaced the messy double-quoting-string logic with the
   command-join module

Changelog-type: patch

* chore: downgrade codespell to 1.9.2 because the latest version has issues running on Windows
(we'll hopefully be able to revert this commit when codespell works again properly!)

.travis.yml

@@ -34,7 +34,7 @@ before_install:
 
 install:
   - if [[ "$TRAVIS_OS_NAME" == "osx" ]]; then
-      pip install codespell;
+      pip install codespell==1.9.2;
       npm install -g bower;
       brew install afsctool;
       brew install jq;

appveyor.yml

@@ -32,7 +32,7 @@ install:
   - set PATH=C:\Program Files (x86)\NSIS;%PATH%
   - set PATH=C:\MinGW\bin;%PATH%
   - set PATH=C:\MinGW\msys\1.0\bin;%PATH%
-  - pip install codespell
+  - pip install codespell==1.9.2
   - make info
   - make electron-develop
 

lib/child-writer/writer-proxy.js

@@ -18,6 +18,7 @@
 
 const Bluebird = require('bluebird');
 const childProcess = require('child_process');
+const commandJoin = require('command-join');
 const isElevated = Bluebird.promisify(require('is-elevated'));
 const ipc = require('node-ipc');
 const _ = require('lodash');
@@ -141,9 +142,7 @@ return isElevated().then((elevated) => {
       return commandPrefix.concat(process.argv);
     });
 
-    const command = _.join(_.map(commandArguments, (argument) => {
-      return `"${argument.replace(/(")/g, '\\$1')}"`;
-    }), ' ');
+    const command = commandJoin(commandArguments);
 
     // For debugging purposes
     console.log(`Running: ${command}`);

lib/gui/models/supported-formats.js

@@ -115,7 +115,7 @@ SupportedFormats.service('SupportedFormatsModel', function() {
    * }
    */
   this.isSupportedImage = (imagePath) => {
-    const extension = _.replace(path.extname(imagePath), '.', '').toLowerCase();
+    const extension = _.toLower(_.replace(path.extname(imagePath), '.', ''));
 
     if (_.some([
       _.includes(this.getNonCompressedExtensions(), extension),

lib/gui/modules/analytics.js

@@ -59,7 +59,7 @@ const analytics = angular.module(MODULE_NAME, [
  * }
  */
 const getHostArchitecture = () => {
-  return _.includes([ 'ia32', 'x64' ], process.arch) ? arch().replace('x86', 'ia32') : process.arch;
+  return _.includes([ 'ia32', 'x64' ], process.arch) ? _.replace(arch(), 'x86', 'ia32') : process.arch;
 };
 
 // Mixpanel integration

lib/image-stream/archive.js

@@ -172,7 +172,7 @@ exports.extractImage = (archive, hooks) => {
   return hooks.getEntries(archive).then((entries) => {
 
     const imageEntries = _.filter(entries, (entry) => {
-      const extension = _.replace(path.extname(entry.name), '.', '').toLowerCase();
+      const extension = _.toLower(_.replace(path.extname(entry.name), '.', ''));
       return _.includes(IMAGE_EXTENSIONS, extension);
     });
 

npm-shrinkwrap.json

@@ -213,6 +213,11 @@
       "from": "columnify@>=1.5.1 <2.0.0",
       "resolved": "https://registry.npmjs.org/columnify/-/columnify-1.5.4.tgz"
     },
+    "command-join": {
+      "version": "2.0.0",
+      "from": "command-join@latest",
+      "resolved": "https://registry.npmjs.org/command-join/-/command-join-2.0.0.tgz"
+    },
     "commander": {
       "version": "2.1.0",
       "from": "commander@>=2.1.0 <2.2.0",

package.json

@@ -74,6 +74,7 @@
     "bluebird": "^3.0.5",
     "bootstrap-sass": "^3.3.5",
     "chalk": "^1.1.3",
+    "command-join": "^2.0.0",
     "deep-map-keys": "^1.2.0",
     "drivelist": "^5.0.16",
     "electron-is-running-in-asar": "^1.0.0",

scripts/build/docker/Dockerfile-i686

@@ -29,4 +29,4 @@ RUN npm config set spin=false
 RUN npm install -g bower asar electron-installer-debian
 
 # Python
-RUN pip install codespell
+RUN pip install codespell==1.9.2

scripts/build/docker/Dockerfile-x86_64

@@ -29,4 +29,4 @@ RUN npm config set spin=false
 RUN npm install -g bower asar electron-installer-debian
 
 # Python
-RUN pip install codespell
+RUN pip install codespell==1.9.2

scripts/build/docker/Dockerfile.template

@@ -29,4 +29,4 @@ RUN npm config set spin=false
 RUN npm install -g bower asar electron-installer-debian
 
 # Python
-RUN pip install codespell
+RUN pip install codespell==1.9.2

tests/gui/components/svg-icon.spec.js

@@ -1,6 +1,7 @@
 'use strict';
 
 const m = require('mochainon');
+const _ = require('lodash');
 const fs = require('fs');
 const path = require('path');
 const angular = require('angular');
@@ -33,9 +34,9 @@ describe('Browser: SVGIcon', function() {
 
     it('should inline the svg contents in the element', function() {
       const icon = '../../../../../lib/gui/assets/etcher.svg';
-      let iconContents = fs.readFileSync(path.join(__dirname, '../../../lib/gui/assets/etcher.svg'), {
+      let iconContents = _.split(fs.readFileSync(path.join(__dirname, '../../../lib/gui/assets/etcher.svg'), {
         encoding: 'utf8'
-      }).split(/\r?\n/);
+      }), /\r?\n/);
 
       // Injecting XML as HTML causes the XML header to be commented out.
       // Modify here to ease assertions later on.

tests/gui/models/supported-formats.spec.js

@@ -80,7 +80,7 @@ describe('Browser: SupportedFormats', function() {
 
       it('should ignore casing when determining extension validity', function() {
         const nonCompressedExtension = _.first(SupportedFormatsModel.getNonCompressedExtensions());
-        const imagePath = `/path/to/foo.${nonCompressedExtension.toUpperCase()}`;
+        const imagePath = `/path/to/foo.${_.toUpper(nonCompressedExtension)}`;
         const isSupported = SupportedFormatsModel.isSupportedImage(imagePath);
         m.chai.expect(isSupported).to.be.true;
       });

tests/shared/utils.spec.js

@@ -125,7 +125,7 @@ describe('Shared: Utils', function() {
 
   });
 
-  describe('hideAbsolutePathsInObject()', function() {
+  describe('.hideAbsolutePathsInObject()', function() {
 
     it('should return undefined if given undefined', function() {
       m.chai.expect(utils.hideAbsolutePathsInObject(undefined)).to.be.undefined;