selenium-webdriver 3.0.0-beta-1 → 3.0.1

package/lib/proxy.js

@@ -76,6 +76,34 @@ exports.manual = function(options) {
 };
 
 
+/**
+ * Creates a proxy configuration for a socks proxy.
+ *
+ * __Example:__
+ *
+ *     const {Capabilities} = require('selenium-webdriver');
+ *     const proxy = require('selenium-webdriver/lib/proxy');
+ *
+ *     let capabilities = new Capabilities();
+ *     capabilities.setProxy(proxy.socks('localhost:1234', 'bob', 'password'));
+ *
+ *
+ * @param {string} host The proxy host, in the form `hostname:port`.
+ * @param {string} username The user name to authenticate as.
+ * @param {string} password The password to authenticate with.
+ * @return {!ProxyConfig} A new proxy configuration object.
+ * @see https://en.wikipedia.org/wiki/SOCKS
+ */
+exports.socks = function(host, username, password) {
+  return /** @type {!ProxyConfig} */({
+    proxyType: 'manual',
+    socksProxy: host,
+    socksUsername: username,
+    socksPassword: password
+  });
+};
+
+
 /**
  * Configures WebDriver to configure the browser proxy using the PAC file at
  * the given URL.

package/lib/session.js

@@ -17,7 +17,7 @@
 
 'use strict';
 
-const Capabilities = require('./capabilities').Capabilities;
+const {Capabilities} = require('./capabilities');
 
 
 /**

package/lib/test/build.js

@@ -21,8 +21,7 @@ const spawn = require('child_process').spawn,
     fs = require('fs'),
     path = require('path');
 
-const isDevMode = require('../devmode'),
-    promise = require('../promise');
+const isDevMode = require('../devmode');
 
 var projectRoot = path.normalize(path.join(__dirname, '../../../../..'));
 
@@ -69,7 +68,7 @@ Build.prototype.onlyOnce = function() {
 
 /**
  * Executes the build.
- * @return {!webdriver.promise.Promise} A promise that will be resolved when
+ * @return {!Promise} A promise that will be resolved when
  *     the build has completed.
  * @throws {Error} If no targets were specified.
  */
@@ -86,7 +85,7 @@ Build.prototype.go = function() {
     });
 
     if (!targets.length) {
-      return promise.fulfilled();
+      return Promise.resolve();
     }
   }
 
@@ -100,31 +99,30 @@ Build.prototype.go = function() {
     cmd = path.join(projectRoot, 'go');
   }
 
