Skip to content

Commit 5a84ff3

Browse files
Siqi-ShanSiqi-Shan
committed
fix: generate correct ios test build spec
Signed-off-by: Kevin Shan <[email protected]>
1 parent 6c69ae5 commit 5a84ff3

File tree

2 files changed

+25
-21
lines changed

2 files changed

+25
-21
lines changed

‎.codebuild/ios_canary_workflow.yml‎

Lines changed: 20 additions & 20 deletions
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,7 @@ batch:
2727
env:
2828
compute-type: BUILD_GENERAL1_LARGE
2929
variables:
30-
TEST_SUITE: src/__tests__/build-app-ios.test.ts
30+
TEST_SUITE: src/__tests__/build-app-swift.test.ts
3131
CLI_REGION: ap-east-1
3232
CANARY_METRIC_NAME: IosAppBuildCodegenSuccessRate
3333
depend-on:
@@ -37,7 +37,7 @@ batch:
3737
env:
3838
compute-type: BUILD_GENERAL1_LARGE
3939
variables:
40-
TEST_SUITE: src/__tests__/build-app-ios.test.ts
40+
TEST_SUITE: src/__tests__/build-app-swift.test.ts
4141
CLI_REGION: ap-northeast-1
4242
CANARY_METRIC_NAME: IosAppBuildCodegenSuccessRate
4343
depend-on:
@@ -47,7 +47,7 @@ batch:
4747
env:
4848
compute-type: BUILD_GENERAL1_LARGE
4949
variables:
50-
TEST_SUITE: src/__tests__/build-app-ios.test.ts
50+
TEST_SUITE: src/__tests__/build-app-swift.test.ts
5151
CLI_REGION: ap-northeast-2
5252
CANARY_METRIC_NAME: IosAppBuildCodegenSuccessRate
5353
depend-on:
@@ -57,7 +57,7 @@ batch:
5757
env:
5858
compute-type: BUILD_GENERAL1_LARGE
5959
variables:
60-
TEST_SUITE: src/__tests__/build-app-ios.test.ts
60+
TEST_SUITE: src/__tests__/build-app-swift.test.ts
6161
CLI_REGION: ap-northeast-3
6262
CANARY_METRIC_NAME: IosAppBuildCodegenSuccessRate
6363
depend-on:
@@ -67,7 +67,7 @@ batch:
6767
env:
6868
compute-type: BUILD_GENERAL1_LARGE
6969
variables:
70-
TEST_SUITE: src/__tests__/build-app-ios.test.ts
70+
TEST_SUITE: src/__tests__/build-app-swift.test.ts
7171
CLI_REGION: ap-south-1
7272
CANARY_METRIC_NAME: IosAppBuildCodegenSuccessRate
7373
depend-on:
@@ -77,7 +77,7 @@ batch:
7777
env:
7878
compute-type: BUILD_GENERAL1_LARGE
7979
variables:
80-
TEST_SUITE: src/__tests__/build-app-ios.test.ts
80+
TEST_SUITE: src/__tests__/build-app-swift.test.ts
8181
CLI_REGION: ap-southeast-1
8282
CANARY_METRIC_NAME: IosAppBuildCodegenSuccessRate
8383
depend-on:
@@ -87,7 +87,7 @@ batch:
8787
env:
8888
compute-type: BUILD_GENERAL1_LARGE
8989
variables:
90-
TEST_SUITE: src/__tests__/build-app-ios.test.ts
90+
TEST_SUITE: src/__tests__/build-app-swift.test.ts
9191
CLI_REGION: ap-southeast-2
9292
CANARY_METRIC_NAME: IosAppBuildCodegenSuccessRate
9393
depend-on:
@@ -97,7 +97,7 @@ batch:
9797
env:
9898
compute-type: BUILD_GENERAL1_LARGE
9999
variables:
100-
TEST_SUITE: src/__tests__/build-app-ios.test.ts
100+
TEST_SUITE: src/__tests__/build-app-swift.test.ts
101101
CLI_REGION: ca-central-1
102102
CANARY_METRIC_NAME: IosAppBuildCodegenSuccessRate
103103
depend-on:
@@ -107,7 +107,7 @@ batch:
107107
env:
108108
compute-type: BUILD_GENERAL1_LARGE
109109
variables:
110-
TEST_SUITE: src/__tests__/build-app-ios.test.ts
110+
TEST_SUITE: src/__tests__/build-app-swift.test.ts
111111
CLI_REGION: eu-central-1
112112
CANARY_METRIC_NAME: IosAppBuildCodegenSuccessRate
113113
depend-on:
@@ -117,7 +117,7 @@ batch:
117117
env:
118118
compute-type: BUILD_GENERAL1_LARGE
119119
variables:
120-
TEST_SUITE: src/__tests__/build-app-ios.test.ts
120+
TEST_SUITE: src/__tests__/build-app-swift.test.ts
121121
CLI_REGION: eu-north-1
122122
CANARY_METRIC_NAME: IosAppBuildCodegenSuccessRate
123123
depend-on:
@@ -127,7 +127,7 @@ batch:
127127
env:
128128
compute-type: BUILD_GENERAL1_LARGE
129129
variables:
130-
TEST_SUITE: src/__tests__/build-app-ios.test.ts
130+
TEST_SUITE: src/__tests__/build-app-swift.test.ts
131131
CLI_REGION: eu-south-1
132132
CANARY_METRIC_NAME: IosAppBuildCodegenSuccessRate
133133
depend-on:
@@ -137,7 +137,7 @@ batch:
137137
env:
138138
compute-type: BUILD_GENERAL1_LARGE
139139
variables:
140-
TEST_SUITE: src/__tests__/build-app-ios.test.ts
140+
TEST_SUITE: src/__tests__/build-app-swift.test.ts
141141
CLI_REGION: eu-west-1
142142
CANARY_METRIC_NAME: IosAppBuildCodegenSuccessRate
143143
depend-on:
@@ -147,7 +147,7 @@ batch:
147147
env:
148148
compute-type: BUILD_GENERAL1_LARGE
149149
variables:
150-
TEST_SUITE: src/__tests__/build-app-ios.test.ts
150+
TEST_SUITE: src/__tests__/build-app-swift.test.ts
151151
CLI_REGION: eu-west-2
152152
CANARY_METRIC_NAME: IosAppBuildCodegenSuccessRate
153153
depend-on:
@@ -157,7 +157,7 @@ batch:
157157
env:
158158
compute-type: BUILD_GENERAL1_LARGE
159159
variables:
160-
TEST_SUITE: src/__tests__/build-app-ios.test.ts
160+
TEST_SUITE: src/__tests__/build-app-swift.test.ts
161161
CLI_REGION: eu-west-3
162162
CANARY_METRIC_NAME: IosAppBuildCodegenSuccessRate
163163
depend-on:
@@ -167,7 +167,7 @@ batch:
167167
env:
168168
compute-type: BUILD_GENERAL1_LARGE
169169
variables:
170-
TEST_SUITE: src/__tests__/build-app-ios.test.ts
170+
TEST_SUITE: src/__tests__/build-app-swift.test.ts
171171
CLI_REGION: me-south-1
172172
CANARY_METRIC_NAME: IosAppBuildCodegenSuccessRate
173173
depend-on:
@@ -177,7 +177,7 @@ batch:
177177
env:
178178
compute-type: BUILD_GENERAL1_LARGE
179179
variables:
180-
TEST_SUITE: src/__tests__/build-app-ios.test.ts
180+
TEST_SUITE: src/__tests__/build-app-swift.test.ts
181181
CLI_REGION: sa-east-1
182182
CANARY_METRIC_NAME: IosAppBuildCodegenSuccessRate
183183
depend-on:
@@ -187,7 +187,7 @@ batch:
187187
env:
188188
compute-type: BUILD_GENERAL1_LARGE
189189
variables:
190-
TEST_SUITE: src/__tests__/build-app-ios.test.ts
190+
TEST_SUITE: src/__tests__/build-app-swift.test.ts
191191
CLI_REGION: us-east-1
192192
CANARY_METRIC_NAME: IosAppBuildCodegenSuccessRate
193193
depend-on:
@@ -197,7 +197,7 @@ batch:
197197
env:
198198
compute-type: BUILD_GENERAL1_LARGE
199199
variables:
200-
TEST_SUITE: src/__tests__/build-app-ios.test.ts
200+
TEST_SUITE: src/__tests__/build-app-swift.test.ts
201201
CLI_REGION: us-east-2
202202
CANARY_METRIC_NAME: IosAppBuildCodegenSuccessRate
203203
depend-on:
@@ -207,7 +207,7 @@ batch:
207207
env:
208208
compute-type: BUILD_GENERAL1_LARGE
209209
variables:
210-
TEST_SUITE: src/__tests__/build-app-ios.test.ts
210+
TEST_SUITE: src/__tests__/build-app-swift.test.ts
211211
CLI_REGION: us-west-1
212212
CANARY_METRIC_NAME: IosAppBuildCodegenSuccessRate
213213
depend-on:
@@ -217,7 +217,7 @@ batch:
217217
env:
218218
compute-type: BUILD_GENERAL1_LARGE
219219
variables:
220-
TEST_SUITE: src/__tests__/build-app-ios.test.ts
220+
TEST_SUITE: src/__tests__/build-app-swift.test.ts
221221
CLI_REGION: us-west-2
222222
CANARY_METRIC_NAME: IosAppBuildCodegenSuccessRate
223223
depend-on:

