breaking(typings): Update typings and remove patch since we are now using selenium 3.x

Author: NickTomlin

README.md

@@ -11,18 +11,8 @@ You can view the full list of commands in [`lib/index.ts`](lib/index.ts#L8).
 Usage
 -----
 
-If you are using a version of `selenium-webdriver` below `3.0.0-beta-1`, you
-must use the `patch()` function before you create your webdriver instance:
+Use WebDriver JS Extender's `extend` function on your webdriver instance:
 
-```js
-require('webdriver-js-extender').patch(
-    require('selenium-webdriver/lib/command'),
-    require('selenium-webdriver/executors'),
-    require('selenium-webdriver/http'));
-```
-
-Once you've patched `selenium-webdriver` (or if you're using version `3.x`), all
-you need to do is run the `extend` function on your webdriver instance:
 
 ```js
   var extendedWebdriver = require('webdriver-js-extender').extend(webdriver);

lib/index.ts

@@ -155,34 +155,3 @@ export function extend(baseDriver: WebDriver, fallbackGracefully = false): Exten
 
   return extendedDriver;
 }
-
-/**
- * Patches webdriver so that the extender can define new commands.
- *
- * @example
- * patch(require('selenium-webdriver/lib/command'),
- *     require('selenium-webdriver/executors'),
- *     require('selenium-webdriver/http'));
- *
- * @param {*} lib_command The object at 'selenium-webdriver/lib/command'
- * @param {*} executors The object at 'selenium-webdriver/executors'
- * @param {*} http The object at 'selenium-webdriver/http'
- */
-export function patch(lib_command: any, executors: any, http: any) {
-  if (lib_command.DeferredExecutor === undefined) {
-    throw new Error(
-        'The version of `selenium-webdriver` you provided does ' +
-        'not use Deferred Executors.  Are you using version 3.x or above? If ' +
-        'so, you do not need to call the `patch()` function.');
-  }
-  lib_command.DeferredExecutor = DeferredExecutor;
-  executors.DeferredExecutor = DeferredExecutor;
-  // Based off of
-  // https://github.com/SeleniumHQ/selenium/blob/selenium-2.53.0/javascript/node/selenium-webdriver/executors.js#L45
-  executors.createExecutor = (url: any, opt_agent?: any, opt_proxy?: any) => {
-    return new DeferredExecutor(wdpromise.when(url, (url: any) => {
-      var client = new http.HttpClient(url, opt_agent, opt_proxy);
-      return new http.Executor(client);
-    }));
-  };
-}

package.json

@@ -22,8 +22,8 @@
   "license": "MIT",
   "typings": "built/lib/index.d.ts",
   "dependencies": {
-    "@types/selenium-webdriver": "^2.53.35",
-    "selenium-webdriver": "^2.53.2"
+    "@types/selenium-webdriver": "^3.0.0",
+    "selenium-webdriver": "^3.0.1"
   },
   "devDependencies": {
     "@types/jasmine": "^2.5.37",

spec/command_tests/helpers.ts

@@ -5,7 +5,7 @@ import {MockAppium} from '../mock-server';
 import {Session, CommandList} from '../mock-server/interfaces';
 import {session as commandList} from '../mock-server/commands';
 import {Command} from 'selenium-mock';
-import {extend, patch, ExtendedWebDriver} from '../../lib';
+import {extend, ExtendedWebDriver} from '../../lib';
 let portfinder = require('portfinder');
 
 
@@ -28,9 +28,6 @@ export function initMockSeleniumStandaloneServerAndGetDriverFactory(annotateComm
   let server: MockAppium;
   let port: number;
   beforeAll((done) => {
-    patch(require('selenium-webdriver/lib/command'),
-        require('selenium-webdriver/executors'),
-        require('selenium-webdriver/http'));
     portfinder.getPort((err: Error, p: number) => {
       if (err) {
         done.fail(err);

spec/command_tests/normal_spec.ts

@@ -106,7 +106,7 @@ describe('normal tests', () => {
     }).then((fileContents) => {
       expect(fileContents).toBe('bar');
       return driver.pullFolderFromDevice('/tmp/wd_js_ext/folder');
-    }).then((folderContents) => {
+    }).then((folderContents: any) => {
       expect(folderContents['a.txt']).toBe('x');
       expect(folderContents['b.txt']).toBe('y');
       expect(folderContents['c.txt']).toBe('z');

spec/index_spec.ts

@@ -1,5 +1,5 @@
 import * as webdriver from 'selenium-webdriver';
-import {extend, patch} from '../lib';
+import {extend} from '../lib';
 import {DeferredExecutor} from '../lib/deferred_executor';
 import {buildMockDriver, Data} from './mockdriver';
 
@@ -28,12 +28,4 @@ describe('extender', () => {
       done();
     });
   });
-
-  it('should patch selenium-webdriver', () => {
-    patch(require('selenium-webdriver/lib/command'),
-        require('selenium-webdriver/executors'),
-        require('selenium-webdriver/http'));
-    expect(require('selenium-webdriver/executors').createExecutor(
-      'http://localhost')).toEqual(jasmine.any(DeferredExecutor));
-  });
 });

spec/mockdriver.ts

@@ -1,5 +1,5 @@
 import * as webdriver from 'selenium-webdriver';
-let buildPath = require('selenium-webdriver/http').buildPath;
+let buildPath = require('selenium-webdriver/lib/http').buildPath;
 
 export interface Data {
   sessionId: string;