selenium-webdriver 4.1.0 → 4.1.1

package/lib/webdriver.js

@@ -250,30 +250,30 @@ class IWebDriver {
    *     result.
    * @template T
    */
-  execute(command) { } // eslint-disable-line
+  execute(command) {} // eslint-disable-line
 
   /**
    * Sets the {@linkplain input.FileDetector file detector} that should be
    * used with this instance.
    * @param {input.FileDetector} detector The detector to use or `null`.
    */
-  setFileDetector(detector) { } // eslint-disable-line
+  setFileDetector(detector) {} // eslint-disable-line
 
   /**
    * @return {!command.Executor} The command executor used by this instance.
    */
-  getExecutor() { }
+  getExecutor() {}
 
   /**
    * @return {!Promise<!Session>} A promise for this client's session.
    */
-  getSession() { }
+  getSession() {}
 
   /**
    * @return {!Promise<!Capabilities>} A promise that will resolve with
    *     the this instance's capabilities.
    */
-  getCapabilities() { }
+  getCapabilities() {}
 
   /**
    * Terminates the browser session. After calling quit, this instance will be
@@ -283,7 +283,7 @@ class IWebDriver {
    * @return {!Promise<void>} A promise that will be resolved when the
    *     command has completed.
    */
-  quit() { }
+  quit() {}
 
   /**
    * Creates a new action sequence using this driver. The sequence will not be
@@ -296,7 +296,7 @@ class IWebDriver {
    *     for details).
    * @return {!input.Actions} A new action sequence for this instance.
    */
-  actions(options) { } // eslint-disable-line
+  actions(options) {} // eslint-disable-line
 
   /**
    * Executes a snippet of JavaScript in the context of the currently selected
@@ -334,7 +334,7 @@ class IWebDriver {
    *    scripts return value.
    * @template T
    */
-  executeScript(script, ...args) { } // eslint-disable-line
+  executeScript(script, ...args) {} // eslint-disable-line
 
   /**
    * Executes a snippet of asynchronous JavaScript in the context of the
@@ -411,7 +411,7 @@ class IWebDriver {
    *     value.
    * @template T
    */
-  executeAsyncScript(script, ...args) { } // eslint-disable-line
+  executeAsyncScript(script, ...args) {} // eslint-disable-line
 
   /**
    * Waits for a condition to evaluate to a "truthy" value. The condition may be
@@ -462,7 +462,7 @@ class IWebDriver {
     timeout = undefined, // eslint-disable-line
     message = undefined, // eslint-disable-line
     pollTimeout = undefined // eslint-disable-line
-  ) { }
+  ) {}
 
   /**
    * Makes the driver sleep for the given amount of time.
@@ -471,7 +471,7 @@ class IWebDriver {
    * @return {!Promise<void>} A promise that will be resolved when the sleep has
    *     finished.
    */
-  sleep(ms) { } // eslint-disable-line
+  sleep(ms) {} // eslint-disable-line
 
   /**
    * Retrieves the current window handle.
@@ -479,7 +479,7 @@ class IWebDriver {
    * @return {!Promise<string>} A promise that will be resolved with the current
    *     window handle.
    */
-  getWindowHandle() { }
+  getWindowHandle() {}
 
   /**
    * Retrieves a list of all available window handles.
@@ -487,7 +487,7 @@ class IWebDriver {
    * @return {!Promise<!Array<string>>} A promise that will be resolved with an
    *     array of window handles.
    */
-  getAllWindowHandles() { }
+  getAllWindowHandles() {}
 
   /**
    * Retrieves the current page's source. The returned source is a representation
@@ -497,7 +497,7 @@ class IWebDriver {
    * @return {!Promise<string>} A promise that will be resolved with the current
    *     page source.
    */
-  getPageSource() { }
+  getPageSource() {}
 
   /**
    * Closes the current window.
@@ -505,7 +505,7 @@ class IWebDriver {
    * @return {!Promise<void>} A promise that will be resolved when this command
    *     has completed.
    */
-  close() { }
+  close() {}
 
   /**
    * Navigates to the given URL.
@@ -514,7 +514,7 @@ class IWebDriver {
    * @return {!Promise<void>} A promise that will be resolved when the document
    *     has finished loading.
    */
-  get(url) { } // eslint-disable-line
+  get(url) {} // eslint-disable-line
 
   /**
    * Retrieves the URL for the current page.
@@ -522,7 +522,7 @@ class IWebDriver {
    * @return {!Promise<string>} A promise that will be resolved with the
    *     current URL.
    */
-  getCurrentUrl() { }
+  getCurrentUrl() {}
 
   /**
    * Retrieves the current page title.
@@ -530,7 +530,7 @@ class IWebDriver {
    * @return {!Promise<string>} A promise that will be resolved with the current
    *     page's title.
    */
-  getTitle() { }
+  getTitle() {}
 
   /**
    * Locates an element on the page. If the element cannot be found, a
@@ -570,7 +570,7 @@ class IWebDriver {
    *     commands against the located element. If the element is not found, the
    *     element will be invalidated and all scheduled commands aborted.
    */
-  findElement(locator) { } // eslint-disable-line
+  findElement(locator) {} // eslint-disable-line
 
   /**
    * Search for multiple elements on the page. Refer to the documentation on
@@ -580,7 +580,7 @@ class IWebDriver {
    * @return {!Promise<!Array<!WebElement>>} A promise that will resolve to an
    *     array of WebElements.
    */
-  findElements(locator) { } // eslint-disable-line
+  findElements(locator) {} // eslint-disable-line
 
   /**
    * Takes a screenshot of the current page. The driver makes a best effort to
@@ -594,23 +594,23 @@ class IWebDriver {
    * @return {!Promise<string>} A promise that will be resolved to the
    *     screenshot as a base-64 encoded PNG.
    */
-  takeScreenshot() { }
+  takeScreenshot() {}
 
   /**
    * @return {!Options} The options interface for this instance.
    */
-  manage() { }
+  manage() {}
 
   /**
    * @return {!Navigation} The navigation interface for this instance.
    */
-  navigate() { }
+  navigate() {}
 
   /**
    * @return {!TargetLocator} The target locator interface for this
    *     instance.
    */
-  switchTo() { }
+  switchTo() {}
 
   /**
    *
@@ -629,7 +629,7 @@ class IWebDriver {
    *         shrinkToFit: (boolean|undefined)
    *         pageRanges: (<Array>|undefined)}} options.
    */
-  printPage(options) { } // eslint-disable-line
+  printPage(options) {} // eslint-disable-line
 }
 
 /**
@@ -669,7 +669,7 @@ class WebDriver {
     // If session is a rejected promise, add a no-op rejection handler.
     // This effectively hides setup errors until users attempt to interact
     // with the session.
-    this.session_.catch(function () { })
+    this.session_.catch(function () {})
 
     /** @private {!command.Executor} */
     this.executor_ = executor
