From 17f67436e2179a333cea1e5f467a0bcbbace439a Mon Sep 17 00:00:00 2001 From: Billy Vong Date: Thu, 1 May 2025 12:55:25 -0400 Subject: [PATCH] test(js): Increase timeout for `ChartWidgetLoader` test This flaked here https://sentry.sentry.io/issues/6580195034/?project=4857230&referrer=github-pr-bot - lets try bumping up the timeout --- .../chartWidgetLoader-unmocked-imports.spec.tsx | 11 ++++++++--- 1 file changed, 8 insertions(+), 3 deletions(-) diff --git a/static/app/components/charts/chartWidgetLoader-unmocked-imports.spec.tsx b/static/app/components/charts/chartWidgetLoader-unmocked-imports.spec.tsx index f6a915978e135b..0b621b6e9de577 100644 --- a/static/app/components/charts/chartWidgetLoader-unmocked-imports.spec.tsx +++ b/static/app/components/charts/chartWidgetLoader-unmocked-imports.spec.tsx @@ -282,9 +282,14 @@ describe('ChartWidgetLoader - unmocked imports', () => { // We only need to check that the dynamic import completes for these tests as that means ChartWidgetLoader is able to load all widgets expect(screen.getByTestId('loading-placeholder')).toBeInTheDocument(); - await waitFor(() => { - expect(screen.queryByTestId('loading-placeholder')).not.toBeInTheDocument(); - }); + await waitFor( + () => { + expect(screen.queryByTestId('loading-placeholder')).not.toBeInTheDocument(); + }, + { + timeout: 2000, + } + ); expect(TimeSeriesWidgetVisualization).toHaveBeenCalledWith( expect.objectContaining({