From c15c924c499c34c60e823ace238d99af907dbd78 Mon Sep 17 00:00:00 2001
From: Harrison Healey <harrisonmhealey@gmail.com>
Date: Wed, 2 Oct 2024 22:27:44 -0400
Subject: [PATCH] Disable PerformanceReporter tests (#28426)

---
 .../src/utils/performance_telemetry/reporter.test.ts         | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/webapp/channels/src/utils/performance_telemetry/reporter.test.ts b/webapp/channels/src/utils/performance_telemetry/reporter.test.ts
index 40cba5d238..3d2b9c4510 100644
--- a/webapp/channels/src/utils/performance_telemetry/reporter.test.ts
+++ b/webapp/channels/src/utils/performance_telemetry/reporter.test.ts
@@ -19,7 +19,10 @@ jest.mock('web-vitals/attribution');
 
 const siteUrl = 'http://localhost:8065';
 
-describe('PerformanceReporter', () => {
+// These tests are good to have, but they're incredibly unreliable in CI. These should be uncommented when making
+// changes to this code.
+// eslint-disable-next-line no-only-tests/no-only-tests
+describe.skip('PerformanceReporter', () => {
     afterEach(() => {
         performance.clearMarks();
         performance.clearMeasures();