chore: Updated dev dependencies

package.json

@@ -84,7 +84,7 @@
   },
   "devDependencies": {
     "babel-core": "6.26.0",
-    "babel-eslint": "8.2.2",
+    "babel-eslint": "8.2.5",
     "babel-istanbul": "0.12.2",
     "babel-istanbul-loader": "0.1.0",
     "babel-loader": "7.1.3",
@@ -94,38 +94,38 @@
     "babel-plugin-transform-runtime": "6.15.0",
     "babel-preset-es2015": "6.24.1",
     "babel-preset-stage-2": "6.24.1",
-    "chai": "3.5.0",
+    "chai": "4.1.2",
     "chai-as-promised": "7.1.1",
-    "conventional-changelog-cli": "1.3.13",
+    "conventional-changelog-cli": "2.0.1",
     "conventional-changelog-lint": "2.1.1",
     "copy-dir": "0.3.0",
-    "coveralls": "2.13.3",
+    "coveralls": "3.0.2",
     "deepcopy": "0.6.3",
     "doctoc": "1.3.1",
-    "eslint": "4.18.1",
+    "eslint": "5.0.1",
     "eslint-plugin-async-await": "0.0.0",
-    "eslint-plugin-flowtype": "2.46.1",
-    "eslint-plugin-import": "2.8.0",
-    "flow-bin": "0.68.0",
-    "grunt": "1.0.2",
+    "eslint-plugin-flowtype": "2.49.3",
+    "eslint-plugin-import": "2.13.0",
+    "flow-bin": "0.75.0",
+    "grunt": "1.0.3",
     "grunt-contrib-clean": "1.1.0",
     "grunt-contrib-copy": "1.0.0",
-    "grunt-contrib-watch": "1.0.0",
-    "grunt-coveralls": "1.0.1",
-    "grunt-eslint": "20.1.0",
+    "grunt-contrib-watch": "1.1.0",
+    "grunt-coveralls": "2.0.0",
+    "grunt-eslint": "21.0.0",
     "grunt-flowbin": "0.2.0",
     "grunt-mocha-test": "0.13.3",
     "grunt-newer": "1.3.0",
-    "grunt-webpack": "3.0.2",
+    "grunt-webpack": "3.1.2",
     "load-grunt-configs": "1.0.0",
-    "load-grunt-tasks": "3.5.2",
-    "mocha": "4.1.0",
-    "mocha-multi": "0.11.1",
+    "load-grunt-tasks": "4.0.0",
+    "mocha": "5.2.0",
+    "mocha-multi": "1.0.1",
     "nsp": "3.2.1",
     "object.entries": "1.0.4",
     "object.values": "1.0.4",
     "prettyjson": "1.2.1",
-    "sinon": "4.4.2",
+    "sinon": "6.1.2",
     "webpack": "3.11.0",
     "webpack-dev-server": "2.11.1",
     "yauzl": "2.9.1"

tests/unit/test-cmd/test.run.js

@@ -115,10 +115,12 @@ describe('run', () => {
 
     // The runner should receive the same parameters as the options sent
     // to run() with just a few minor adjustments.
-    const expectedRunnerParams = { ...runOptions };
-    expectedRunnerParams.firefoxBinary = runOptions.firefox;
+    const expectedRunnerParams = {
+      ...runOptions,
+      firefoxBinary: runOptions.firefox,
+      customPrefs: runOptions.pref,
+    };
     delete expectedRunnerParams.firefox;
-    expectedRunnerParams.customPrefs = runOptions.pref;
     delete expectedRunnerParams.pref;
 
     assert.deepEqual({

tests/unit/test-extension-runners/test.extension-runners.js

@@ -418,7 +418,6 @@ describe('util/extension-runners', () => {
       sinon.assert.calledWith(
         createWatcher,
         sinon.match({
-          client: sentArgs.client,
           sourceDir: sentArgs.sourceDir,
           artifactsDir: sentArgs.artifactsDir,
           ignoreFiles: sentArgs.ignoreFiles,

tests/unit/test-firefox/test.preferences.js

@@ -92,13 +92,13 @@ describe('firefox/preferences', () => {
       assert.throws(
         () => coerceCLICustomPreference(['test.invalid.prop']),
         UsageError,
-        'UsageError: Incomplete custom preference: "test.invalid.prop". ' +
+        'Incomplete custom preference: "test.invalid.prop". ' +
         'Syntax expected: "prefname=prefvalue".'
       );
 
       assert.throws(() => coerceCLICustomPreference(['*&%£=true']),
                     UsageError,
-                    'UsageError: Invalid custom preference name: *&%£');
+                    'Invalid custom preference name: *&%£');
     });
 
   });

tests/unit/test.config.js

@@ -400,7 +400,7 @@ describe('config', () => {
       };
       assert.throws(() => {
         applyConf({...params, configObject});
-      }, UsageError, 'UsageError: The config option "source-dir" must be ' +
+      }, UsageError, 'The config option "source-dir" must be ' +
         'specified in camel case: "sourceDir"');
     });
 
@@ -418,7 +418,7 @@ describe('config', () => {
       };
       assert.throws(() => {
         applyConf({...params, configObject});
-      }, UsageError, 'UsageError: The config file ' +
+      }, UsageError, 'The config file ' +
       'at some/path/to/config.js specified an unknown option: "randomDir"');
     });
 
@@ -437,7 +437,7 @@ describe('config', () => {
       assert.throws(
         () => applyConf({...params, configObject}),
         UsageError,
-        'UsageError: The config file at some/path/to/config.js specified the ' +
+        'The config file at some/path/to/config.js specified the ' +
         'type of "retries" incorrectly as "string" (expected type "number")'
       );
     });
@@ -456,7 +456,7 @@ describe('config', () => {
       };
       assert.throws(() => {
         applyConf({...params, configObject});
-      }, UsageError, 'UsageError: The config file at some/path/to/config.js ' +
+      }, UsageError, 'The config file at some/path/to/config.js ' +
         'specified the type of "sourceDir" incorrectly');
     });
 
@@ -689,7 +689,7 @@ describe('config', () => {
       assert.throws(
         () => applyConf({...params, configObject}),
         UsageError,
-        'UsageError: The config file at some/path/to/config.js ' +
+        'The config file at some/path/to/config.js ' +
         'specified the type of "apiUrl" incorrectly'
       );
     });
@@ -714,7 +714,7 @@ describe('config', () => {
       };
       assert.throws(() => {
         applyConf({...params, configObject});
-      }, UsageError, 'UsageError: The config option "api-url"' +
+      }, UsageError, 'The config option "api-url"' +
         ' must be specified in camel case: "apiUrl"');
     });
 
