selenium-webdriver 4.18.1 → 4.20.0

package/lib/capabilities.js

@@ -246,6 +246,11 @@ class Capabilities {
     this.map_ = new Map(other)
   }
 
+  /** @return {number} The number of capabilities set. */
+  get size() {
+    return this.map_.size
+  }
+
   /**
    * @return {!Capabilities} A basic set of capabilities for Chrome.
    */
@@ -315,11 +320,6 @@ class Capabilities {
     return this.map_.keys()
   }
 
-  /** @return {number} The number of capabilities set. */
-  get size() {
-    return this.map_.size
-  }
-
   /**
    * Merges another set of capabilities into this instance.
    * @param {!(Capabilities|Map<String, ?>|Object<string, ?>)} other The other

package/lib/http.js

@@ -25,7 +25,7 @@
 
 'use strict'
 
-const path = require('path')
+const path = require('node:path')
 const cmd = require('./command')
 const error = require('./error')
 const logging = require('./logging')
@@ -147,21 +147,24 @@ const Atom = {
 function post(path) {
   return resource('POST', path)
 }
+
 function del(path) {
   return resource('DELETE', path)
 }
+
 function get(path) {
   return resource('GET', path)
 }
+
 function resource(method, path) {
   return { method: method, path: path }
 }
 
 /** @typedef {{method: string, path: string}} */
-var CommandSpec // eslint-disable-line
+var CommandSpec
 
 /** @typedef {function(!cmd.Command): !cmd.Command} */
-var CommandTransformer // eslint-disable-line
+var CommandTransformer
 
 class InternalTypeError extends TypeError {}
 
@@ -334,7 +337,7 @@ class Client {
    * @return {!Promise<Response>} A promise that will be fulfilled with the
    *     server's response.
    */
-  send(httpRequest) {} // eslint-disable-line
+  send(httpRequest) {}
 }
 
 /**
@@ -469,6 +472,7 @@ class Executor {
 function tryParse(str) {
   try {
     return JSON.parse(str)
+    /*eslint no-unused-vars: "off"*/
   } catch (ignored) {
     // Do nothing.
   }

package/lib/input.js

@@ -162,7 +162,6 @@ class FileDetector {
    * @package
    */
   handleFile(_driver, path) {
-    // eslint-disable-line
     return Promise.resolve(path)
   }
 }

package/lib/pinnedScript.js

@@ -15,7 +15,7 @@
 // specific language governing permissions and limitations
 // under the License.
 
