Skip to content

Commit a45446c

Browse files
fix report, add skipped to report
Signed-off-by: Nikita Korolev <nikita.korolev@flant.com>
1 parent 4f571e1 commit a45446c

2 files changed

Lines changed: 17 additions & 11 deletions

File tree

.github/scripts/js/e2e/report/messenger-report.js

Lines changed: 7 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -67,6 +67,7 @@ function createMissingReport(clusterName, fallback = {}) {
6767
passed: 0,
6868
failed: 0,
6969
errors: 0,
70+
skipped: 0,
7071
total: 0,
7172
successRate: 0,
7273
},
@@ -320,18 +321,18 @@ function buildMainMessage(orderedReports) {
320321
lines.push("### Test results");
321322
lines.push("");
322323
lines.push(
323-
"| Cluster | ✅ Passed | ❌ Failed | ⚠️ Errors | Total | Success Rate |"
324+
"| Cluster | ✅ Passed | ⏭️ Skipped | ❌ Failed | ⚠️ Errors | Total | Success Rate |"
324325
);
325-
lines.push("|---|---:|---:|---:|---:|---:|");
326+
lines.push("|---|---:|---:|---:|---:|---:|---:|");
326327

327328
for (const report of testsReports) {
328329
const metrics = report.metrics || {};
329330
lines.push(
330331
`| ${formatClusterLink(report)} | ${metrics.passed || 0} | ${
331-
metrics.failed || 0
332-
} | ${metrics.errors || 0} | ${metrics.total || 0} | ${formatRate(
333-
metrics.successRate
334-
)} |`
332+
metrics.skipped || 0
333+
} | ${metrics.failed || 0} | ${metrics.errors || 0} | ${
334+
metrics.total || 0
335+
} | ${formatRate(metrics.successRate)} |`
335336
);
336337
}
337338

.github/scripts/js/e2e/report/messenger-report.test.js

Lines changed: 10 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -81,10 +81,11 @@ describe("messenger-report", () => {
8181
startedAt: "2026-04-15T09:30:44",
8282
metrics: {
8383
passed: 12,
84+
skipped: 2,
8485
failed: 1,
8586
errors: 0,
86-
total: 13,
87-
successRate: 92.31,
87+
total: 15,
88+
successRate: 80,
8889
},
8990
failedTests: ["[It] fails"],
9091
})
@@ -118,7 +119,7 @@ describe("messenger-report", () => {
118119

119120
expect(result.message).toContain("### Test results");
120121
expect(result.message).toContain(
121-
"| [replicated](https://example.invalid/replicated) | 12 | 1 | 0 | 13 | 92.31% |"
122+
"| [replicated](https://example.invalid/replicated) | 12 | 2 | 1 | 0 | 15 | 80.00% |"
122123
);
123124
expect(result.message).toContain("### Cluster failures");
124125
expect(result.message).toContain(
@@ -200,10 +201,11 @@ describe("messenger-report", () => {
200201
startedAt: "2026-04-15T09:30:44",
201202
metrics: {
202203
passed: 10,
204+
skipped: 1,
203205
failed: 1,
204206
errors: 0,
205-
total: 11,
206-
successRate: 90.91,
207+
total: 12,
208+
successRate: 83.33,
207209
},
208210
failedTests: ["[It] fails"],
209211
})
@@ -266,6 +268,7 @@ describe("messenger-report", () => {
266268
startedAt: "2026-04-15T09:30:44",
267269
metrics: {
268270
passed: 11,
271+
skipped: 0,
269272
failed: 0,
270273
errors: 0,
271274
total: 11,
@@ -311,6 +314,7 @@ describe("messenger-report", () => {
311314
startedAt: "2026-04-15T09:30:44",
312315
metrics: {
313316
passed: 11,
317+
skipped: 0,
314318
failed: 0,
315319
errors: 0,
316320
total: 11,
@@ -356,6 +360,7 @@ describe("messenger-report", () => {
356360
startedAt: "2026-04-15T09:30:44",
357361
metrics: {
358362
passed: 11,
363+
skipped: 0,
359364
failed: 0,
360365
errors: 0,
361366
total: 11,

0 commit comments

Comments
 (0)