‎scripts/split-canary-tests.ts‎

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -25,6 +25,7 @@ enum ComputeType {
2525
}
2626
type PlatformConfig = {
2727
platform: string;
28+
testName: string;
2829
metric: string;
2930
outputPath: string;
3031
}
@@ -66,16 +67,19 @@ const getPlatformsConfig = (): PlatformConfig[] => {
6667
return [
6768
{
6869
platform: 'ts',
70+
testName: 'ts',
6971
metric: 'TsAppBuildCodegenSuccessRate',
7072
outputPath: getOutputPath('ts_canary_workflow.yml')
7173
},
7274
{
7375
platform: 'ios',
76+
testName: 'swift',
7477
metric: 'IosAppBuildCodegenSuccessRate',
7578
outputPath: getOutputPath('ios_canary_workflow.yml')
7679
},
7780
{
7881
platform: 'android',
82+
testName: 'android',
7983
metric: 'AndroidAppBuildCodegenSuccessRate',
8084
outputPath: getOutputPath('android_canary_workflow.yml')
8185
},
@@ -129,7 +133,7 @@ const getBuildspec = (jobConfig: JobConfig): string => {
129133
};
130134

131135
const getTestSuite = (jobConfig: JobConfig): string => {
132-
return `src/__tests__/build-app-${jobConfig.platformConfig.platform}.test.ts`;
136+
return `src/__tests__/build-app-${jobConfig.platformConfig.testName}.test.ts`;
133137
};
134138

135139
const main = (): void => {

0 commit comments

Comments
 (0)