-
Notifications
You must be signed in to change notification settings - Fork 44
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
chore(deps): update dependency org.seleniumhq.selenium:selenium-java to v4 #3522
chore(deps): update dependency org.seleniumhq.selenium:selenium-java to v4 #3522
Conversation
e6dc225
to
e94c97c
Compare
e94c97c
to
b7ed99c
Compare
cd6cf31
to
478d694
Compare
478d694
to
0e6eb55
Compare
0e6eb55
to
90f66b0
Compare
90f66b0
to
f624192
Compare
7a28b99
to
a3260a6
Compare
a3260a6
to
fbfe3a7
Compare
fbfe3a7
to
9202569
Compare
9202569
to
a32ebee
Compare
a32ebee
to
99d27d6
Compare
8ebbea8
to
668dbf1
Compare
668dbf1
to
da17d1c
Compare
e637aae
to
cdb2ea4
Compare
cdb2ea4
to
3ed7215
Compare
3ed7215
to
34c2ded
Compare
34c2ded
to
e474179
Compare
069c054
to
1165ebc
Compare
1165ebc
to
3b5303c
Compare
3b5303c
to
0368034
Compare
a74b973
to
3d43c0e
Compare
3d43c0e
to
b25e29f
Compare
b25e29f
to
c9c34d4
Compare
c9c34d4
to
999b5bf
Compare
999b5bf
to
f33ae75
Compare
f33ae75
to
bd7041d
Compare
eb1d801
to
9c83bdc
Compare
9c83bdc
to
1a2287e
Compare
1a2287e
to
c0087ba
Compare
c0087ba
to
c9f09a0
Compare
c9f09a0
to
86199da
Compare
WrapsElement moved out of 'internal' as per SeleniumHQ/selenium#6881
Newer Selenium uses Duration rather than int (seconds) for waits and timeouts.
This function has been removed. Indeed, FirefoxBinary has been deprecated and we're meant to move to FirefoxOptions.
Okay, so with all the test runs we're having I've gone and addressed all the issues. I've run a few test suites locally and they run. I've also tried some of the flaky ones and they also continue to sometimes flake. But things seem to run faster, and I believe we get better error messages. Let's see what happens. Considering everything on tests is flaking at the moment, if this doesn't make anything materially worse I suggest we merge it in. |
Edited/Blocked NotificationRenovate will not automatically rebase this PR, because it does not recognize the last commit author and assumes somebody else may have edited the PR. You can manually request rebase by checking the rebase/retry box above. |
So the good news is, everything still works as well as it did before. But, the bad news is, everything still works as well as it did before. That is, the same tests are still flaking like they always did. But, at least the upgrade worked - in that it didn't make anything worse. Let's get this reviewed and merged an then we can go from there. |
@@ -175,7 +173,6 @@ private FirefoxDriver getDriver() { | |||
} else { | |||
binary = new FirefoxBinary(); | |||
} | |||
binary.setTimeout(SECONDS.toMillis(120)); |
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.
setTimeout
is no longer available. And we shouldn't even be using FirefoxBinary
any more. The commit for this is 792b61c
@@ -142,7 +140,6 @@ public WebDriver getDriver(Class<?> clazz) throws IOException { | |||
} else { | |||
binary = new FirefoxBinary(); | |||
} | |||
binary.setTimeout(SECONDS.toMillis(120)); |
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.
setTimeout
is no longer available. And we shouldn't even be using FirefoxBinary
any more. The commit for this is 792b61c
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.
LGTM, just one minor suggestion.
This PR contains the following updates:
3.141.59
->4.26.0
Warning
Some dependencies could not be looked up. Check the Dependency Dashboard for more information.
Release Notes
SeleniumHQ/selenium (org.seleniumhq.selenium:selenium-java)
v4.26.0
v4.25.0
v4.24.0
v4.23.0
v4.22.0
v4.21.0
v4.20.0
v4.19.0
v4.18.1
v4.18.0
v4.17.0
v4.16.0
v4.15.0
v4.14.0
v4.13.0
v4.12.0
v4.11.0
v4.10.0
v4.9.1
v4.9.0
v4.8.2
v4.8.1
v4.8.0
v4.7.1
v4.7.0
v4.6.0
Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.
This PR was generated by Mend Renovate. View the repository job log.