From bebb10e4957bd1bb67ac799397825d340c8bd029 Mon Sep 17 00:00:00 2001 From: Josh Soref <2119212+jsoref@users.noreply.github.com> Date: Tue, 26 Apr 2022 13:35:01 -0400 Subject: [PATCH] chore(frontend-tests): Spelling (#19853) * spelling: against Signed-off-by: Josh Soref * spelling: been Signed-off-by: Josh Soref * spelling: charts Signed-off-by: Josh Soref * spelling: clicking Signed-off-by: Josh Soref * spelling: columns Signed-off-by: Josh Soref * spelling: duplicate Signed-off-by: Josh Soref * spelling: especially Signed-off-by: Josh Soref * spelling: extensions Signed-off-by: Josh Soref * spelling: fields Signed-off-by: Josh Soref * spelling: filter Signed-off-by: Josh Soref * spelling: for Signed-off-by: Josh Soref * spelling: label Signed-off-by: Josh Soref * spelling: labeled Signed-off-by: Josh Soref * spelling: nativefilter Signed-off-by: Josh Soref * spelling: registry Signed-off-by: Josh Soref * spelling: render Signed-off-by: Josh Soref * spelling: resizable Signed-off-by: Josh Soref * spelling: response Signed-off-by: Josh Soref * spelling: successful Signed-off-by: Josh Soref * spelling: transform Signed-off-by: Josh Soref * spelling: unfortunately Signed-off-by: Josh Soref * spelling: until Signed-off-by: Josh Soref * spelling: virtual Signed-off-by: Josh Soref * spelling: wrapper Signed-off-by: Josh Soref Co-authored-by: Josh Soref --- .../cypress/integration/dashboard/key_value.test.ts | 4 ++-- .../integration/explore/visualizations/bubble.test.js | 4 ++-- .../cypress/integration/sqllab/query.test.ts | 2 +- .../templates/test/plugin/transformProps.test.erb | 4 ++-- .../NumberFormatterRegistrySingleton.test.ts | 2 +- .../superset-ui-core/test/query/api/v1/makeApi.test.ts | 2 +- .../time-format/TimeFormatterRegistrySingleton.test.ts | 2 +- .../test/translation/Translator.test.ts | 2 +- .../test/BoxPlot/transformProps.test.ts | 4 ++-- .../test/Funnel/transformProps.test.ts | 4 ++-- .../test/Graph/transformProps.test.ts | 6 +++--- .../test/Pie/transformProps.test.ts | 4 ++-- .../test/Timeseries/transformProps.test.ts | 2 +- .../test/Tree/transformProps.test.ts | 6 +++--- .../test/Treemap/transformProps.test.ts | 4 ++-- .../plugin-chart-echarts/test/utils/series.test.ts | 2 +- .../plugin-chart-table/test/TableChart.test.tsx | 4 ++-- .../test/legacyPlugin/transformProps.test.ts | 4 ++-- .../components/Datasource/DatasourceEditor.test.jsx | 10 +++++----- .../components/Form/LabeledErrorBoundInput.test.jsx | 2 +- .../CrossFilterScopingForm.test.tsx | 2 +- .../components/filterscope/FilterScope.test.tsx | 4 ++-- .../FiltersConfigForm/getControlItemsMap.test.tsx | 6 +++--- .../dashboard/util/getDetailedComponentWidth.test.js | 6 +++--- .../components/controls/withAsyncVerification.test.tsx | 2 +- .../src/views/CRUD/chart/ChartList.test.jsx | 4 ++-- .../src/views/CRUD/data/dataset/DatasetList.test.jsx | 2 +- superset-frontend/src/views/CRUD/utils.test.tsx | 2 +- 28 files changed, 51 insertions(+), 51 deletions(-) diff --git a/superset-frontend/cypress-base/cypress/integration/dashboard/key_value.test.ts b/superset-frontend/cypress-base/cypress/integration/dashboard/key_value.test.ts index 24b6ff0aa7..1738ea5bd3 100644 --- a/superset-frontend/cypress-base/cypress/integration/dashboard/key_value.test.ts +++ b/superset-frontend/cypress-base/cypress/integration/dashboard/key_value.test.ts @@ -27,7 +27,7 @@ interface QueryString { native_filters_key: string; } -xdescribe('nativefiler url param key', () => { +xdescribe('nativefilter url param key', () => { // const urlParams = { param1: '123', param2: 'abc' }; before(() => { cy.login(); @@ -36,7 +36,7 @@ xdescribe('nativefiler url param key', () => { let initialFilterKey: string; it('should have cachekey in nativefilter param', () => { // things in `before` will not retry and the `waitForChartLoad` check is - // especically flaky and may need more retries + // especially flaky and may need more retries cy.visit(WORLD_HEALTH_DASHBOARD); WORLD_HEALTH_CHARTS.forEach(waitForChartLoad); cy.wait(1000); // wait for key to be published (debounced) diff --git a/superset-frontend/cypress-base/cypress/integration/explore/visualizations/bubble.test.js b/superset-frontend/cypress-base/cypress/integration/explore/visualizations/bubble.test.js index 7ed17b1a4d..9bd91f37c5 100644 --- a/superset-frontend/cypress-base/cypress/integration/explore/visualizations/bubble.test.js +++ b/superset-frontend/cypress-base/cypress/integration/explore/visualizations/bubble.test.js @@ -57,8 +57,8 @@ describe('Visualization > Bubble', () => { }); // Number of circles are pretty unstable when there are a lot of circles - // Since main functionality is already covered in fitler test below, - // skip this test untill we find a solution. + // Since main functionality is already covered in filter test below, + // skip this test until we find a solution. it.skip('should work', () => { cy.visitChartByParams(JSON.stringify(BUBBLE_FORM_DATA)).then(() => { cy.wait('@getJson').then(xhr => { diff --git a/superset-frontend/cypress-base/cypress/integration/sqllab/query.test.ts b/superset-frontend/cypress-base/cypress/integration/sqllab/query.test.ts index f5033313fc..33b7caf551 100644 --- a/superset-frontend/cypress-base/cypress/integration/sqllab/query.test.ts +++ b/superset-frontend/cypress-base/cypress/integration/sqllab/query.test.ts @@ -114,7 +114,7 @@ describe('SqlLab query panel', () => { cy.wait('@sqlLabQuery'); - // Save results to check agains below + // Save results to check against below selectResultsTab().then(resultsA => { initialResultsTable = resultsA[0]; }); diff --git a/superset-frontend/packages/generator-superset/generators/plugin-chart/templates/test/plugin/transformProps.test.erb b/superset-frontend/packages/generator-superset/generators/plugin-chart/templates/test/plugin/transformProps.test.erb index 608f3084fa..5081180b6b 100644 --- a/superset-frontend/packages/generator-superset/generators/plugin-chart/templates/test/plugin/transformProps.test.erb +++ b/superset-frontend/packages/generator-superset/generators/plugin-chart/templates/test/plugin/transformProps.test.erb @@ -19,7 +19,7 @@ import { ChartProps } from '@superset-ui/core'; import transformProps from '../../src/plugin/transformProps'; -describe('<%= packageLabel %> tranformProps', () => { +describe('<%= packageLabel %> transformProps', () => { const formData = { colorScheme: 'bnbColors', datasource: '3__table', @@ -39,7 +39,7 @@ describe('<%= packageLabel %> tranformProps', () => { }], }); - it('should tranform chart props for viz', () => { + it('should transform chart props for viz', () => { expect(transformProps(chartProps)).toEqual({ width: 800, height: 600, diff --git a/superset-frontend/packages/superset-ui-core/test/number-format/NumberFormatterRegistrySingleton.test.ts b/superset-frontend/packages/superset-ui-core/test/number-format/NumberFormatterRegistrySingleton.test.ts index b978f5b27d..dbb7eac6df 100644 --- a/superset-frontend/packages/superset-ui-core/test/number-format/NumberFormatterRegistrySingleton.test.ts +++ b/superset-frontend/packages/superset-ui-core/test/number-format/NumberFormatterRegistrySingleton.test.ts @@ -26,7 +26,7 @@ import { describe('NumberFormatterRegistrySingleton', () => { describe('getNumberFormatterRegistry()', () => { - it('returns a NumberFormatterRegisry', () => { + it('returns a NumberFormatterRegistry', () => { expect(getNumberFormatterRegistry()).toBeInstanceOf( NumberFormatterRegistry, ); diff --git a/superset-frontend/packages/superset-ui-core/test/query/api/v1/makeApi.test.ts b/superset-frontend/packages/superset-ui-core/test/query/api/v1/makeApi.test.ts index 899011b281..f8cd445250 100644 --- a/superset-frontend/packages/superset-ui-core/test/query/api/v1/makeApi.test.ts +++ b/superset-frontend/packages/superset-ui-core/test/query/api/v1/makeApi.test.ts @@ -204,7 +204,7 @@ describe('makeApi()', () => { expect(result).toBe('ok?'); }); - it('should return raw resposnse when responseType=raw', async () => { + it('should return raw response when responseType=raw', async () => { expect.assertions(2); const api = makeApi({ method: 'DELETE', diff --git a/superset-frontend/packages/superset-ui-core/test/time-format/TimeFormatterRegistrySingleton.test.ts b/superset-frontend/packages/superset-ui-core/test/time-format/TimeFormatterRegistrySingleton.test.ts index eb406c3ca8..b0a0c0306f 100644 --- a/superset-frontend/packages/superset-ui-core/test/time-format/TimeFormatterRegistrySingleton.test.ts +++ b/superset-frontend/packages/superset-ui-core/test/time-format/TimeFormatterRegistrySingleton.test.ts @@ -31,7 +31,7 @@ import TimeFormatterRegistry from '../../src/time-format/TimeFormatterRegistry'; describe('TimeFormatterRegistrySingleton', () => { describe('getTimeFormatterRegistry()', () => { - it('returns a TimeFormatterRegisry', () => { + it('returns a TimeFormatterRegistry', () => { expect(getTimeFormatterRegistry()).toBeInstanceOf(TimeFormatterRegistry); }); }); diff --git a/superset-frontend/packages/superset-ui-core/test/translation/Translator.test.ts b/superset-frontend/packages/superset-ui-core/test/translation/Translator.test.ts index 703d9a5d30..9466294aca 100644 --- a/superset-frontend/packages/superset-ui-core/test/translation/Translator.test.ts +++ b/superset-frontend/packages/superset-ui-core/test/translation/Translator.test.ts @@ -156,7 +156,7 @@ describe('Translator', () => { it('throw warning on duplicates', () => { expect(() => { addTranslations({ - haha: ['this is duplciate'], + haha: ['this is duplicate'], }); }).toThrow('Duplicate translation key "haha"'); expect(t('haha')).toEqual('Hahaha'); diff --git a/superset-frontend/plugins/plugin-chart-echarts/test/BoxPlot/transformProps.test.ts b/superset-frontend/plugins/plugin-chart-echarts/test/BoxPlot/transformProps.test.ts index 08234fb61e..29b912b48c 100644 --- a/superset-frontend/plugins/plugin-chart-echarts/test/BoxPlot/transformProps.test.ts +++ b/superset-frontend/plugins/plugin-chart-echarts/test/BoxPlot/transformProps.test.ts @@ -20,7 +20,7 @@ import { ChartProps, SqlaFormData } from '@superset-ui/core'; import { EchartsBoxPlotChartProps } from '../../src/BoxPlot/types'; import transformProps from '../../src/BoxPlot/transformProps'; -describe('BoxPlot tranformProps', () => { +describe('BoxPlot transformProps', () => { const formData: SqlaFormData = { datasource: '5__table', granularity_sqla: 'ds', @@ -68,7 +68,7 @@ describe('BoxPlot tranformProps', () => { ], }); - it('should tranform chart props for viz', () => { + it('should transform chart props for viz', () => { expect(transformProps(chartProps as EchartsBoxPlotChartProps)).toEqual( expect.objectContaining({ width: 800, diff --git a/superset-frontend/plugins/plugin-chart-echarts/test/Funnel/transformProps.test.ts b/superset-frontend/plugins/plugin-chart-echarts/test/Funnel/transformProps.test.ts index 87f377f731..74b0510bc6 100644 --- a/superset-frontend/plugins/plugin-chart-echarts/test/Funnel/transformProps.test.ts +++ b/superset-frontend/plugins/plugin-chart-echarts/test/Funnel/transformProps.test.ts @@ -25,7 +25,7 @@ import { EchartsFunnelLabelTypeType, } from '../../src/Funnel/types'; -describe('Funnel tranformProps', () => { +describe('Funnel transformProps', () => { const formData = { colorScheme: 'bnbColors', datasource: '3__table', @@ -47,7 +47,7 @@ describe('Funnel tranformProps', () => { ], }); - it('should tranform chart props for viz', () => { + it('should transform chart props for viz', () => { expect(transformProps(chartProps as EchartsFunnelChartProps)).toEqual( expect.objectContaining({ width: 800, diff --git a/superset-frontend/plugins/plugin-chart-echarts/test/Graph/transformProps.test.ts b/superset-frontend/plugins/plugin-chart-echarts/test/Graph/transformProps.test.ts index bb73d77f11..831a9bf901 100644 --- a/superset-frontend/plugins/plugin-chart-echarts/test/Graph/transformProps.test.ts +++ b/superset-frontend/plugins/plugin-chart-echarts/test/Graph/transformProps.test.ts @@ -20,8 +20,8 @@ import { ChartProps } from '@superset-ui/core'; import transformProps from '../../src/Graph/transformProps'; import { DEFAULT_GRAPH_SERIES_OPTION } from '../../src/Graph/constants'; -describe('EchartsGraph tranformProps', () => { - it('should tranform chart props for viz without category', () => { +describe('EchartsGraph transformProps', () => { + it('should transform chart props for viz without category', () => { const formData = { colorScheme: 'bnbColors', datasource: '3__table', @@ -149,7 +149,7 @@ describe('EchartsGraph tranformProps', () => { ); }); - it('should tranform chart props for viz with category and falsey normalization', () => { + it('should transform chart props for viz with category and falsey normalization', () => { const formData = { colorScheme: 'bnbColors', datasource: '3__table', diff --git a/superset-frontend/plugins/plugin-chart-echarts/test/Pie/transformProps.test.ts b/superset-frontend/plugins/plugin-chart-echarts/test/Pie/transformProps.test.ts index 41280c7667..81f40aa9ad 100644 --- a/superset-frontend/plugins/plugin-chart-echarts/test/Pie/transformProps.test.ts +++ b/superset-frontend/plugins/plugin-chart-echarts/test/Pie/transformProps.test.ts @@ -24,7 +24,7 @@ import { import transformProps, { formatPieLabel } from '../../src/Pie/transformProps'; import { EchartsPieChartProps, EchartsPieLabelType } from '../../src/Pie/types'; -describe('Pie tranformProps', () => { +describe('Pie transformProps', () => { const formData: SqlaFormData = { colorScheme: 'bnbColors', datasource: '3__table', @@ -47,7 +47,7 @@ describe('Pie tranformProps', () => { ], }); - it('should tranform chart props for viz', () => { + it('should transform chart props for viz', () => { expect(transformProps(chartProps as EchartsPieChartProps)).toEqual( expect.objectContaining({ width: 800, diff --git a/superset-frontend/plugins/plugin-chart-echarts/test/Timeseries/transformProps.test.ts b/superset-frontend/plugins/plugin-chart-echarts/test/Timeseries/transformProps.test.ts index 4b5a21e53f..1a3fd5b313 100644 --- a/superset-frontend/plugins/plugin-chart-echarts/test/Timeseries/transformProps.test.ts +++ b/superset-frontend/plugins/plugin-chart-echarts/test/Timeseries/transformProps.test.ts @@ -54,7 +54,7 @@ describe('EchartsTimeseries transformProps', () => { queriesData, }; - it('should tranform chart props for viz', () => { + it('should transform chart props for viz', () => { const chartProps = new ChartProps(chartPropsConfig); expect(transformProps(chartProps as EchartsTimeseriesChartProps)).toEqual( expect.objectContaining({ diff --git a/superset-frontend/plugins/plugin-chart-echarts/test/Tree/transformProps.test.ts b/superset-frontend/plugins/plugin-chart-echarts/test/Tree/transformProps.test.ts index bcb762b68c..af3de96086 100644 --- a/superset-frontend/plugins/plugin-chart-echarts/test/Tree/transformProps.test.ts +++ b/superset-frontend/plugins/plugin-chart-echarts/test/Tree/transformProps.test.ts @@ -19,7 +19,7 @@ import { ChartProps } from '@superset-ui/core'; import transformProps from '../../src/Tree/transformProps'; -describe('EchartsTree tranformProps', () => { +describe('EchartsTree transformProps', () => { const formData = { colorScheme: 'bnbColors', datasource: '3__table', @@ -35,7 +35,7 @@ describe('EchartsTree tranformProps', () => { width: 800, height: 600, }; - it('should tranform when parent present before child', () => { + it('should transform when parent present before child', () => { const queriesData = [ { colnames: ['id_column', 'relation_column', 'name_column', 'count'], @@ -102,7 +102,7 @@ describe('EchartsTree tranformProps', () => { }), ); }); - it('should tranform when child is present before parent', () => { + it('should transform when child is present before parent', () => { const queriesData = [ { colnames: ['id_column', 'relation_column', 'name_column', 'count'], diff --git a/superset-frontend/plugins/plugin-chart-echarts/test/Treemap/transformProps.test.ts b/superset-frontend/plugins/plugin-chart-echarts/test/Treemap/transformProps.test.ts index 9bb6fa03ba..141cd90cc7 100644 --- a/superset-frontend/plugins/plugin-chart-echarts/test/Treemap/transformProps.test.ts +++ b/superset-frontend/plugins/plugin-chart-echarts/test/Treemap/transformProps.test.ts @@ -20,7 +20,7 @@ import { ChartProps } from '@superset-ui/core'; import { EchartsTreemapChartProps } from '../../src/Treemap/types'; import transformProps from '../../src/Treemap/transformProps'; -describe('Treemap tranformProps', () => { +describe('Treemap transformProps', () => { const formData = { colorScheme: 'bnbColors', datasource: '3__table', @@ -42,7 +42,7 @@ describe('Treemap tranformProps', () => { ], }); - it('should tranform chart props for viz', () => { + it('should transform chart props for viz', () => { expect(transformProps(chartProps as EchartsTreemapChartProps)).toEqual( expect.objectContaining({ width: 800, diff --git a/superset-frontend/plugins/plugin-chart-echarts/test/utils/series.test.ts b/superset-frontend/plugins/plugin-chart-echarts/test/utils/series.test.ts index 36cb2047f3..ae3871c821 100644 --- a/superset-frontend/plugins/plugin-chart-echarts/test/utils/series.test.ts +++ b/superset-frontend/plugins/plugin-chart-echarts/test/utils/series.test.ts @@ -230,7 +230,7 @@ describe('formatSeriesName', () => { expect(formatSeriesName(12345678.9, { numberFormatter })).toEqual('12.3M'); }); - it('should use default formatting for for date values without formatter', () => { + it('should use default formatting for date values without formatter', () => { expect(formatSeriesName(new Date('2020-09-11'))).toEqual( '2020-09-11T00:00:00.000Z', ); diff --git a/superset-frontend/plugins/plugin-chart-table/test/TableChart.test.tsx b/superset-frontend/plugins/plugin-chart-table/test/TableChart.test.tsx index bcca124a50..867b6e6799 100644 --- a/superset-frontend/plugins/plugin-chart-table/test/TableChart.test.tsx +++ b/superset-frontend/plugins/plugin-chart-table/test/TableChart.test.tsx @@ -67,7 +67,7 @@ describe('plugin-chart-table', () => { }); describe('TableChart', () => { - let wrap: CommonWrapper; // the ReactDataTable wraper + let wrap: CommonWrapper; // the ReactDataTable wrapper let tree: Cheerio; it('render basic data', () => { @@ -93,7 +93,7 @@ describe('plugin-chart-table', () => { , ); tree = wrap.render(); - // should successfull rerender with new props + // should successful rerender with new props const cells = tree.find('td'); expect(tree.find('th').eq(1).text()).toEqual('Sum of Num'); expect(cells.eq(2).text()).toEqual('12.346%'); diff --git a/superset-frontend/plugins/plugin-chart-word-cloud/test/legacyPlugin/transformProps.test.ts b/superset-frontend/plugins/plugin-chart-word-cloud/test/legacyPlugin/transformProps.test.ts index 53f2dcecb3..8873d35ca5 100644 --- a/superset-frontend/plugins/plugin-chart-word-cloud/test/legacyPlugin/transformProps.test.ts +++ b/superset-frontend/plugins/plugin-chart-word-cloud/test/legacyPlugin/transformProps.test.ts @@ -20,7 +20,7 @@ import { ChartProps } from '@superset-ui/core'; import transformProps from '../../src/legacyPlugin/transformProps'; -describe('WordCloud tranformProps', () => { +describe('WordCloud transformProps', () => { const formData = { colorScheme: 'bnbColors', datasource: '3__table', @@ -42,7 +42,7 @@ describe('WordCloud tranformProps', () => { ], }); - it('should tranform chart props for word cloud viz', () => { + it('should transform chart props for word cloud viz', () => { expect(transformProps(chartProps)).toEqual({ width: 800, height: 600, diff --git a/superset-frontend/src/components/Datasource/DatasourceEditor.test.jsx b/superset-frontend/src/components/Datasource/DatasourceEditor.test.jsx index dec75afdc3..7ad7b9f9a8 100644 --- a/superset-frontend/src/components/Datasource/DatasourceEditor.test.jsx +++ b/superset-frontend/src/components/Datasource/DatasourceEditor.test.jsx @@ -88,7 +88,7 @@ describe('DatasourceEditor', () => { 'Certification details', ); - userEvent.type(await inputLabel, 'test_lable'); + userEvent.type(await inputLabel, 'test_label'); userEvent.type(await inputDescription, 'test'); userEvent.type(await inputDtmFormat, 'test'); userEvent.type(await inputCertifiedBy, 'test'); @@ -157,11 +157,11 @@ describe('DatasourceEditor', () => { const physicalRadioBtn = screen.getByRole('radio', { name: /physical \(table or view\)/i, }); - const vituralRadioBtn = screen.getByRole('radio', { + const virtualRadioBtn = screen.getByRole('radio', { name: /virtual \(sql\)/i, }); expect(physicalRadioBtn).toBeEnabled(); - expect(vituralRadioBtn).toBeEnabled(); + expect(virtualRadioBtn).toBeEnabled(); }); it('Source Tab: readOnly mode', () => { @@ -170,11 +170,11 @@ describe('DatasourceEditor', () => { const physicalRadioBtn = screen.getByRole('radio', { name: /physical \(table or view\)/i, }); - const vituralRadioBtn = screen.getByRole('radio', { + const virtualRadioBtn = screen.getByRole('radio', { name: /virtual \(sql\)/i, }); expect(physicalRadioBtn).toBeDisabled(); - expect(vituralRadioBtn).toBeDisabled(); + expect(virtualRadioBtn).toBeDisabled(); }); }); diff --git a/superset-frontend/src/components/Form/LabeledErrorBoundInput.test.jsx b/superset-frontend/src/components/Form/LabeledErrorBoundInput.test.jsx index ebfd2b30e4..a49b2713d0 100644 --- a/superset-frontend/src/components/Form/LabeledErrorBoundInput.test.jsx +++ b/superset-frontend/src/components/Form/LabeledErrorBoundInput.test.jsx @@ -60,7 +60,7 @@ describe('LabeledErrorBoundInput', () => { expect(textboxInput).toBeVisible(); expect(errorText).toBeVisible(); }); - it('renders a LabledErrorBoundInput with a InfoTooltip', async () => { + it('renders a LabeledErrorBoundInput with a InfoTooltip', async () => { defaultProps.hasTooltip = true; render(); diff --git a/superset-frontend/src/dashboard/components/CrossFilterScopingModal/CrossFilterScopingForm/CrossFilterScopingForm.test.tsx b/superset-frontend/src/dashboard/components/CrossFilterScopingModal/CrossFilterScopingForm/CrossFilterScopingForm.test.tsx index 39cb6f92e1..f1093e2a9e 100644 --- a/superset-frontend/src/dashboard/components/CrossFilterScopingModal/CrossFilterScopingForm/CrossFilterScopingForm.test.tsx +++ b/superset-frontend/src/dashboard/components/CrossFilterScopingModal/CrossFilterScopingForm/CrossFilterScopingForm.test.tsx @@ -51,7 +51,7 @@ test('Should send correct props', () => { ); }); -test('Should get correct filds', () => { +test('Should get correct fields', () => { const props = createProps(); render(); expect(props.form.getFieldValue).toBeCalledTimes(2); diff --git a/superset-frontend/src/dashboard/components/filterscope/FilterScope.test.tsx b/superset-frontend/src/dashboard/components/filterscope/FilterScope.test.tsx index 920b5200b0..186079d8f2 100644 --- a/superset-frontend/src/dashboard/components/filterscope/FilterScope.test.tsx +++ b/superset-frontend/src/dashboard/components/filterscope/FilterScope.test.tsx @@ -139,7 +139,7 @@ const createProps = () => ({ type CheckboxState = 'checked' | 'unchecked' | 'indeterminate'; /** - * Unfortunatelly react-checkbox-tree doesn't provide an easy way to + * Unfortunately react-checkbox-tree doesn't provide an easy way to * access the checkbox icon. We need this function to find the element. */ function getCheckboxIcon(element: HTMLElement): Element { @@ -151,7 +151,7 @@ function getCheckboxIcon(element: HTMLElement): Element { } /** - * Unfortunatelly when using react-checkbox-tree, the only perceived change of a + * Unfortunately when using react-checkbox-tree, the only perceived change of a * checkbox state change is the fill color of the SVG icon. */ function getCheckboxState(name: string): CheckboxState { diff --git a/superset-frontend/src/dashboard/components/nativeFilters/FiltersConfigModal/FiltersConfigForm/getControlItemsMap.test.tsx b/superset-frontend/src/dashboard/components/nativeFilters/FiltersConfigModal/FiltersConfigForm/getControlItemsMap.test.tsx index fca1fb372e..7a3af4fe26 100644 --- a/superset-frontend/src/dashboard/components/nativeFilters/FiltersConfigModal/FiltersConfigForm/getControlItemsMap.test.tsx +++ b/superset-frontend/src/dashboard/components/nativeFilters/FiltersConfigModal/FiltersConfigForm/getControlItemsMap.test.tsx @@ -141,7 +141,7 @@ test('Should render null empty when "controlItems" are falsy', () => { expect(container.children).toHaveLength(0); }); -test('Should render render ControlItems', () => { +test('Should render ControlItems', () => { const props = createProps(); const controlItems = [ @@ -154,7 +154,7 @@ test('Should render render ControlItems', () => { expect(screen.getAllByRole('checkbox')).toHaveLength(2); }); -test('Clickin on checkbox', () => { +test('Clicking on checkbox', () => { const props = createProps(); (getControlItems as jest.Mock).mockReturnValue(createControlItems()); const controlItemsMap = getControlItemsMap(props); @@ -166,7 +166,7 @@ test('Clickin on checkbox', () => { expect(props.forceUpdate).toBeCalled(); }); -test('Clickin on checkbox when resetConfig:flase', () => { +test('Clicking on checkbox when resetConfig:flase', () => { const props = createProps(); (getControlItems as jest.Mock).mockReturnValue([ { name: 'name_1', config: { renderTrigger: true, resetConfig: false } }, diff --git a/superset-frontend/src/dashboard/util/getDetailedComponentWidth.test.js b/superset-frontend/src/dashboard/util/getDetailedComponentWidth.test.js index a99f69fcad..c19306141b 100644 --- a/superset-frontend/src/dashboard/util/getDetailedComponentWidth.test.js +++ b/superset-frontend/src/dashboard/util/getDetailedComponentWidth.test.js @@ -59,7 +59,7 @@ describe('getDetailedComponentWidth', () => { ).toEqual(empty); }); - it('should match component meta width for resizeable components', () => { + it('should match component meta width for resizable components', () => { expect( getDetailedComponentWidth({ component: { id: '', type: types.CHART_TYPE, meta: { width: 1 } }, @@ -76,7 +76,7 @@ describe('getDetailedComponentWidth', () => { getDetailedComponentWidth({ component: { id: '', type: types.COLUMN_TYPE, meta: { width: 3 } }, }), - // note: occupiedWidth is zero for colunns/see test below + // note: occupiedWidth is zero for columns/see test below ).toEqual({ width: 3, occupiedWidth: 0, minimumWidth: 1 }); }); @@ -217,7 +217,7 @@ describe('getDetailedComponentWidth', () => { }, }, }), - // occupiedWidth is zero for colunns/see test below + // occupiedWidth is zero for columns/see test below ).toEqual({ width: 12, occupiedWidth: 0, minimumWidth: 7 }); }); diff --git a/superset-frontend/src/explore/components/controls/withAsyncVerification.test.tsx b/superset-frontend/src/explore/components/controls/withAsyncVerification.test.tsx index 77a8af9374..6be029d7a5 100644 --- a/superset-frontend/src/explore/components/controls/withAsyncVerification.test.tsx +++ b/superset-frontend/src/explore/components/controls/withAsyncVerification.test.tsx @@ -133,7 +133,7 @@ describe('VerifiedMetricsControl', () => { multi: defaultProps.multi, name: defaultProps.name, // in real life, `onChange` should have been called with the updated - // props (both savedMetrics and value should have beend updated), but + // props (both savedMetrics and value should have been updated), but // because of the limitation of enzyme (it cannot get props updated from // useEffect hooks), we are not able to check that here. savedMetrics: defaultProps.savedMetrics, diff --git a/superset-frontend/src/views/CRUD/chart/ChartList.test.jsx b/superset-frontend/src/views/CRUD/chart/ChartList.test.jsx index 394c8b6028..fa5d363c51 100644 --- a/superset-frontend/src/views/CRUD/chart/ChartList.test.jsx +++ b/superset-frontend/src/views/CRUD/chart/ChartList.test.jsx @@ -39,7 +39,7 @@ const mockStore = configureStore([thunk]); const store = mockStore({}); const chartsInfoEndpoint = 'glob:*/api/v1/chart/_info*'; -const chartssOwnersEndpoint = 'glob:*/api/v1/chart/related/owners*'; +const chartsOwnersEndpoint = 'glob:*/api/v1/chart/related/owners*'; const chartsCreatedByEndpoint = 'glob:*/api/v1/chart/related/created_by*'; const chartsEndpoint = 'glob:*/api/v1/chart/*'; const chartsVizTypesEndpoint = 'glob:*/api/v1/chart/viz_types'; @@ -66,7 +66,7 @@ fetchMock.get(chartsInfoEndpoint, { permissions: ['can_read', 'can_write'], }); -fetchMock.get(chartssOwnersEndpoint, { +fetchMock.get(chartsOwnersEndpoint, { result: [], }); fetchMock.get(chartsCreatedByEndpoint, { diff --git a/superset-frontend/src/views/CRUD/data/dataset/DatasetList.test.jsx b/superset-frontend/src/views/CRUD/data/dataset/DatasetList.test.jsx index 0854f006de..ece974e41e 100644 --- a/superset-frontend/src/views/CRUD/data/dataset/DatasetList.test.jsx +++ b/superset-frontend/src/views/CRUD/data/dataset/DatasetList.test.jsx @@ -148,7 +148,7 @@ describe('DatasetList', () => { wrapper.find('[data-test="bulk-select-copy"]').text(), ).toMatchInlineSnapshot(`"0 Selected"`); - // Vitual Selected + // Virtual Selected act(() => { wrapper.find(IndeterminateCheckbox).at(1).props().onChange(checkedEvent); }); diff --git a/superset-frontend/src/views/CRUD/utils.test.tsx b/superset-frontend/src/views/CRUD/utils.test.tsx index dcce0b8369..e727a0c896 100644 --- a/superset-frontend/src/views/CRUD/utils.test.tsx +++ b/superset-frontend/src/views/CRUD/utils.test.tsx @@ -213,7 +213,7 @@ test('successfully modified rison to encode correctly', () => { }); }); -test('checkUploadExtenssions should return valid upload extensions', () => { +test('checkUploadExtensions should return valid upload extensions', () => { const uploadExtensionTest = ['a', 'b', 'c']; const randomExtension = ['a', 'c']; const randomExtensionTwo = ['c'];