Skip to content

Commit 65f4fc5

Browse files
nandajavarmaroboquat
authored andcommitted
Consolidate test alerts
1 parent b4edca5 commit 65f4fc5

File tree

1 file changed

+18
-11
lines changed

1 file changed

+18
-11
lines changed

.werft/installer-tests.ts

Lines changed: 18 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -205,51 +205,51 @@ const TESTS: { [name: string]: InfraConfig } = {
205205
WORKSPACE_TEST: {
206206
phase: "run-workspace-tests",
207207
makeTarget: "run-workspace-tests",
208-
description: "Run the workspace tests test/tests/workspace",
208+
description: "Workspace integration tests",
209209
slackhook: slackHook.get("workspace-jobs"),
210210
},
211211
VSCODE_IDE_TEST: {
212212
phase: "run-vscode-ide-tests",
213213
makeTarget: "run-vscode-ide-tests",
214-
description: "Run the vscode IDE tests test/tests/ide/vscode",
214+
description: "vscode IDE tests",
215215
slackhook: slackHook.get("ide-jobs"),
216216
},
217217
JB_IDE_TEST: {
218218
phase: "run-jb-ide-tests",
219219
makeTarget: "run-jb-ide-tests",
220-
description: "Run the jetbrains IDE tests test/tests/ide/jetbrains",
220+
description: "jetbrains IDE tests",
221221
slackhook: slackHook.get("ide-jobs"),
222222
},
223223
CONTENTSERVICE_TEST: {
224224
phase: "run-cs-component-tests",
225225
makeTarget: "run-cs-component-tests",
226-
description: "Run the content-service tests test/tests/components/content-service",
226+
description: "content-service tests",
227227
},
228228
DB_TEST: {
229229
phase: "run-db-component-tests",
230230
makeTarget: "run-db-component-tests",
231-
description: "Run the database tests test/tests/components/database",
231+
description: "database integration tests",
232232
},
233233
IMAGEBUILDER_TEST: {
234234
phase: "run-ib-component-tests",
235235
makeTarget: "run-ib-component-tests",
236-
description: "Run the image-builder tests test/tests/components/image-builder",
236+
description: "image-builder tests",
237237
},
238238
SERVER_TEST: {
239239
phase: "run-server-component-tests",
240240
makeTarget: "run-server-component-tests",
241-
description: "Run the server tests test/tests/components/server",
241+
description: "server integration tests",
242242
},
243243
WS_DAEMON_TEST: {
244244
phase: "run-wsd-component-tests",
245245
makeTarget: "run-wsd-component-tests",
246-
description: "Run the ws-daemon tests test/tests/components/ws-daemon",
246+
description: "ws-daemon integration tests",
247247
slackhook: slackHook.get("workspace-jobs"),
248248
},
249249
WS_MNGR_TEST: {
250250
phase: "run-wsm-component-tests",
251251
makeTarget: "run-wsm-component-tests",
252-
description: "Run the ws-manager tests test/tests/components/ws-manager",
252+
description: "ws-manager integration tests",
253253
slackhook: slackHook.get("workspace-jobs"),
254254
},
255255
}
@@ -345,6 +345,7 @@ export async function installerTests(config: TestConfig) {
345345

346346
function runIntegrationTests() {
347347
werft.phase("run-integration-tests", "Run all existing integration tests");
348+
const slackAlerts = new Map<string, string>([])
348349
for (let test in TESTS) {
349350
const testPhase = TESTS[test];
350351
const ret = callMakeTargets(testPhase.phase, testPhase.description, testPhase.makeTarget);
@@ -353,10 +354,16 @@ function runIntegrationTests() {
353354
`werft log result -d "failed test" url "${testPhase.description}(Phase ${testPhase.phase}) failed. Please refer logs."`,
354355
);
355356

356-
sendFailureSlackAlert(testPhase.description, new Error("Integration test failed"), testPhase.slackhook)
357+
const msg: string = slackAlerts.get(testPhase.slackhook) || ""
358+
slackAlerts.set(testPhase.slackhook, `${msg}\n${testPhase.description}`)
359+
357360
}
358361
}
359362

363+
slackAlerts.forEach((msg: string, channel: string) => {
364+
sendFailureSlackAlert(msg, new Error("Integration tests failed"), channel)
365+
});
366+
360367
werft.done("run-integration-tests");
361368
}
362369

@@ -411,7 +418,7 @@ function randK8sVersion(config: string): string {
411418
break;
412419
}
413420
case "STANDARD_EKS_TEST": {
414-
options = ["1.21", "1.22"]
421+
options = ["1.21", "1.22", "1.23"]
415422
break;
416423
}
417424
case "STANDARD_K3S_TEST": {

0 commit comments

Comments
 (0)