Update dependencies (#10666)

package.json

@@ -37,7 +37,7 @@
     "appium-android-driver": "2.x",
     "appium-base-driver": "^2.20.2",
     "appium-espresso-driver": "^1.0.0-beta.3",
-    "appium-fake-driver": "^0.3.0",
+    "appium-fake-driver": "^0.x",
     "appium-ios-driver": "2.x",
     "appium-mac-driver": "1.x",
     "appium-selendroid-driver": "1.x",
@@ -50,14 +50,13 @@
     "async-lock": "^1.0.0",
     "asyncbox": "2.x",
     "babel-runtime": "=5.8.24",
-    "bluebird": "2.x",
+    "bluebird": "3.x",
     "continuation-local-storage": "3.x",
-    "dateformat": "^2.0.0",
-    "fkill": "^5.2.0",
+    "dateformat": "^3.0.3",
     "lodash": "4.x",
-    "npmlog": "2.x",
+    "npmlog": "4.x",
     "request": "^2.81.0",
-    "request-promise": "1.x",
+    "request-promise": "4.x",
     "semver": "^5.5.0",
     "source-map-support": "0.x",
     "teen_process": "1.x",
@@ -86,8 +85,8 @@
     "babel-cli": "^6.26.0",
     "babel-eslint": "7.x",
     "babel-preset-env": "^1.6.0",
-    "chai": "3.x",
-    "chai-as-promised": "5.x",
+    "chai": "4.x",
+    "chai-as-promised": "7.x",
     "eslint": "3.x",
     "eslint-config-appium": "2.x",
     "eslint-plugin-babel": "3.x",
@@ -96,11 +95,11 @@
     "eslint-plugin-promise": "3.x",
     "gulp": "3.x",
     "handlebars": "^4.0.10",
-    "mocha": "2.x",
+    "mocha": "5.x",
     "pre-commit": "1.x",
     "replace-ext": "^1.0.0",
-    "sinon": "1.x",
-    "validate.js": "^0.11.1",
+    "sinon": "5.x",
+    "validate.js": "^0.12.0",
     "wd": "1.x",
     "yaml-js": "^0.2.0"
   },

test/config-specs.js

@@ -102,7 +102,7 @@ describe('Config', function () {
         spy = sinon.spy(logger, "warn");
       });
       beforeEach(function () {
-        spy.reset();
+        spy.resetHistory();
       });
       it('should log a warning if node is below 8', function () {
         process.version = 'v7.10.1';

test/driver-e2e-specs.js

@@ -118,7 +118,7 @@ describe('FakeDriver - via HTTP', function () {
       stacktrace.should.match(/FakeDriver.executeCommand/);
 
       // End session
-      await request.delete({url: `${baseUrl}/${value.sessionId}`}).should.eventually.be.resolved;
+      await request.delete({url: `${baseUrl}/${value.sessionId}`});
     });
 
     it('should reject invalid W3C capabilities and respond with a 400 Bad Parameters error', async function () {
@@ -157,7 +157,7 @@ describe('FakeDriver - via HTTP', function () {
       });
 
       // End session
-      await request.delete({ url: `${baseUrl}/${value.sessionId}` }).should.eventually.be.resolved;
+      await request.delete({ url: `${baseUrl}/${value.sessionId}` });
     });
 
     it('should accept a combo of W3C and JSONWP but use JSONWP if desiredCapabilities contains extraneous keys', async function () {
@@ -186,7 +186,7 @@ describe('FakeDriver - via HTTP', function () {
       });
 
       // End session
-      await request.delete({ url: `${baseUrl}/${value.sessionId}` }).should.eventually.be.resolved;
+      await request.delete({ url: `${baseUrl}/${sessionId}` });
     });
 
     it('should reject bad W3C capabilities with a BadParametersError (400)', async function () {
@@ -220,7 +220,7 @@ describe('FakeDriver - via HTTP', function () {
       value.capabilities.should.deep.equal(caps);
 
       // End session
-      await request.delete({ url: `${baseUrl}/${value.sessionId}` }).should.eventually.be.resolved;
+      await request.delete({ url: `${baseUrl}/${value.sessionId}` });
     });
 
     it('should fall back to MJSONWP if w3c caps are invalid', async function () {
@@ -242,7 +242,7 @@ describe('FakeDriver - via HTTP', function () {
       value.should.deep.equal(caps);
 
       // End session
-      await request.delete({ url: `${baseUrl}/${value.sessionId}` }).should.eventually.be.resolved;
+      await request.delete({ url: `${baseUrl}/${sessionId}` });
     });
 
     it('should fall back to MJSONWP if Inner Driver is not ready for W3C', async function () {
@@ -257,7 +257,7 @@ describe('FakeDriver - via HTTP', function () {
           },
         },
       };
-      const createSessionStub = sinon.stub(FakeDriver.prototype, 'createSession', async function (jsonwpCaps) {
+      const createSessionStub = sinon.stub(FakeDriver.prototype, 'createSession').callsFake(async function (jsonwpCaps) {
         const res = await BaseDriver.prototype.createSession.call(this, jsonwpCaps);
         this.protocol.should.equal('MJSONWP');
         return res;
@@ -307,7 +307,8 @@ describe('FakeDriver - via HTTP', function () {
       w3cPayload.value.should.eql(caps);
 
       // End session
-      await request.delete({url: `${baseUrl}/${value.sessionId}`}).should.eventually.be.resolved;
+      await request.delete({url: `${baseUrl}/${mjsonwpSessId}`});
+      await request.delete({url: `${baseUrl}/${w3cSessId}`});
     });
   });
 });
@@ -315,7 +316,7 @@ describe('FakeDriver - via HTTP', function () {
 describe('Logsink', function () {
   let server = null;
   let logs = [];
-  let logHandler = (level, message) => {
+  let logHandler = function (level, message) {
     logs.push([level, message]);
   };
   let args = {

test/driver-specs.js

@@ -197,7 +197,7 @@ describe('AppiumDriver', function () {
       });
       it('should return an empty array of sessions', async function () {
         sessions = await appium.getSessions();
-        sessions.should.be.an.array;
+        sessions.should.be.an('array');
         sessions.should.be.empty;
       });
       it('should return sessions created', async function () {
@@ -205,7 +205,7 @@ describe('AppiumDriver', function () {
         let session2 = (await appium.createSession(_.extend(_.clone(BASE_CAPS), {cap: 'other value'}))).value;
 
         sessions = await appium.getSessions();
-        sessions.should.be.an.array;
+        sessions.should.be.an('array');
         sessions.should.have.length(2);
         sessions[0].id.should.equal(session1[0]);
         sessions[0].capabilities.should.eql(session1[1]);