@@ -765,12 +765,12 @@ class WebDriver {
       this.session_ = Promise.reject(
         new error.NoSuchSessionError(
           'This driver instance does not have a valid session ID ' +
-          '(did you call WebDriver.quit()?) and may no longer be used.'
+            '(did you call WebDriver.quit()?) and may no longer be used.'
         )
       )
 
       // Only want the session rejection to bubble if accessed.
-      this.session_.catch(function () { })
+      this.session_.catch(function () {})
 
       if (this.onQuit_) {
         return this.onQuit_.call(void 0)
@@ -831,15 +831,18 @@ class WebDriver {
             let timeoutMessage = resolveWaitMessage(message)
             reject(
               new error.TimeoutError(
-                `${timeoutMessage}Timed out waiting for promise to resolve after ${Date.now() - start
+                `${timeoutMessage}Timed out waiting for promise to resolve after ${
+                  Date.now() - start
                 }ms`
               )
             )
           } catch (ex) {
             reject(
               new error.TimeoutError(
-                `${ex.message
-                }\nTimed out waiting for promise to resolve after ${Date.now() - start
+                `${
+                  ex.message
+                }\nTimed out waiting for promise to resolve after ${
+                  Date.now() - start
                 }ms`
               )
             )
@@ -869,7 +872,7 @@ class WebDriver {
     if (typeof fn !== 'function') {
       throw TypeError(
         'Wait condition must be a promise-like object, function, or a ' +
-        'Condition object'
+          'Condition object'
       )
     }
 
@@ -921,7 +924,7 @@ class WebDriver {
           if (!(value instanceof WebElement)) {
             throw TypeError(
               'WebElementCondition did not resolve to a WebElement: ' +
-              Object.prototype.toString.call(value)
+                Object.prototype.toString.call(value)
             )
           }
           return value
@@ -1393,7 +1396,7 @@ class WebDriver {
 
         callback(event)
       }
-      
+
       if (params.method === 'Log.entryAdded') {
         const logEventParams = params['params']
         const logEntry = logEventParams['entry']
@@ -1405,7 +1408,6 @@ class WebDriver {
 
         callback(event)
       }
-
     })
     await connection.execute('Runtime.enable', {}, null)
   }
@@ -1783,7 +1785,7 @@ class Options {
       } else if (typeof value !== 'undefined') {
         throw TypeError(
           'invalid timeouts configuration:' +
-          ` expected "${key}" to be a number, got ${typeof value}`
+            ` expected "${key}" to be a number, got ${typeof value}`
         )
       }
     }
@@ -1844,7 +1846,7 @@ function legacyTimeout(driver, type, ms) {
  *
  * @record
  */
-Options.Cookie = function () { }
+Options.Cookie = function () {}
 
 /**
  * The name of the cookie.
@@ -1949,21 +1951,6 @@ class Window {
         new command.Command(command.Name.GET_WINDOW_RECT)
       )
     } catch (ex) {
-      if (ex instanceof error.UnknownCommandError) {
-        let { width, height } = await this.driver_.execute(
-          new command.Command(command.Name.GET_WINDOW_SIZE).setParameter(
-            'windowHandle',
-            'current'
-          )
-        )
-        let { x, y } = await this.driver_.execute(
-          new command.Command(command.Name.GET_WINDOW_POSITION).setParameter(
-            'windowHandle',
-            'current'
-          )
-        )
-        return { x, y, width, height }
-      }
       throw ex
     }
   }
@@ -1993,26 +1980,6 @@ class Window {
         })
       )
     } catch (ex) {
-      if (ex instanceof error.UnknownCommandError) {
-        if (typeof x === 'number' && typeof y === 'number') {
-          await this.driver_.execute(
-            new command.Command(command.Name.SET_WINDOW_POSITION)
-              .setParameter('windowHandle', 'current')
-              .setParameter('x', x)
-              .setParameter('y', y)
-          )
-        }
-
-        if (typeof width === 'number' && typeof height === 'number') {
-          await this.driver_.execute(
-            new command.Command(command.Name.SET_WINDOW_SIZE)
-              .setParameter('windowHandle', 'current')
-              .setParameter('width', width)
-              .setParameter('height', height)
-          )
-        }
-        return this.getRect()
-      }
       throw ex
     }
   }
@@ -2550,18 +2517,18 @@ class WebElement {
    */
   async sendKeys(...args) {
     let keys = []
-      ; (await Promise.all(args)).forEach((key) => {
-        let type = typeof key
-        if (type === 'number') {
-          key = String(key)
-        } else if (type !== 'string') {
-          throw TypeError('each key must be a number of string; got ' + type)
-        }
+    ;(await Promise.all(args)).forEach((key) => {
+      let type = typeof key
+      if (type === 'number') {
+        key = String(key)
+      } else if (type !== 'string') {
+        throw TypeError('each key must be a number of string; got ' + type)
+      }
 
-        // The W3C protocol requires keys to be specified as an array where
-        // each element is a single key.
-        keys.push(...key.split(''))
-      })
+      // The W3C protocol requires keys to be specified as an array where
+      // each element is a single key.
+      keys.push(...key.split(''))
+    })
 
     if (!this.driver_.fileDetector_) {
       return this.execute_(
@@ -2571,10 +2538,15 @@ class WebElement {
       )
     }
 
-    keys = await this.driver_.fileDetector_.handleFile(
-      this.driver_,
-      keys.join('')
-    )
+    try {
+      keys = await this.driver_.fileDetector_.handleFile(
+        this.driver_,
+        keys.join('')
+      )
+    } catch (ex) {
+      console.log('Error trying parse string as a file with file detector; sending keys instead' + ex)
+    }
+
     return this.execute_(
       new command.Command(command.Name.SEND_KEYS_TO_ELEMENT)
         .setParameter('text', keys)
@@ -2651,6 +2623,36 @@ class WebElement {
     )
   }
 
+  /**
+   * Get the value of the given attribute of the element.
+   * <p>
+   * This method, unlike {@link #getAttribute(String)}, returns the value of the attribute with the
+   * given name but not the property with the same name.
+   * <p>
+   * The following are deemed to be "boolean" attributes, and will return either "true" or null:
+   * <p>
+   * async, autofocus, autoplay, checked, compact, complete, controls, declare, defaultchecked,
+   * defaultselected, defer, disabled, draggable, ended, formnovalidate, hidden, indeterminate,
+   * iscontenteditable, ismap, itemscope, loop, multiple, muted, nohref, noresize, noshade,
+   * novalidate, nowrap, open, paused, pubdate, readonly, required, reversed, scoped, seamless,
+   * seeking, selected, truespeed, willvalidate
+   * <p>
+   * See <a href="https://w3c.github.io/webdriver/#get-element-attribute">W3C WebDriver specification</a>
+   * for more details.
+   *
+   * @param attributeName The name of the attribute.
+   * @return The attribute's value or null if the value is not set.
+   */
+
+  getDomAttribute(attributeName) {
+    return this.execute_(
+      new command.Command(command.Name.GET_DOM_ATTRIBUTE).setParameter(
+        'name',
+        attributeName
+      )
+    )
+  }
+
   /**
    * Get the given property of the referenced web element
    * @param {string} propertyName The name of the attribute to query.
@@ -2719,15 +2721,7 @@ class WebElement {
         new command.Command(command.Name.GET_ELEMENT_RECT)
       )
     } catch (err) {
-      if (err instanceof error.UnknownCommandError) {
-        const { width, height } = await this.execute_(
-          new command.Command(command.Name.GET_ELEMENT_SIZE)
-        )
-        const { x, y } = await this.execute_(
-          new command.Command(command.Name.GET_ELEMENT_LOCATION)
-        )
-        return { x, y, width, height }
-      }
+        throw err;
     }
   }
 
@@ -2764,8 +2758,8 @@ class WebElement {
     const form = this.findElement({ xpath: './ancestor-or-self::form' })
     this.driver_.executeScript(
       "var e = arguments[0].ownerDocument.createEvent('Event');" +
-      "e.initEvent('submit', true, true);" +
-      'if (arguments[0].dispatchEvent(e)) { arguments[0].submit() }',
+        "e.initEvent('submit', true, true);" +
+        'if (arguments[0].dispatchEvent(e)) { arguments[0].submit() }',
       form
     )
   }
@@ -2861,7 +2855,6 @@ class WebElementPromise extends WebElement {
  * retrieve elements that live in the DOM below the ShadowRoot.
  */
 class ShadowRoot {
-
   constructor(driver, id) {
     this.driver_ = driver
     this.id_ = id
@@ -2891,7 +2884,8 @@ class ShadowRoot {
     return (
       obj &&
       typeof obj === 'object' &&
-      typeof obj[SHADOWROOT_ID_KEY] === 'string')
+      typeof obj[SHADOWROOT_ID_KEY] === 'string'
+    )
   }
 
   /**
@@ -3005,7 +2999,7 @@ class ShadowRootPromise extends ShadowRoot {
   /**
    * @param {!WebDriver} driver The parent WebDriver instance for this
    *     element.
-   * @param {!Promise<!ShadowRoot>} el A promise
+   * @param {!Promise<!ShadowRoot>} shadow A promise
    *     that will resolve to the promised element.
    */
   constructor(driver, shadow) {

package/net/index.js

@@ -35,10 +35,10 @@ function getLoInterface() {
  * @param {string} family The IP family (IPv4 or IPv6). Defaults to IPv4.
  * @return {(string|undefined)} The located IP address or undefined.
  */
-function getAddress(loopback, family) {
+function getIPAddress(loopback, family) {
   let interfaces
   if (loopback) {
-    let lo = getLoInterface()
+    const lo = getLoInterface()
     interfaces = lo ? [lo] : null
   }
   interfaces = interfaces || os.networkInterfaces()
@@ -63,8 +63,8 @@ function getAddress(loopback, family) {
  * @param {string=} family The IP family to retrieve. Defaults to "IPv4".
  * @return {(string|undefined)} The IP address or undefined if not available.
  */
-exports.getAddress = function (family = 'IPv4') {
-  return getAddress(false, family)
+function getAddress(family = 'IPv4') {
+  return getIPAddress(false, family)
 }
 
 /**
@@ -72,8 +72,8 @@ exports.getAddress = function (family = 'IPv4') {
  * @param {string=} family The IP family to retrieve. Defaults to "IPv4".
  * @return {(string|undefined)} The IP address or undefined if not available.
  */
-exports.getLoopbackAddress = function (family = 'IPv4') {
-  return getAddress(true, family)
+function getLoopbackAddress(family = 'IPv4') {
+  return getIPAddress(true, family)
 }
 
 /**
@@ -84,7 +84,7 @@ exports.getLoopbackAddress = function (family = 'IPv4') {
  * @return {{host: string, port: ?number}} A host and port. If no port is
  *     present in the argument `hostport`, port is null.
  */
-exports.splitHostAndPort = function (hostport) {
+function splitHostAndPort(hostport) {
   let lastIndex = hostport.lastIndexOf(':')
   if (lastIndex < 0) {
     return { host: hostport, port: null }
@@ -105,3 +105,10 @@ exports.splitHostAndPort = function (hostport) {
   let port = parseInt(hostport.slice(lastIndex + 1), 10)
   return { host, port }
 }
+
+// PUBLIC API
+module.exports = {
+  splitHostAndPort,
+  getLoopbackAddress,
+  getAddress,
+}

package/net/portprober.js

@@ -70,6 +70,7 @@ function findFreePort(opt_host) {
 }
 
 // PUBLIC API
-
-exports.findFreePort = findFreePort
-exports.isFree = isFree
+module.exports = {
+  findFreePort,
+  isFree,
+}

package/opera.js

@@ -72,12 +72,12 @@
 
 'use strict'
 
-const http = require('./http'),
-  io = require('./io'),
-  capabilities = require('./lib/capabilities'),
-  Symbols = require('./lib/symbols'),
-  webdriver = require('./lib/webdriver'),
-  remote = require('./remote')
+const http = require('./http')
+const io = require('./io')
+const capabilities = require('./lib/capabilities')
+const Symbols = require('./lib/symbols')
+const webdriver = require('./lib/webdriver')
+const remote = require('./remote')
 
 /**
  * Name of the OperaDriver executable.
@@ -153,7 +153,7 @@ class ServiceBuilder extends remote.DriverService.Builder {
 }
 
 /** @type {remote.DriverService} */
-var defaultService = null
+let defaultService = null
 
 /**
  * Sets the default service to use for new OperaDriver instances.
@@ -187,7 +187,7 @@ function getDefaultService() {
  * @type {string}
  * @const
  */
-var OPTIONS_CAPABILITY_KEY = 'chromeOptions'
+const OPTIONS_CAPABILITY_KEY = 'chromeOptions'
 
 /**
  * Class for managing {@linkplain Driver OperaDriver} specific options.
@@ -217,8 +217,8 @@ class Options {
    * @return {!Options} The OperaDriver options.
    */
   static fromCapabilities(caps) {
-    var options
-    var o = caps.get(OPTIONS_CAPABILITY_KEY)
+    let options
+    const o = caps.get(OPTIONS_CAPABILITY_KEY)
     if (o instanceof Options) {
       options = o
     } else if (o) {
@@ -249,8 +249,8 @@ class Options {
    * @param {...(string|!Array.<string>)} var_args The arguments to add.
    * @return {!Options} A self reference.
    */
-  addArguments(var_args) { // eslint-disable-line
-    this.args_ = this.args_.concat.apply(this.args_, arguments)
+  addArguments(...var_args) {
+    this.args_ = this.args_.concat.apply(this.args_, var_args)
     return this
   }
 
@@ -262,11 +262,8 @@ class Options {
    *     extensions to add.
    * @return {!Options} A self reference.
    */
-  addExtensions(var_args) { // eslint-disable-line
-    this.extensions_ = this.extensions_.concat.apply(
-      this.extensions_,
-      arguments
-    )
+  addExtensions(...var_args) {
+    this.extensions_ = this.extensions_.concat.apply(this.extensions_, var_args)
     return this
   }
 
@@ -358,11 +355,11 @@ class Driver extends webdriver.WebDriver {
    * @return {!Driver} A new driver instance.
    */
   static createSession(opt_config, opt_service) {
-    var service = opt_service || getDefaultService()
-    var client = service.start().then((url) => new http.HttpClient(url))
-    var executor = new http.Executor(client)
+    const service = opt_service || getDefaultService()
+    const client = service.start().then((url) => new http.HttpClient(url))
+    const executor = new http.Executor(client)
 
-    var caps =
+    let caps =
       opt_config instanceof Options
         ? opt_config.toCapabilities()
         : opt_config || capabilities.Capabilities.opera()
@@ -370,7 +367,7 @@ class Driver extends webdriver.WebDriver {
     // On Linux, the OperaDriver does not look for Opera on the PATH, so we
     // must explicitly find it. See: operachromiumdriver #9.
     if (process.platform === 'linux') {
-      var options = Options.fromCapabilities(caps)
+      const options = Options.fromCapabilities(caps)
       if (!options.binary_) {
         let exe = io.findInPath('opera', true)
         if (!exe) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "selenium-webdriver",
-  "version": "4.1.0",
+  "version": "4.1.1",
   "description": "The official WebDriver JavaScript bindings from the Selenium project",
   "license": "Apache-2.0",
   "keywords": [