-const crypto = require('crypto')
+const crypto = require('node:crypto')
 
 class PinnedScript {
   constructor(script) {

package/lib/select.js

@@ -51,23 +51,23 @@ class ISelect {
    * @return {!Promise<boolean>} Whether this select element supports selecting multiple options at the same time? This
    * is done by checking the value of the "multiple" attribute.
    */
-  isMultiple() {} // eslint-disable-line
+  isMultiple() {}
 
   /**
    * @return {!Promise<!Array<!WebElement>>} All options belonging to this select tag
    */
-  getOptions() {} // eslint-disable-line
+  getOptions() {}
 
   /**
    * @return {!Promise<!Array<!WebElement>>} All selected options belonging to this select tag
    */
-  getAllSelectedOptions() {} // eslint-disable-line
+  getAllSelectedOptions() {}
 
   /**
    * @return {!Promise<!WebElement>} The first selected option in this select tag (or the currently selected option in a
    * normal select)
    */
-  getFirstSelectedOption() {} // eslint-disable-line
+  getFirstSelectedOption() {}
 
   /**
    * Select all options that display text matching the argument. That is, when given "Bar" this
@@ -105,7 +105,7 @@ class ISelect {
    *
    * @return {Promise<void>}
    */
-  deselectAll() {} // eslint-disable-line
+  deselectAll() {}
 
   /**
    * Deselect all options that display text matching the argument. That is, when given "Bar" this
@@ -161,9 +161,9 @@ class Select {
    *
    * <example>
    <select id="selectbox">
-    <option value="1">Option 1</option>
-    <option value="2">Option 2</option>
-    <option value="3">Option 3</option>
+   <option value="1">Option 1</option>
+   <option value="2">Option 2</option>
+   <option value="3">Option 3</option>
    </select>
    const selectBox = await driver.findElement(By.id("selectbox"));
    await selectObject.selectByIndex(1);

package/lib/until.js

@@ -43,9 +43,9 @@
 
 const by = require('./by')
 const error = require('./error')
-const webdriver = require('./webdriver'),
-  Condition = webdriver.Condition,
-  WebElementCondition = webdriver.WebElementCondition
+const webdriver = require('./webdriver')
+const Condition = webdriver.Condition
+const WebElementCondition = webdriver.WebElementCondition
 
 /**
  * Creates a condition that will wait until the input driver is able to switch

package/lib/util.js

@@ -38,6 +38,7 @@ function isPromise(value) {
     // Use array notation so the Closure compiler does not obfuscate away our
     // contract.
     return (typeof value === 'object' || typeof value === 'function') && typeof value['then'] === 'function'
+    /*eslint no-unused-vars: "off"*/
   } catch (ex) {
     return false
   }

package/lib/virtual_authenticator.js

@@ -128,6 +128,14 @@ class Credential {
     this._signCount = signCount
   }
 
+  static createResidentCredential(id, rpId, userHandle, privateKey, signCount) {
+    return new Credential(id, true, rpId, userHandle, privateKey, signCount)
+  }
+
+  static createNonResidentCredential(id, rpId, privateKey, signCount) {
+    return new Credential(id, false, rpId, null, privateKey, signCount)
+  }
+
   id() {
     return this._id
   }
@@ -169,10 +177,6 @@ class Credential {
     return new Credential(id, true, rpId, userHandle, privateKey, signCount)
   }
 
-  static createResidentCredential(id, rpId, userHandle, privateKey, signCount) {
-    return new Credential(id, true, rpId, userHandle, privateKey, signCount)
-  }
-
   /**
    * Creates a non-resident (i.e. stateless) credential.
    * @param id Unique base64 encoded string.
@@ -186,10 +190,6 @@ class Credential {
     return new Credential(id, false, rpId, null, privateKey, signCount)
   }
 
-  static createNonResidentCredential(id, rpId, privateKey, signCount) {
-    return new Credential(id, false, rpId, null, privateKey, signCount)
-  }
-
   toDict() {
     let credentialData = {
       credentialId: Buffer.from(this._id).toString('base64url'),

package/lib/webdriver.js

@@ -32,9 +32,9 @@ const Symbols = require('./symbols')
 const cdp = require('../devtools/CDPConnection')
 const WebSocket = require('ws')
 const http = require('../http/index')
-const fs = require('fs')
+const fs = require('node:fs')
 const { Capabilities } = require('./capabilities')
-const path = require('path')
+const path = require('node:path')
 const { NoSuchElementError } = require('./error')
 const cdpTargets = ['page', 'browser']
 const { Credential } = require('./virtual_authenticator')
@@ -461,7 +461,7 @@ class IWebDriver {
     condition, // eslint-disable-line
     timeout = undefined, // eslint-disable-line
     message = undefined, // eslint-disable-line
-    pollTimeout = undefined // eslint-disable-line
+    pollTimeout = undefined, // eslint-disable-line
   ) {}
 
   /**
@@ -887,6 +887,7 @@ class WebDriver {
     }
 
     const driver = this
+
     function evaluateCondition() {
       return new Promise((resolve, reject) => {
         try {
@@ -1379,6 +1380,7 @@ class WebDriver {
       null,
     )
   }
+
   /**
    *
    * @param connection
@@ -1966,6 +1968,7 @@ class Options {
     let cmd = new command.Command(command.Name.SET_TIMEOUT)
 
     let valid = false
+
     function setParam(key, value) {
       if (value === null || typeof value === 'number') {
         valid = true
@@ -1974,6 +1977,7 @@ class Options {
         throw TypeError('invalid timeouts configuration:' + ` expected "${key}" to be a number, got ${typeof value}`)
       }
     }
+
     setParam('implicit', implicit)
     setParam('pageLoad', pageLoad)
     setParam('script', script)
@@ -2854,6 +2858,7 @@ class WebElement {
   getAccessibleName() {
     return this.execute_(new command.Command(command.Name.GET_COMPUTED_LABEL))
   }
+
   /**
    * Returns an object describing an element's location, in pixels relative to
    * the document element, and the element's size in pixels.

package/net/index.js

@@ -17,7 +17,7 @@
 
 'use strict'
 
-const os = require('os')
+const os = require('node:os')
 
 function getLoInterface() {
   let name

package/net/portprober.js

@@ -17,7 +17,7 @@
 
 'use strict'
 
-const net = require('net')
+const net = require('node:net')
 
 /**
  * Tests if a port is free.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "selenium-webdriver",
-  "version": "4.18.1",
+  "version": "4.20.0",
   "description": "The official WebDriver JavaScript bindings from the Selenium project",
   "license": "Apache-2.0",
   "keywords": [
@@ -24,30 +24,38 @@
   },
   "dependencies": {
     "jszip": "^3.10.1",
-    "tmp": "^0.2.1",
-    "ws": ">=8.14.2"
+    "tmp": "^0.2.3",
+    "ws": ">=8.16.0"
   },
   "devDependencies": {
-    "eslint": "^8.50.0",
-    "eslint-config-prettier": "^9.0.0",
+    "@bazel/runfiles": "^5.8.1",
+    "@eslint/js": "^9.1.1",
+    "eslint": "^9.1.0",
+    "eslint-config-prettier": "^9.1.0",
+    "eslint-plugin-mocha": "^10.4.3",
+    "eslint-plugin-n": "^17.2.1",
     "eslint-plugin-no-only-tests": "^3.1.0",
-    "eslint-plugin-node": "^11.1.0",
-    "eslint-plugin-prettier": "^5.0.0",
-    "express": "^4.18.2",
-    "mocha": "^10.2.0",
+    "eslint-plugin-prettier": "^5.1.3",
+    "express": "^4.19.2",
+    "globals": "^15.0.0",
+    "mocha": "^10.4.0",
+    "mocha-junit-reporter": "^2.2.1",
     "multer": "^1.4.5-lts.1",
-    "prettier": "^3.0.3",
+    "prettier": "^3.2.5",
     "serve-index": "^1.9.1",
-    "sinon": "^16.1.0"
+    "sinon": "^17.0.1"
   },
   "scripts": {
-    "lint": "eslint --ignore-pattern node_modules --ignore-pattern generator --ext js lib/http.js \"**/*.js\"",
-    "lint:fix": "eslint --ignore-pattern node_modules --ignore-pattern generator --ext js lib/http.js \"**/*.js\" --fix",
+    "lint": "eslint .",
+    "lint:fix": "eslint . --fix",
     "test": "npm run lint && mocha -t 600000 --recursive test",
     "test-jasmine": "bazel test //javascript/node/selenium-webdriver:tests"
   },
   "mocha": {
     "recursive": true,
     "timeout": 600000
+  },
+  "publishConfig": {
+    "registry": "https://registry.npmjs.org/"
   }
 }