@@ -738,7 +738,7 @@ describe('config', () => {
       };
       assert.throws(() => {
         applyConf({...params, configObject});
-      }, UsageError, 'UsageError: The config file at ' +
+      }, UsageError, 'The config file at ' +
       'some/path/to/config.js specified an unknown option: "randomOption"');
     });
 
@@ -762,7 +762,7 @@ describe('config', () => {
       };
       assert.throws(() => {
         applyConf({...params, configObject});
-      }, UsageError, 'UsageError: The config file at some/path/to/config.js ' +
+      }, UsageError, 'The config file at some/path/to/config.js ' +
         'specified the type of "apiUrl" incorrectly');
     });
 
@@ -794,7 +794,7 @@ describe('config', () => {
       };
       assert.throws(() => {
         applyConf({...params, configObject});
-      }, UsageError, 'UsageError: The config file at some/path/to/config.js ' +
+      }, UsageError, 'The config file at some/path/to/config.js ' +
         'specified the type of "apiUrl" incorrectly');
     });
 
@@ -821,7 +821,7 @@ describe('config', () => {
           applyConf({...params, configObject});
         },
         WebExtError,
-        'WebExtError: Option: apiUrl was defined without a type.'
+        'Option: apiUrl was defined without a type.'
       );
     });
 
@@ -855,7 +855,7 @@ describe('config', () => {
           argv: program.yargs.exitProcess(false).argv,
           options: program.options,
           configObject});
-      }, UsageError, 'UsageError: The config file at some/path/to/config.js ' +
+      }, UsageError, 'The config file at some/path/to/config.js ' +
         'specified the type of "apiUrl" incorrectly as "number"' +
         ' (expected type "string")');
     });