chore(frontend-tests): Spelling (#19853)

* spelling: against

Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>

* spelling: been

Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>

* spelling: charts

Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>

* spelling: clicking

Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>

* spelling: columns

Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>

* spelling: duplicate

Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>

* spelling: especially

Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>

* spelling: extensions

Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>

* spelling: fields

Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>

* spelling: filter

Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>

* spelling: for

Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>

* spelling: label

Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>

* spelling: labeled

Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>

* spelling: nativefilter

Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>

* spelling: registry

Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>

* spelling: render

Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>

* spelling: resizable

Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>

* spelling: response

Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>

* spelling: successful

Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>

* spelling: transform

Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>

* spelling: unfortunately

Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>

* spelling: until

Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>

* spelling: virtual

Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>

* spelling: wrapper

Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>

Co-authored-by: Josh Soref <jsoref@users.noreply.github.com>

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)

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 => {

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];
     });

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,

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,
       );

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<JsonValue, number, 'raw'>({
       method: 'DELETE',

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);
     });
   });

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');

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,

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,

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',

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,

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({

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'],

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,

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',
     );

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', () => {
         <TableChart {...transformProps(testData.advanced)} sticky={false} />,
       );
       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%');

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,

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();
     });
   });
 

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(<LabeledErrorBoundInput {...defaultProps} />);
 

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(<CrossFilterScopingForm {...(props as any)} />);
   expect(props.form.getFieldValue).toBeCalledTimes(2);

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 {

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 } },

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 });
     });