Skip to content

Commit 82a3bd5

Browse files
alan-agius4mhevery
authored andcommitted
ci: disable sauce labs safari and ios tests (angular#36973)
With this change we disable SauceLabs tests on Safari and iOS as they are current broken. Issue to investigate further: angular#36975 PR Close angular#36973
1 parent 5f3fba4 commit 82a3bd5

File tree

1 file changed

+11
-4
lines changed

1 file changed

+11
-4
lines changed

browser-providers.conf.js

Lines changed: 11 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -32,10 +32,17 @@ var CIconfiguration = {
3232
'Android8': {unitTest: {target: 'SL', required: true}, e2e: {target: null, required: true}},
3333
'Android9': {unitTest: {target: 'SL', required: true}, e2e: {target: null, required: true}},
3434
'Android10': {unitTest: {target: 'SL', required: true}, e2e: {target: null, required: true}},
35-
'Safari12': {unitTest: {target: 'SL', required: true}, e2e: {target: null, required: true}},
36-
'Safari13': {unitTest: {target: 'SL', required: true}, e2e: {target: null, required: true}},
37-
'iOS12': {unitTest: {target: 'SL', required: true}, e2e: {target: null, required: true}},
38-
'iOS13': {unitTest: {target: 'SL', required: true}, e2e: {target: null, required: true}},
35+
// Disable all Safari and iOS tests because of incorrect results
36+
// ex:
37+
// Mobile Safari 13.0.0 (iOS 13.0.0) styling static template only should capture static values in TStylingKey FAILED
38+
// Expected $.content = 'dynamic' to equal '"dynamic"'.
39+
// Mobile Safari 12.0.0 (iOS 12.0.0) styling should handle values wrapped into SafeValue FAILED
40+
// Expected 'url("http://angular-ci.local:9876/1.png")' to contain 'url("/1.png")'.s
41+
// Tracking in: https://github.com/angular/angular/issues/36975
42+
'Safari12': {unitTest: {target: 'SL', required: false}, e2e: {target: null, required: true}},
43+
'Safari13': {unitTest: {target: 'SL', required: false}, e2e: {target: null, required: true}},
44+
'iOS12': {unitTest: {target: 'SL', required: false}, e2e: {target: null, required: true}},
45+
'iOS13': {unitTest: {target: 'SL', required: false}, e2e: {target: null, required: true}},
3946
'WindowsPhone': {unitTest: {target: 'BS', required: false}, e2e: {target: null, required: true}}
4047
};
4148

0 commit comments

Comments
 (0)