Saucelabs: remove custom selenium version for Microsoft Edge #23895
Labels
Comments
|
@zouhir quick question - this code was developed when MicrosoftEdge meant EdgeHTML as far as I am aware. I assume Sauce now interprets 'MicrosoftEdge' as Chromium-based edge? |
stephenmcgruer
added a commit
that referenced
this issue
Jun 1, 2020
This appears to be a leftover from the previous use of the sauce runner, assumedly when there was some bug that required a specific version of selenium to be used (it was sadly undocumented). Fixes #23895
Open
4 tasks
|
@stephenmcgruer I confirm your assumption is correct. I found this good reference for the support matrix in Webdriver+Saucelabs: https://wiki.saucelabs.com/display/DOCS/Platform+Configurator#/ MicrosoftEdge is used for both EdgeHTML and Chromium, the version decides which one to launch: |
moz-v2v-gh
pushed a commit
to mozilla/gecko-dev
that referenced
this issue
Jun 8, 2020
…sets selenium version for edge, a=testonly Automatic update from web-platform-tests [Sauce] Remove workaround that manually sets selenium version for edge (#23897) This appears to be a leftover from the previous use of the sauce runner, assumedly when there was some bug that required a specific version of selenium to be used (it was sadly undocumented). Fixes web-platform-tests/wpt#23895 -- wpt-commits: d5aae4ab7f99e024f70aa0d93ca3748809995263 wpt-pr: 23897
xeonchen
pushed a commit
to xeonchen/gecko
that referenced
this issue
Jun 8, 2020
…sets selenium version for edge, a=testonly Automatic update from web-platform-tests [Sauce] Remove workaround that manually sets selenium version for edge (#23897) This appears to be a leftover from the previous use of the sauce runner, assumedly when there was some bug that required a specific version of selenium to be used (it was sadly undocumented). Fixes web-platform-tests/wpt#23895 -- wpt-commits: d5aae4ab7f99e024f70aa0d93ca3748809995263 wpt-pr: 23897
moz-v2v-gh
pushed a commit
to mozilla/gecko-dev
that referenced
this issue
Jun 9, 2020
…sets selenium version for edge, a=testonly Automatic update from web-platform-tests [Sauce] Remove workaround that manually sets selenium version for edge (#23897) This appears to be a leftover from the previous use of the sauce runner, assumedly when there was some bug that required a specific version of selenium to be used (it was sadly undocumented). Fixes web-platform-tests/wpt#23895 -- wpt-commits: d5aae4ab7f99e024f70aa0d93ca3748809995263 wpt-pr: 23897 Differential Revision: https://phabricator.services.mozilla.com/D78777
xeonchen
pushed a commit
to xeonchen/gecko
that referenced
this issue
Jun 9, 2020
…sets selenium version for edge, a=testonly Automatic update from web-platform-tests [Sauce] Remove workaround that manually sets selenium version for edge (#23897) This appears to be a leftover from the previous use of the sauce runner, assumedly when there was some bug that required a specific version of selenium to be used (it was sadly undocumented). Fixes web-platform-tests/wpt#23895 -- wpt-commits: d5aae4ab7f99e024f70aa0d93ca3748809995263 wpt-pr: 23897 Differential Revision: https://phabricator.services.mozilla.com/D78777
moz-v2v-gh
pushed a commit
to mozilla/gecko-dev
that referenced
this issue
Jun 10, 2020
…sets selenium version for edge, a=testonly Automatic update from web-platform-tests [Sauce] Remove workaround that manually sets selenium version for edge (#23897) This appears to be a leftover from the previous use of the sauce runner, assumedly when there was some bug that required a specific version of selenium to be used (it was sadly undocumented). Fixes web-platform-tests/wpt#23895 -- wpt-commits: d5aae4ab7f99e024f70aa0d93ca3748809995263 wpt-pr: 23897 Differential Revision: https://phabricator.services.mozilla.com/D78777
xeonchen
pushed a commit
to xeonchen/gecko
that referenced
this issue
Jun 10, 2020
…sets selenium version for edge, a=testonly Automatic update from web-platform-tests [Sauce] Remove workaround that manually sets selenium version for edge (#23897) This appears to be a leftover from the previous use of the sauce runner, assumedly when there was some bug that required a specific version of selenium to be used (it was sadly undocumented). Fixes web-platform-tests/wpt#23895 -- wpt-commits: d5aae4ab7f99e024f70aa0d93ca3748809995263 wpt-pr: 23897 Differential Revision: https://phabricator.services.mozilla.com/D78777
gecko-dev-updater
pushed a commit
to marco-c/gecko-dev-wordified-and-comments-removed
that referenced
this issue
Jun 11, 2020
…sets selenium version for edge, a=testonly Automatic update from web-platform-tests [Sauce] Remove workaround that manually sets selenium version for edge (#23897) This appears to be a leftover from the previous use of the sauce runner, assumedly when there was some bug that required a specific version of selenium to be used (it was sadly undocumented). Fixes web-platform-tests/wpt#23895 -- wpt-commits: d5aae4ab7f99e024f70aa0d93ca3748809995263 wpt-pr: 23897 UltraBlame original commit: 6419d0a1929a03ecdff8277391b079cdf690d799
gecko-dev-updater
pushed a commit
to marco-c/gecko-dev-wordified-and-comments-removed
that referenced
this issue
Jun 11, 2020
…sets selenium version for edge, a=testonly Automatic update from web-platform-tests [Sauce] Remove workaround that manually sets selenium version for edge (#23897) This appears to be a leftover from the previous use of the sauce runner, assumedly when there was some bug that required a specific version of selenium to be used (it was sadly undocumented). Fixes web-platform-tests/wpt#23895 -- wpt-commits: d5aae4ab7f99e024f70aa0d93ca3748809995263 wpt-pr: 23897 Differential Revision: https://phabricator.services.mozilla.com/D78777 UltraBlame original commit: e31b0d2ecada5d458864f8a789138795ed283b25
gecko-dev-updater
pushed a commit
to marco-c/gecko-dev-wordified-and-comments-removed
that referenced
this issue
Jun 11, 2020
…sets selenium version for edge, a=testonly Automatic update from web-platform-tests [Sauce] Remove workaround that manually sets selenium version for edge (#23897) This appears to be a leftover from the previous use of the sauce runner, assumedly when there was some bug that required a specific version of selenium to be used (it was sadly undocumented). Fixes web-platform-tests/wpt#23895 -- wpt-commits: d5aae4ab7f99e024f70aa0d93ca3748809995263 wpt-pr: 23897 Differential Revision: https://phabricator.services.mozilla.com/D78777 UltraBlame original commit: ca68e559f5c23350273d8afcc1d5044d3d905fe0
gecko-dev-updater
pushed a commit
to marco-c/gecko-dev-comments-removed
that referenced
this issue
Jun 12, 2020
…sets selenium version for edge, a=testonly Automatic update from web-platform-tests [Sauce] Remove workaround that manually sets selenium version for edge (#23897) This appears to be a leftover from the previous use of the sauce runner, assumedly when there was some bug that required a specific version of selenium to be used (it was sadly undocumented). Fixes web-platform-tests/wpt#23895 -- wpt-commits: d5aae4ab7f99e024f70aa0d93ca3748809995263 wpt-pr: 23897 UltraBlame original commit: 6419d0a1929a03ecdff8277391b079cdf690d799
gecko-dev-updater
pushed a commit
to marco-c/gecko-dev-comments-removed
that referenced
this issue
Jun 12, 2020
…sets selenium version for edge, a=testonly Automatic update from web-platform-tests [Sauce] Remove workaround that manually sets selenium version for edge (#23897) This appears to be a leftover from the previous use of the sauce runner, assumedly when there was some bug that required a specific version of selenium to be used (it was sadly undocumented). Fixes web-platform-tests/wpt#23895 -- wpt-commits: d5aae4ab7f99e024f70aa0d93ca3748809995263 wpt-pr: 23897 Differential Revision: https://phabricator.services.mozilla.com/D78777 UltraBlame original commit: e31b0d2ecada5d458864f8a789138795ed283b25
gecko-dev-updater
pushed a commit
to marco-c/gecko-dev-comments-removed
that referenced
this issue
Jun 12, 2020
…sets selenium version for edge, a=testonly Automatic update from web-platform-tests [Sauce] Remove workaround that manually sets selenium version for edge (#23897) This appears to be a leftover from the previous use of the sauce runner, assumedly when there was some bug that required a specific version of selenium to be used (it was sadly undocumented). Fixes web-platform-tests/wpt#23895 -- wpt-commits: d5aae4ab7f99e024f70aa0d93ca3748809995263 wpt-pr: 23897 Differential Revision: https://phabricator.services.mozilla.com/D78777 UltraBlame original commit: ca68e559f5c23350273d8afcc1d5044d3d905fe0
gecko-dev-updater
pushed a commit
to marco-c/gecko-dev-wordified
that referenced
this issue
Jun 12, 2020
…sets selenium version for edge, a=testonly Automatic update from web-platform-tests [Sauce] Remove workaround that manually sets selenium version for edge (#23897) This appears to be a leftover from the previous use of the sauce runner, assumedly when there was some bug that required a specific version of selenium to be used (it was sadly undocumented). Fixes web-platform-tests/wpt#23895 -- wpt-commits: d5aae4ab7f99e024f70aa0d93ca3748809995263 wpt-pr: 23897 UltraBlame original commit: 6419d0a1929a03ecdff8277391b079cdf690d799
gecko-dev-updater
pushed a commit
to marco-c/gecko-dev-wordified
that referenced
this issue
Jun 12, 2020
…sets selenium version for edge, a=testonly Automatic update from web-platform-tests [Sauce] Remove workaround that manually sets selenium version for edge (#23897) This appears to be a leftover from the previous use of the sauce runner, assumedly when there was some bug that required a specific version of selenium to be used (it was sadly undocumented). Fixes web-platform-tests/wpt#23895 -- wpt-commits: d5aae4ab7f99e024f70aa0d93ca3748809995263 wpt-pr: 23897 Differential Revision: https://phabricator.services.mozilla.com/D78777 UltraBlame original commit: e31b0d2ecada5d458864f8a789138795ed283b25
gecko-dev-updater
pushed a commit
to marco-c/gecko-dev-wordified
that referenced
this issue
Jun 12, 2020
…sets selenium version for edge, a=testonly Automatic update from web-platform-tests [Sauce] Remove workaround that manually sets selenium version for edge (#23897) This appears to be a leftover from the previous use of the sauce runner, assumedly when there was some bug that required a specific version of selenium to be used (it was sadly undocumented). Fixes web-platform-tests/wpt#23895 -- wpt-commits: d5aae4ab7f99e024f70aa0d93ca3748809995263 wpt-pr: 23897 Differential Revision: https://phabricator.services.mozilla.com/D78777 UltraBlame original commit: ca68e559f5c23350273d8afcc1d5044d3d905fe0
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Currently, there seem to be a check for when the
--sauce-browserpassed in the runner CLI args is MicrosoftEdge and it makes it use older selenium version, I am not aware of any issues preventing Chromium based MS Edge to work onselenium@latestso I believe it's safe to remove.wpt/tools/wptrunner/wptrunner/browsers/sauce.py
Line 72 in d3ea885
The text was updated successfully, but these errors were encountered: