Skip to content

Commit

Permalink
[JS] Do not need browserVersion if Selenium is managing it #12641
Browse files Browse the repository at this point in the history
  • Loading branch information
harsha509 committed Aug 30, 2023
1 parent a124440 commit 1c036ab
Showing 1 changed file with 6 additions and 0 deletions.
6 changes: 6 additions & 0 deletions javascript/node/selenium-webdriver/common/seleniumManager.js
Original file line number Diff line number Diff line change
Expand Up @@ -25,6 +25,7 @@ const { platform } = require('process')
const path = require('path')
const fs = require('fs')
const spawnSync = require('child_process').spawnSync
const { Capability } = require('../lib/capabilities')

let debugMessagePrinted = false;

Expand Down Expand Up @@ -126,6 +127,11 @@ function driverLocation(options) {
)
}

// Once driverPath is available, delete browserVersion from payload
if (output.result.driver_path) {
options.delete(Capability.BROWSER_VERSION)
}

logOutput(output)
return {
driverPath: output.result.driver_path,
Expand Down

0 comments on commit 1c036ab

Please sign in to comment.