From 888f43c6ad8444100cdcbf42c2ee0ae4f80c5ea3 Mon Sep 17 00:00:00 2001 From: Maxime Beauchemin Date: Wed, 23 Nov 2022 10:19:59 -0800 Subject: [PATCH] chore: rename 'Change dataset' action to 'Swap dataset' (#22197) --- .../src/components/Datasource/ChangeDatasourceModal.tsx | 2 +- .../controls/DatasourceControl/DatasourceControl.test.tsx | 8 ++++---- .../components/controls/DatasourceControl/index.jsx | 4 ++-- 3 files changed, 7 insertions(+), 7 deletions(-) diff --git a/superset-frontend/src/components/Datasource/ChangeDatasourceModal.tsx b/superset-frontend/src/components/Datasource/ChangeDatasourceModal.tsx index a7c29c51d9..fc31c0ec7c 100644 --- a/superset-frontend/src/components/Datasource/ChangeDatasourceModal.tsx +++ b/superset-frontend/src/components/Datasource/ChangeDatasourceModal.tsx @@ -251,7 +251,7 @@ const ChangeDatasourceModal: FunctionComponent = ({ show={show} onHide={onHide} responsive - title={t('Change dataset')} + title={t('Swap dataset')} width={confirmChange ? '432px' : ''} height={confirmChange ? 'auto' : '540px'} hideFooter={!confirmChange} diff --git a/superset-frontend/src/explore/components/controls/DatasourceControl/DatasourceControl.test.tsx b/superset-frontend/src/explore/components/controls/DatasourceControl/DatasourceControl.test.tsx index 8e41812fb8..231300711e 100644 --- a/superset-frontend/src/explore/components/controls/DatasourceControl/DatasourceControl.test.tsx +++ b/superset-frontend/src/explore/components/controls/DatasourceControl/DatasourceControl.test.tsx @@ -94,17 +94,17 @@ test('Should open a menu', async () => { render(); expect(screen.queryByText('Edit dataset')).not.toBeInTheDocument(); - expect(screen.queryByText('Change dataset')).not.toBeInTheDocument(); + expect(screen.queryByText('Swap dataset')).not.toBeInTheDocument(); expect(screen.queryByText('View in SQL Lab')).not.toBeInTheDocument(); userEvent.click(screen.getByTestId('datasource-menu-trigger')); expect(await screen.findByText('Edit dataset')).toBeInTheDocument(); - expect(screen.getByText('Change dataset')).toBeInTheDocument(); + expect(screen.getByText('Swap dataset')).toBeInTheDocument(); expect(screen.getByText('View in SQL Lab')).toBeInTheDocument(); }); -test('Click on Change dataset option', async () => { +test('Click on Swap dataset option', async () => { const props = createProps(); SupersetClientGet.mockImplementation( async ({ endpoint }: { endpoint: string }) => { @@ -123,7 +123,7 @@ test('Click on Change dataset option', async () => { userEvent.click(screen.getByTestId('datasource-menu-trigger')); await act(async () => { - userEvent.click(screen.getByText('Change dataset')); + userEvent.click(screen.getByText('Swap dataset')); }); expect( screen.getByText( diff --git a/superset-frontend/src/explore/components/controls/DatasourceControl/index.jsx b/superset-frontend/src/explore/components/controls/DatasourceControl/index.jsx index 266593adae..2dfa78363f 100644 --- a/superset-frontend/src/explore/components/controls/DatasourceControl/index.jsx +++ b/superset-frontend/src/explore/components/controls/DatasourceControl/index.jsx @@ -302,7 +302,7 @@ class DatasourceControl extends React.PureComponent { )} )} - {t('Change dataset')} + {t('Swap dataset')} {datasource && ( {t('View in SQL Lab')} )} @@ -421,7 +421,7 @@ class DatasourceControl extends React.PureComponent { this.handleMenuItemClick({ key: CHANGE_DATASET }) } > - {t('Change dataset')} + {t('Swap dataset')}