-  var result = promise.defer();
-  spawn(cmd, args, {
-    cwd: projectRoot,
-    env: process.env,
-    stdio: ['ignore', process.stdout, process.stderr]
-  }).on('exit', function(code, signal) {
-    if (code === 0) {
-      targets.forEach(function(target) {
-        builtTargets[target] = 1;
-      });
-      return result.fulfill();
-    }
-
-    var msg = 'Unable to build artifacts';
-    if (code) {  // May be null.
-      msg += '; code=' + code;
-    }
-    if (signal) {
-      msg += '; signal=' + signal;
-    }
-
-    result.reject(Error(msg));
+  return new Promise((resolve, reject) => {
+    spawn(cmd, args, {
+      cwd: projectRoot,
+      env: process.env,
+      stdio: ['ignore', process.stdout, process.stderr]
+    }).on('exit', function(code, signal) {
+      if (code === 0) {
+        targets.forEach(function(target) {
+          builtTargets[target] = 1;
+        });
+        return resolve();
+      }
+
+      var msg = 'Unable to build artifacts';
+      if (code) {  // May be null.
+        msg += '; code=' + code;
+      }
+      if (signal) {
+        msg += '; signal=' + signal;
+      }
+
+      reject(Error(msg));
+    });
   });
-
-  return result.promise;
 };
 
 

package/lib/test/data/formPage.html

@@ -45,6 +45,7 @@ There should be a form here:
 
     <select name="no-select" disabled="disabled">
       <option value="foo">Foo</option>
+      <option value="bar">Bar</option>
     </select>
 
 		<select name="select_empty_multiple" multiple>

package/lib/test/fileserver.js

@@ -28,8 +28,7 @@ var serveIndex = require('serve-index');
 
 var Server = require('./httpserver').Server,
     resources = require('./resources'),
-    isDevMode = require('../devmode'),
-    promise = require('../promise');
+    isDevMode = require('../devmode');
 
 var WEB_ROOT = '/common';
 var JS_ROOT = '/javascript';
@@ -194,15 +193,18 @@ function sendDelayedResponse(request, response) {
 }
 
 
-function handleUpload(request, response, next) {
-  multer({
-    inMemory: true,
-    onFileUploadComplete: function(file) {
+function handleUpload(request, response) {
+  let upload = multer({storage: multer.memoryStorage()}).any();
+  upload(request, response, function(err) {
+    if (err) {
+      response.writeHead(500);
+      response.end(err + '');
+    } else {
       response.writeHead(200);
-      response.write(file.buffer);
+      response.write(request.files[0].buffer);
       response.end('<script>window.top.window.onUploadDone();</script>');
     }
-  })(request, response, function() {});
+  });
 }
 
 
@@ -269,7 +271,7 @@ function sendIndex(request, response) {
 /**
  * Starts the server on the specified port.
  * @param {number=} opt_port The port to use, or 0 for any free port.
- * @return {!webdriver.promise.Promise.<Host>} A promise that will resolve
+ * @return {!Promise<Host>} A promise that will resolve
  *     with the server host when it has fully started.
  */
 exports.start = server.start.bind(server);
@@ -277,7 +279,7 @@ exports.start = server.start.bind(server);
 
 /**
  * Stops the server.
- * @return {!webdriver.promise.Promise} A promise that will resolve when the
+ * @return {!Promise} A promise that will resolve when the
  *     server has closed all connections.
  */
 exports.stop = server.stop.bind(server);

package/lib/test/httpserver.js

@@ -50,14 +50,14 @@ var Server = function(requestHandler) {
    * Starts the server on the given port. If no port, or 0, is provided,
    * the server will be started on a random port.
    * @param {number=} opt_port The port to start on.
-   * @return {!webdriver.promise.Promise.<Host>} A promise that will resolve
+   * @return {!Promise<Host>} A promise that will resolve
    *     with the server host when it has fully started.
    */
   this.start = function(opt_port) {
     assert(typeof opt_port !== 'function',
            "start invoked with function, not port (mocha callback)?");
     var port = opt_port || portprober.findFreePort('localhost');
-    return promise.when(port, function(port) {
+    return Promise.resolve(port).then(port => {
       return promise.checkedNodeCall(
           server.listen.bind(server, port, 'localhost'));
     }).then(function() {
@@ -67,13 +67,11 @@ var Server = function(requestHandler) {
 
   /**
    * Stops the server.
-   * @return {!webdriver.promise.Promise} A promise that will resolve when the
+   * @return {!Promise} A promise that will resolve when the
    *     server has closed all connections.
    */
   this.stop = function() {
-    var d = promise.defer();
-    server.close(d.fulfill);
-    return d.promise;
+    return new Promise(resolve => server.close(resolve));
   };
 
   /**

package/lib/test/index.js

@@ -24,6 +24,7 @@ var build = require('./build'),
     webdriver = require('../../'),
     flow = webdriver.promise.controlFlow(),
     firefox = require('../../firefox'),
+    safari = require('../../safari'),
     remote = require('../../remote'),
     testing = require('../../testing'),
     fileserver = require('./fileserver');
@@ -113,6 +114,8 @@ var browsersToTest = (function() {
       console.log('Running tests using loopback address')
     }
   }
+  console.log(
+      'Promise manager is enabled? ' + webdriver.promise.USE_PROMISE_MANAGER);
 
   return browsers;
 })();
@@ -215,27 +218,31 @@ function suite(fn, opt_options) {
 
   try {
 
+    before(function() {
+      if (isDevMode) {
+        return build.of(
+            '//javascript/atoms/fragments:is-displayed',
+            '//javascript/webdriver/atoms:getAttribute')
+            .onlyOnce().go();
+      }
+    });
+
     // Server is only started if required for a specific config.
-    testing.after(function() {
+    after(function() {
       if (seleniumServer) {
         return seleniumServer.stop();
       }
     });
 
     browsers.forEach(function(browser) {
-      testing.describe('[' + browser + ']', function() {
+      describe('[' + browser + ']', function() {
 
         if (isDevMode && nativeRun) {
           if (browser === LEGACY_FIREFOX) {
-            testing.before(function() {
+            before(function() {
               return build.of('//javascript/firefox-driver:webdriver')
                   .onlyOnce().go();
             });
-          } else if (browser === webdriver.Browser.SAFARI) {
-            testing.before(function() {
-              return build.of('//javascript/safari-driver:client')
-                  .onlyOnce().go();
-            });
           }
         }
 
@@ -247,7 +254,7 @@ function suite(fn, opt_options) {
                 serverJar, {loopback: useLoopback});
           }
 
-          testing.before(function() {
+          before(function() {
             this.timeout(0);
             return seleniumServer.start(60 * 1000);
           });
@@ -263,14 +270,14 @@ function suite(fn, opt_options) {
 
 // GLOBAL TEST SETUP
 
-testing.before(function() {
+before(function() {
    // Do not pass register fileserver.start directly with testing.before,
    // as start takes an optional port, which before assumes is an async
    // callback.
    return fileserver.start();
 });
 
-testing.after(function() {
+after(function() {
    return fileserver.stop();
 });
 

package/lib/test/promise.js

@@ -0,0 +1,79 @@
+// Licensed to the Software Freedom Conservancy (SFC) under one
+// or more contributor license agreements.  See the NOTICE file
+// distributed with this work for additional information
+// regarding copyright ownership.  The SFC licenses this file
+// to you under the Apache License, Version 2.0 (the
+// "License"); you may not use this file except in compliance
+// with the License.  You may obtain a copy of the License at
+//
+//   http://www.apache.org/licenses/LICENSE-2.0
+//
+// Unless required by applicable law or agreed to in writing,
+// software distributed under the License is distributed on an
+// "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+// KIND, either express or implied.  See the License for the
+// specific language governing permissions and limitations
+// under the License.
+
+'use strict';
+
+
+/**
+ * Configures a block of mocha tests, ensuring the promise manager is either
+ * enabled or disabled for the tests' execution.
+ * for their execution.
+ */
+function withPromiseManager(enabled, fn) {
+  describe(`SELENIUM_PROMISE_MANAGER=${enabled}`, function() {
+    let saved;
+
+    function changeState() {
+      saved = process.env.SELENIUM_PROMISE_MANAGER;
+      process.env.SELENIUM_PROMISE_MANAGER = enabled;
+    }
+
+    function restoreState() {
+      if (saved === undefined) {
+        delete process.env.SELENIUM_PROMISE_MANAGER;
+      } else {
+        process.env.SELENIUM_PROMISE_MANAGER = saved;
+      }
+    }
+
+    before(changeState);
+    after(restoreState);
+
+    try {
+      changeState();
+      fn();
+    } finally {
+      restoreState();
+    }
+  });
+};
+
+
+/**
+ * Defines a set of tests to run both with and without the promise manager
+ * enabled.
+ */
+exports.promiseManagerSuite = function(fn) {
+  withPromiseManager(true, fn);
+  withPromiseManager(false, fn);
+};
+
+
+/**
+ * Ensures the promise manager is enabled when the provided tests run.
+ */
+exports.enablePromiseManager = function(fn) {
+  withPromiseManager(true, fn);
+};
+
+
+/**
+ * Ensures the promise manager is disabled when the provided tests run.
+ */
+exports.disablePromiseManager = function(fn) {
+  withPromiseManager(false, fn);
+};