package/remote/index.js

@@ -17,7 +17,7 @@
 
 'use strict'
 
-const url = require('url')
+const url = require('node:url')
 
 const httpUtil = require('../http/util')
 const io = require('../io')

package/remote/util.js

@@ -17,8 +17,8 @@
 
 'use strict'
 
-const path = require('path')
-const cp = require('child_process')
+const path = require('node:path')
+const cp = require('node:child_process')
 const logging = require('../lib/logging')
 
 /**

package/safari.js

@@ -25,10 +25,10 @@ const http = require('./http')
 const remote = require('./remote')
 const webdriver = require('./lib/webdriver')
 const { Browser, Capabilities } = require('./lib/capabilities')
-const { getPath } = require('./common/driverFinder')
+const { getBinaryPaths } = require('./common/driverFinder')
 
 /**
- * Creates {@link selenium-webdriver/remote.DriverService} instances that manage
+ * Creates {@link remote.DriverService} instances that manage
  * a [safaridriver] server in a child process.
  *
  * [safaridriver]: https://developer.apple.com/library/prerelease/content/releasenotes/General/WhatsNewInSafari/Articles/Safari_10_0.html#//apple_ref/doc/uid/TP40014305-CH11-DontLinkElementID_28
@@ -123,7 +123,7 @@ class Driver extends webdriver.WebDriver {
 
     let service = new ServiceBuilder(exe).build()
     if (!service.getExecutable()) {
-      service.setExecutable(getPath(caps).driverPath)
+      service.setExecutable(getBinaryPaths(caps).driverPath)
     }
     let executor = new http.Executor(service.start().then((url) => new http.HttpClient(url)))
 

package/testing/index.js

@@ -32,7 +32,7 @@
 
 'use strict'
 
-const { isatty } = require('tty')
+const { isatty } = require('node:tty')
 const chrome = require('../chrome')
 const edge = require('../edge')
 const firefox = require('../firefox')
@@ -41,7 +41,7 @@ const remote = require('../remote')
 const safari = require('../safari')
 const { Browser } = require('../lib/capabilities')
 const { Builder } = require('../index')
-const { getPath } = require('../common/driverFinder')
+const { getBinaryPaths } = require('../common/driverFinder')
 
 /**
  * Describes a browser targeted by a {@linkplain suite test suite}.
@@ -72,15 +72,19 @@ TargetBrowser.prototype.platform
 function color(c, s) {
   return isatty(process.stdout) ? `\u001b[${c}m${s}\u001b[0m` : s
 }
+
 function green(s) {
   return color(32, s)
 }
+
 function cyan(s) {
   return color(36, s)
 }
+
 function info(msg) {
   console.info(`${green('[INFO]')} ${msg}`)
 }
+
 function warn(msg) {
   console.warn(`${cyan('[WARNING]')} ${msg}`)
 }
@@ -118,15 +122,15 @@ function getAvailableBrowsers() {
   info(`Searching for WebDriver executables installed on the current system...`)
 
   let targets = [
-    [getPath(new chrome.Options()), Browser.CHROME],
-    [getPath(new edge.Options()), Browser.EDGE],
-    [getPath(new firefox.Options()), Browser.FIREFOX],
+    [getBinaryPaths(new chrome.Options()), Browser.CHROME],
+    [getBinaryPaths(new edge.Options()), Browser.EDGE],
+    [getBinaryPaths(new firefox.Options()), Browser.FIREFOX],
   ]
   if (process.platform === 'win32') {
-    targets.push([getPath(new ie.Options()), Browser.INTERNET_EXPLORER])
+    targets.push([getBinaryPaths(new ie.Options()), Browser.INTERNET_EXPLORER])
   }
   if (process.platform === 'darwin') {
-    targets.push([getPath(new safari.Options()), Browser.SAFARI])
+    targets.push([getBinaryPaths(new safari.Options()), Browser.SAFARI])
   }
 
   let availableBrowsers = []
@@ -298,6 +302,11 @@ class Environment {
         builder.setCapability('moz:debuggerAddress', true)
       }
 
+      // Enable BiDi for supporting browsers.
+      if (browser.name === Browser.FIREFOX || browser.name === Browser.CHROME || browser.name === Browser.EDGE) {
+        builder.setCapability('webSocketUrl', true)
+      }
+
       if (typeof urlOrServer === 'string') {
         builder.usingServer(urlOrServer)
       } else if (urlOrServer) {
@@ -396,7 +405,7 @@ function suite(fn, options = undefined) {
           seleniumServer = new remote.SeleniumServer(seleniumJar)
 
           const startTimeout = 65 * 1000
-          // eslint-disable-next-line no-inner-declarations
+
           function startSelenium() {
             if (typeof this.timeout === 'function') {
               this.timeout(startTimeout) // For mocha.