Skip to content

Commit 0f1da49

Browse files
devversionkara
authored andcommitted
ci: temporarily disable chrome beta for saucelabs (angular#29518)
We need to temporarily disable Chrome beta within the `legacy-saucelabs-tests` and `test_saucelabs_bazel` job. This is necessary because Saucelabs added Chrome v74 to their beta version channel without providing the corresponding chromedriver that is *required* for that beta version. Explicitly specifying a chromedriver within the Selenium browser capabilities doesn't seem to work because Saucelabs seems to mirror their supported chromedriver versions. Meaning that we can't explicitly use chromedriver.storage.googleapis.com/index.html?path=74.0.3729.6/ Re-enabling tracked with: FW-1205 PR Close angular#29518
1 parent a5c9fa3 commit 0f1da49

File tree

1 file changed

+4
-1
lines changed

1 file changed

+4
-1
lines changed

Diff for: browser-providers.conf.js

+4-1
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,10 @@
1414
var CIconfiguration = {
1515
'Chrome': {unitTest: {target: 'SL', required: true}, e2e: {target: null, required: true}},
1616
'Firefox': {unitTest: {target: 'SL', required: true}, e2e: {target: null, required: true}},
17-
'ChromeBeta': {unitTest: {target: 'SL', required: true}, e2e: {target: null, required: false}},
17+
// Temporarily disabled because Saucelabs now runs Chrome v74 as beta version. Unfortunately
18+
// Saucelabs doesn't provide a Chromedriver version that works for that beta version though.
19+
// TODO(FW-1205): Re-enable once Saucelabs supports Chromedriver v74.0.3729.6.
20+
'ChromeBeta': {unitTest: {target: 'SL', required: false}, e2e: {target: null, required: false}},
1821
'ChromeDev': {unitTest: {target: 'SL', required: false}, e2e: {target: null, required: true}},
1922
// FirefoxBeta and FirefoxDev should be target:'BS' or target:'SL', and required:true
2023
// Currently deactivated due to https://github.com/angular/angular/issues/7560

0 commit comments

Comments
 (0)