This is an automated email from the ASF dual-hosted git repository.

enzomartellucci pushed a commit to branch enxdev/fix/chart-error-alerts
in repository https://gitbox.apache.org/repos/asf/superset.git

commit 9d0d365b71378f6f96283184f69cee2a428030d5
Author: Enzo Martellucci <[email protected]>
AuthorDate: Mon Feb 16 15:16:26 2026 +0100

    test(chart): add test verifying chart error banners are not dismissible
---
 .../src/components/Chart/ChartErrorMessage.test.tsx         | 13 +++++++++++++
 1 file changed, 13 insertions(+)

diff --git a/superset-frontend/src/components/Chart/ChartErrorMessage.test.tsx 
b/superset-frontend/src/components/Chart/ChartErrorMessage.test.tsx
index 1680b0e7602..48c19c80616 100644
--- a/superset-frontend/src/components/Chart/ChartErrorMessage.test.tsx
+++ b/superset-frontend/src/components/Chart/ChartErrorMessage.test.tsx
@@ -82,4 +82,17 @@ describe('ChartErrorMessage', () => {
     expect(screen.getByText('Test error')).toBeInTheDocument();
     expect(screen.getByText('Test subtitle')).toBeInTheDocument();
   });
+
+  test('chart error banner is not dismissible', () => {
+    mockUseChartOwnerNames.mockReturnValue({
+      result: null,
+      status: ResourceStatus.Loading,
+      error: null,
+    });
+    render(<ChartErrorMessage {...defaultProps} />);
+
+    expect(
+      screen.queryByRole('button', { name: 'Close' }),
+    ).not.toBeInTheDocument();
+  });
 });

Reply via email to