Skip to content

Commit 083f975

Browse files
committed
Relax perf test throughput requirements in CI to avoid flakes
1 parent 5b53ab1 commit 083f975

File tree

1 file changed

+6
-0
lines changed

1 file changed

+6
-0
lines changed

test/performance/performance-test-helpers.ts

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -104,6 +104,12 @@ export function assertPerformance(
104104
maxErrors?: number;
105105
}
106106
): void {
107+
if (process.env.CI) {
108+
// CI can be slow, so we relax the throughput requirements plenty - we'll still spot this locally
109+
// or spot enormous CI regressions (and we can still monitor trends regardless).
110+
thresholds.minThroughput = thresholds.minThroughput ? thresholds.minThroughput * 0.6 : undefined;
111+
}
112+
107113
if (thresholds.minThroughput !== undefined) {
108114
expect(result.throughput).to.be.greaterThan(
109115
thresholds.minThroughput,

0 commit comments

Comments
 (0)