Skip to content

Commit 232fb64

Browse files
committed
chore: skip grid performance tests by default
Performance tests are flaky in CI due to resource constraints. They can be run explicitly with RUN_PERF_TESTS=1.
1 parent 9e18773 commit 232fb64

1 file changed

Lines changed: 18 additions & 0 deletions

File tree

tests/grid-performance.spec.ts

Lines changed: 18 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -130,7 +130,19 @@ function logResults(
130130
console.log(` Expected min FPS: ${expected.minFps}`);
131131
}
132132

133+
/**
134+
* Performance tests are skipped by default as they can be flaky in CI due to
135+
* resource constraints. To run these tests explicitly:
136+
*
137+
* RUN_PERF_TESTS=1 npx playwright test grid-performance.spec.ts
138+
*/
133139
test.describe('grid scroll performance benchmarks', () => {
140+
// Skip by default - these tests are flaky in CI due to resource constraints
141+
test.skip(
142+
!process.env.RUN_PERF_TESTS,
143+
'Performance tests skipped. Set RUN_PERF_TESTS=1 to run.'
144+
);
145+
134146
// Run tests serially to avoid resource contention
135147
test.describe.configure({ mode: 'serial' });
136148

@@ -304,6 +316,12 @@ test.describe('grid scroll performance benchmarks', () => {
304316
});
305317

306318
test.describe('grid performance stress tests', () => {
319+
// Skip by default - these tests are flaky in CI due to resource constraints
320+
test.skip(
321+
!process.env.RUN_PERF_TESTS,
322+
'Performance tests skipped. Set RUN_PERF_TESTS=1 to run.'
323+
);
324+
307325
// Run tests serially to avoid resource contention
308326
test.describe.configure({ mode: 'serial' });
309327

0 commit comments

Comments
 (0)