-
Notifications
You must be signed in to change notification settings - Fork 15
Bump selenium-webdriver from 3.142.7 to 4.1.0 #894
Conversation
I had a quick go at testing this locally and it appears to work OK, aside from a few option tweaks. The only irritation is that we'll see a deprecation warning after every scenario until dbalatero/capybara-chromedriver-logger#34 is merged. |
f29629e
to
2482445
Compare
After this is merged, we can get rid of the deprecation messages by merging: #933 |
65d975e
to
6fc5077
Compare
Bumps [selenium-webdriver](https://github.com/SeleniumHQ/selenium) from 3.142.7 to 4.1.0. - [Release notes](https://github.com/SeleniumHQ/selenium/releases) - [Changelog](https://github.com/SeleniumHQ/selenium/blob/trunk/rb/CHANGES) - [Commits](https://github.com/SeleniumHQ/selenium/commits/selenium-4.1.0) --- updated-dependencies: - dependency-name: selenium-webdriver dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <[email protected]>
289fc19
to
bada8ed
Compare
features/support/env.rb
Outdated
options: options, | ||
desired_capabilities: capabilities | ||
) | ||
capabilities: capabilities |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
capabilities: capabilities | |
capabilities: [capabilities, options] |
According to the deprecation message we can just combine the two:
:options as a parameter for driver initialization is deprecated. Use :capabilities with an Array of value capabilities/options if necessary instead.
I've tested with an array and both sets are indeed passed forward.
Is there an advantage to the chromeOptions
approach?
bada8ed
to
998bc68
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Works for me locally testing Integration ✅.* I watched the output for a while and scanned back through it; I can't see any warnings - nice one!
*I get two failures for ab_testing
, which mirror Integration at the moment.
Bumps selenium-webdriver from 3.142.7 to 4.1.0.
Release notes
Sourced from selenium-webdriver's releases.
... (truncated)
Changelog
Sourced from selenium-webdriver's changelog.
... (truncated)
Commits
Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting
@dependabot rebase
.Dependabot commands and options
You can trigger Dependabot actions by commenting on this PR:
@dependabot rebase
will rebase this PR@dependabot recreate
will recreate this PR, overwriting any edits that have been made to it@dependabot merge
will merge this PR after your CI passes on it@dependabot squash and merge
will squash and merge this PR after your CI passes on it@dependabot cancel merge
will cancel a previously requested merge and block automerging@dependabot reopen
will reopen this PR if it is closed@dependabot close
will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually@dependabot ignore this major version
will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)@dependabot ignore this minor version
will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)@dependabot ignore this dependency
will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)