Skip to content

Commit a79b7a6

Browse files
committed
Update total checks calulations
1 parent d8ed25c commit a79b7a6

File tree

1 file changed

+9
-3
lines changed

1 file changed

+9
-3
lines changed

src/markdownRenderer.ts

+9-3
Original file line numberDiff line numberDiff line change
@@ -268,24 +268,30 @@ export function getChecksMarkdown(
268268
// Aggregate checks by name
269269
const checksByName: Record<
270270
string,
271-
{ success_count: number; fail_count: number }
271+
{ success_count: number; fail_count: number; total_count: number }
272272
> = {}
273273

274274
// Group checks by name and aggregate metrics
275275
checks.forEach((check) => {
276276
const { name, metric_summary } = check
277277
if (!checksByName[name]) {
278-
checksByName[name] = { success_count: 0, fail_count: 0 }
278+
checksByName[name] = {
279+
success_count: 0,
280+
fail_count: 0,
281+
total_count: 0,
282+
}
279283
}
280284
checksByName[name].success_count += metric_summary.success_count
281285
checksByName[name].fail_count += metric_summary.fail_count
286+
checksByName[name].total_count +=
287+
metric_summary.success_count + metric_summary.fail_count
282288
})
283289
// List failed checks (those with fail_count > 0)
284290
Object.entries(checksByName)
285291
.filter(([, metrics]) => metrics.fail_count > 0)
286292
.forEach(([name, metrics]) => {
287293
markdownSections.push(
288-
` - \`${name}\`: Failed **${formatNumber(metrics.fail_count)}**, out of **${formatNumber(metrics.success_count + metrics.fail_count)}** times.`
294+
` - \`${name}\`: Failed **${formatNumber(metrics.fail_count)}**, out of **${formatNumber(metrics.total_count)}** times.`
289295
)
290296
})
291297
}

0 commit comments

Comments
 (0)