Commit cfc2d95f authored by Zhaoyang Li's avatar Zhaoyang Li Committed by Commit Bot

[iOS][code coverage] Run iOS CQ suites in iOS coverage builder.

iOS coverage builder should run the same tests in the same variants as
CQ builders. This change aligned the tests. Also added comment lines to
remind people keeping them in sync and fixed a typo.

Bug: 1090188
Change-Id: Iabd33a1226550ab1a26bc2613f8e707a56f942ad
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2469617Reviewed-by: default avatarGarrett Beaty <gbeaty@chromium.org>
Commit-Queue: Zhaoyang Li <zhaoyangli@chromium.org>
Cr-Commit-Position: refs/heads/master@{#817227}
parent c247207a
This source diff could not be displayed because it is too large. You can view the blob instead.
...@@ -5748,31 +5748,35 @@ ...@@ -5748,31 +5748,35 @@
} }
}, },
# This suite is a union of ios_simulator_tests and
# ios_simulator_full_configs_tests.
'ios_code_coverage_tests': { 'ios_code_coverage_tests': {
'ios_common_tests': { 'ios_common_tests': {
'variants': [ 'variants': [
'SIM_IPHONE_6S_12_4',
'SIM_IPHONE_6S_13_6', 'SIM_IPHONE_6S_13_6',
'SIM_IPHONE_6S_14_0',
], ],
}, },
'ios_eg2_cq_tests': { 'ios_eg2_cq_tests': {
'mixins': ['xcode_parallelization'], 'mixins': ['xcode_parallelization'],
'variants': [ 'variants': [
'SIM_IPHONE_6S_12_4',
'SIM_IPAD_AIR_2_12_4',
'SIM_IPHONE_6S_13_6', 'SIM_IPHONE_6S_13_6',
'SIM_IPHONE_7_13_6', 'SIM_IPHONE_7_13_6',
'SIM_IPAD_AIR_2_13_6', 'SIM_IPAD_AIR_2_13_6',
'SIM_IPHONE_6S_14_0',
'SIM_IPHONE_7_14_0',
'SIM_IPAD_AIR_2_14_0',
], ],
}, },
'ios_eg2_tests': { 'ios_eg2_tests': {
'mixins': ['xcode_parallelization'], 'mixins': ['xcode_parallelization'],
'variants': [ 'variants': [
'SIM_IPAD_AIR_2_12_4',
'SIM_IPHONE_X_12_4',
'SIM_IPHONE_7_13_6', 'SIM_IPHONE_7_13_6',
'SIM_IPAD_AIR_2_13_6', 'SIM_IPAD_AIR_2_13_6',
'SIM_IPHONE_X_13_6', 'SIM_IPHONE_X_13_6',
'SIM_IPHONE_7_14_0',
'SIM_IPAD_AIR_2_14_0',
'SIM_IPHONE_X_14_0',
], ],
}, },
'ios_screen_size_dependent_tests': { 'ios_screen_size_dependent_tests': {
...@@ -5780,7 +5784,11 @@ ...@@ -5780,7 +5784,11 @@
'SIM_IPHONE_6S_PLUS_13_6', 'SIM_IPHONE_6S_PLUS_13_6',
'SIM_IPHONE_6S_13_6', 'SIM_IPHONE_6S_13_6',
'SIM_IPHONE_SE_1ST_GEN_13_6', 'SIM_IPHONE_SE_1ST_GEN_13_6',
'SIM_IPAD_AIR_2_13_6' 'SIM_IPAD_AIR_2_13_6',
'SIM_IPHONE_6S_PLUS_14_0',
'SIM_IPHONE_6S_14_0',
'SIM_IPHONE_SE_1ST_GEN_14_0',
'SIM_IPAD_AIR_2_14_0',
], ],
}, },
}, },
...@@ -5794,7 +5802,8 @@ ...@@ -5794,7 +5802,8 @@
} }
}, },
'ios_simulator_full_configs_test': { # Please also change ios_code_coverage_tests for any change in this suite.
'ios_simulator_full_configs_tests': {
'ios_eg2_tests': { 'ios_eg2_tests': {
'mixins': ['xcode_parallelization'], 'mixins': ['xcode_parallelization'],
'variants': [ 'variants': [
...@@ -5870,6 +5879,7 @@ ...@@ -5870,6 +5879,7 @@
}, },
}, },
# Please also change ios_code_coverage_tests for any change in this suite.
'ios_simulator_tests': { 'ios_simulator_tests': {
'ios_common_tests': { 'ios_common_tests': {
'variants': [ 'variants': [
......
...@@ -4761,7 +4761,7 @@ ...@@ -4761,7 +4761,7 @@
'xctest', 'xctest',
], ],
'test_suites': { 'test_suites': {
'isolated_scripts': 'ios_simulator_full_configs_test' 'isolated_scripts': 'ios_simulator_full_configs_tests'
}, },
}, },
'ios-simulator-noncq': { 'ios-simulator-noncq': {
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment