-
Notifications
You must be signed in to change notification settings - Fork 2.5k
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鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix safari on sauce labs: lock selenium-webdriver
to 4.1.0 for now
#4827
Conversation
487fbc2
to
f6a511c
Compare
package.json
Outdated
@@ -101,7 +101,7 @@ | |||
"prettier": "2.7.1", | |||
"promise-polyfill": "8.2.3", | |||
"sauce-connect-launcher": "1.3.2", | |||
"selenium-webdriver": "4.3.1", | |||
"selenium-webdriver": "4.0.0", |
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.
Nice find.
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.
it didn't actually fix it. Just trying a few different things
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.
ah wait yeh it's actual failures now 馃
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.
will do some more digging and try and figure out which version broke
ok it looks like it broke with 4.1.1, so something in SeleniumHQ/selenium@selenium-4.1.0...selenium-4.1.1-javascript |
7ebc98e
to
384cd5e
Compare
selenium-webdriver
to 4.1.1 for now
to make safari on SauceLabs work
384cd5e
to
ecb2560
Compare
selenium-webdriver
to 4.1.1 for nowselenium-webdriver
to 4.1.0 for now
This PR will...
Lock
selenium-webdriver
to 4.1.0 for now so that SauceLabs works with Safari. Also prevents PR's with new versions from being auto merged, which would happen given the safari tests are optional.Will open a ticket with sauce labs to see if they can explain why the newer versions are getting this error. Maybe they need to update the server on their end 馃し
Why is this Pull Request needed?
Fix safari tests.