From d329eff0da34b24eaf8b68ba9bf29f049eee8b6a Mon Sep 17 00:00:00 2001 From: Eric Wei Date: Wed, 5 Jul 2023 12:50:45 -0700 Subject: [PATCH] [Backport 2.x] Backport/backport 588 to 2.x (#600) * Merge pull request #588 from mengweieric/feature/prerelease-code-cleanup (cherry picked from commit a85b716f4069e70cd2736e0fab7ec32a89907f8d) * update snapshots Signed-off-by: Eric Wei --------- Signed-off-by: Eric Wei Co-authored-by: Anirudha (Ani) Jadhav --- .../__snapshots__/utils.test.tsx.snap | 9 -- .../__snapshots__/config_panel.test.tsx.snap | 20 ----- .../shared_components.test.tsx.snap | 90 ------------------- .../__tests__/shared_components.test.tsx | 12 --- .../plotly_viz_editor/index.tsx | 1 + .../visualization_specs/default_spec.ts | 2 - .../__snapshots__/assets.test.tsx.snap | 64 ------------- .../assets/__tests__/assets.test.tsx | 56 ------------ .../visualizations/assets/chart_bar.tsx | 29 ------ .../assets/chart_bar_horizontal.tsx | 33 ------- .../visualizations/assets/chart_datatable.tsx | 35 -------- .../visualizations/assets/chart_line.tsx | 29 ------ .../visualizations/assets/chart_pie.tsx | 31 ------- .../__tests__/__snapshots__/bar.test.tsx.snap | 1 - .../__snapshots__/gauge.test.tsx.snap | 1 - .../__snapshots__/heatmap.test.tsx.snap | 1 - .../__snapshots__/histogram.test.tsx.snap | 1 - .../horizontal_bar.test.tsx.snap | 1 - .../__snapshots__/line.test.tsx.snap | 1 - .../__snapshots__/metrics.test.tsx.snap | 1 - .../__tests__/__snapshots__/pie.test.tsx.snap | 1 - .../__snapshots__/text.test.tsx.snap | 1 - .../__snapshots__/treemap.test.tsx.snap | 1 - .../visualizations/charts/bar/bar_type.ts | 2 - .../charts/bubble/bubble_type.ts | 2 - .../charts/data_table/data_table_type.ts | 4 - .../charts/financial/gauge/gauge_type.ts | 2 - .../visualizations/charts/lines/line_type.ts | 4 - .../visualizations/charts/maps/heatmap.tsx | 3 +- .../charts/maps/heatmap_type.ts | 4 - .../charts/maps/treemap_type.ts | 6 +- .../charts/metrics/metrics_type.ts | 3 - .../visualizations/charts/pie/pie_type.ts | 5 -- .../visualizations/charts/text/text_type.ts | 2 - 34 files changed, 3 insertions(+), 455 deletions(-) delete mode 100644 public/components/visualizations/assets/__tests__/__snapshots__/assets.test.tsx.snap delete mode 100644 public/components/visualizations/assets/__tests__/assets.test.tsx delete mode 100644 public/components/visualizations/assets/chart_bar.tsx delete mode 100644 public/components/visualizations/assets/chart_bar_horizontal.tsx delete mode 100644 public/components/visualizations/assets/chart_datatable.tsx delete mode 100644 public/components/visualizations/assets/chart_line.tsx delete mode 100644 public/components/visualizations/assets/chart_pie.tsx diff --git a/public/components/custom_panels/helpers/__tests__/__snapshots__/utils.test.tsx.snap b/public/components/custom_panels/helpers/__tests__/__snapshots__/utils.test.tsx.snap index 2201cdec0..c6f88b0ad 100644 --- a/public/components/custom_panels/helpers/__tests__/__snapshots__/utils.test.tsx.snap +++ b/public/components/custom_panels/helpers/__tests__/__snapshots__/utils.test.tsx.snap @@ -409,7 +409,6 @@ exports[`Utils helper functions renders displayVisualization function 1`] = ` "fillopacity": 100, "fulllabel": "Vertical bar", "groupwidth": 0.7, - "icon": [Function], "icontype": "visBarVerticalStacked", "id": "bar", "label": "Vertical bar", @@ -858,7 +857,6 @@ exports[`Utils helper functions renders displayVisualization function 1`] = ` "fillopacity": 100, "fulllabel": "Vertical bar", "groupwidth": 0.7, - "icon": [Function], "icontype": "visBarVerticalStacked", "id": "bar", "label": "Vertical bar", @@ -1314,7 +1312,6 @@ exports[`Utils helper functions renders displayVisualization function 1`] = ` "fillopacity": 100, "fulllabel": "Vertical bar", "groupwidth": 0.7, - "icon": [Function], "icontype": "visBarVerticalStacked", "id": "bar", "label": "Vertical bar", @@ -2015,7 +2012,6 @@ exports[`Utils helper functions renders displayVisualization function 2`] = ` ], }, "fulllabel": "Time series", - "icon": [Function], "icontype": "visLine", "id": "line", "label": "Time series", @@ -2532,7 +2528,6 @@ exports[`Utils helper functions renders displayVisualization function 2`] = ` ], }, "fulllabel": "Time series", - "icon": [Function], "icontype": "visLine", "id": "line", "label": "Time series", @@ -3063,7 +3058,6 @@ exports[`Utils helper functions renders displayVisualization function 2`] = ` ], }, "fulllabel": "Time series", - "icon": [Function], "icontype": "visLine", "id": "line", "label": "Time series", @@ -3739,7 +3733,6 @@ exports[`Utils helper functions renders displayVisualization function 3`] = ` "fillopacity": 100, "fulllabel": "Horizontal bar", "groupwidth": 0.7, - "icon": [Function], "icontype": "visBarHorizontalStacked", "id": "horizontal_bar", "label": "Horizontal bar", @@ -4188,7 +4181,6 @@ exports[`Utils helper functions renders displayVisualization function 3`] = ` "fillopacity": 100, "fulllabel": "Horizontal bar", "groupwidth": 0.7, - "icon": [Function], "icontype": "visBarHorizontalStacked", "id": "horizontal_bar", "label": "Horizontal bar", @@ -4644,7 +4636,6 @@ exports[`Utils helper functions renders displayVisualization function 3`] = ` "fillopacity": 100, "fulllabel": "Horizontal bar", "groupwidth": 0.7, - "icon": [Function], "icontype": "visBarHorizontalStacked", "id": "horizontal_bar", "label": "Horizontal bar", diff --git a/public/components/event_analytics/explorer/visualizations/config_panel/__tests__/__snapshots__/config_panel.test.tsx.snap b/public/components/event_analytics/explorer/visualizations/config_panel/__tests__/__snapshots__/config_panel.test.tsx.snap index a712df4bb..4d7f5fb29 100644 --- a/public/components/event_analytics/explorer/visualizations/config_panel/__tests__/__snapshots__/config_panel.test.tsx.snap +++ b/public/components/event_analytics/explorer/visualizations/config_panel/__tests__/__snapshots__/config_panel.test.tsx.snap @@ -617,7 +617,6 @@ exports[`Config panel component Renders config panel with visualization data 1`] "fillopacity": 100, "fulllabel": "Vertical bar", "groupwidth": 0.7, - "icon": [Function], "icontype": "visBarVerticalStacked", "id": "bar", "label": "Vertical bar", @@ -933,7 +932,6 @@ exports[`Config panel component Renders config panel with visualization data 1`] "fillopacity": 100, "fulllabel": "Vertical bar", "groupwidth": 0.7, - "icon": [Function], "iconType": "visBarVerticalStacked", "icontype": "visBarVerticalStacked", "id": "bar", @@ -1232,7 +1230,6 @@ exports[`Config panel component Renders config panel with visualization data 1`] "fillopacity": 100, "fulllabel": "Horizontal bar", "groupwidth": 0.7, - "icon": [Function], "icontype": "visBarHorizontalStacked", "id": "horizontal_bar", "label": "Horizontal bar", @@ -1560,7 +1557,6 @@ exports[`Config panel component Renders config panel with visualization data 1`] ], }, "fulllabel": "Time series", - "icon": [Function], "icontype": "visLine", "id": "line", "label": "Time series", @@ -1801,7 +1797,6 @@ exports[`Config panel component Renders config panel with visualization data 1`] ], }, "fulllabel": "Pie", - "icon": [Function], "icontype": "visPie", "id": "pie", "label": "Pie", @@ -2241,7 +2236,6 @@ exports[`Config panel component Renders config panel with visualization data 1`] ], }, "fulllabel": "Hubble", - "icon": [Function], "icontype": "heatmap", "id": "heatmap", "label": "Heatmap", @@ -2294,7 +2288,6 @@ exports[`Config panel component Renders config panel with visualization data 1`] ], }, "fulllabel": "Markdown", - "icon": [Function], "icontype": "visText", "id": "text", "label": "Markdown", @@ -2622,7 +2615,6 @@ exports[`Config panel component Renders config panel with visualization data 1`] "fillopacity": 100, "fulllabel": "Vertical bar", "groupwidth": 0.7, - "icon": [Function], "iconType": "visBarVerticalStacked", "icontype": "visBarVerticalStacked", "id": "bar", @@ -2953,7 +2945,6 @@ exports[`Config panel component Renders config panel with visualization data 1`] "fillopacity": 100, "fulllabel": "Vertical bar", "groupwidth": 0.7, - "icon": [Function], "iconType": "visBarVerticalStacked", "icontype": "visBarVerticalStacked", "id": "bar", @@ -3290,7 +3281,6 @@ exports[`Config panel component Renders config panel with visualization data 1`] fillopacity={100} fulllabel="Vertical bar" groupwidth={0.7} - icon={[Function]} iconType="visBarVerticalStacked" icontype="visBarVerticalStacked" id="bar" @@ -3560,7 +3550,6 @@ exports[`Config panel component Renders config panel with visualization data 1`] "fillopacity": 100, "fulllabel": "Vertical bar", "groupwidth": 0.7, - "icon": [Function], "iconType": "visBarVerticalStacked", "icontype": "visBarVerticalStacked", "id": "bar", @@ -3898,7 +3887,6 @@ exports[`Config panel component Renders config panel with visualization data 1`] fillopacity={100} fulllabel="Vertical bar" groupwidth={0.7} - icon={[Function]} iconType="visBarVerticalStacked" icontype="visBarVerticalStacked" id="bar" @@ -4205,7 +4193,6 @@ exports[`Config panel component Renders config panel with visualization data 1`] fillopacity={100} fulllabel="Vertical bar" groupwidth={0.7} - icon={[Function]} icontype="visBarVerticalStacked" id="bar" labelangle={0} @@ -5282,7 +5269,6 @@ exports[`Config panel component Renders config panel with visualization data 1`] "fillopacity": 100, "fulllabel": "Vertical bar", "groupwidth": 0.7, - "icon": [Function], "icontype": "visBarVerticalStacked", "id": "bar", "label": "Vertical bar", @@ -6211,7 +6197,6 @@ exports[`Config panel component Renders config panel with visualization data 1`] "fillopacity": 100, "fulllabel": "Vertical bar", "groupwidth": 0.7, - "icon": [Function], "icontype": "visBarVerticalStacked", "id": "bar", "label": "Vertical bar", @@ -7181,7 +7166,6 @@ exports[`Config panel component Renders config panel with visualization data 1`] "fillopacity": 100, "fulllabel": "Vertical bar", "groupwidth": 0.7, - "icon": [Function], "icontype": "visBarVerticalStacked", "id": "bar", "label": "Vertical bar", @@ -7886,7 +7870,6 @@ exports[`Config panel component Renders config panel with visualization data 1`] "fillopacity": 100, "fulllabel": "Vertical bar", "groupwidth": 0.7, - "icon": [Function], "icontype": "visBarVerticalStacked", "id": "bar", "label": "Vertical bar", @@ -8858,7 +8841,6 @@ exports[`Config panel component Renders config panel with visualization data 1`] "fillopacity": 100, "fulllabel": "Vertical bar", "groupwidth": 0.7, - "icon": [Function], "icontype": "visBarVerticalStacked", "id": "bar", "label": "Vertical bar", @@ -10329,7 +10311,6 @@ exports[`Config panel component Renders config panel with visualization data 1`] "fillopacity": 100, "fulllabel": "Vertical bar", "groupwidth": 0.7, - "icon": [Function], "icontype": "visBarVerticalStacked", "id": "bar", "label": "Vertical bar", @@ -11821,7 +11802,6 @@ exports[`Config panel component Renders config panel with visualization data 1`] "fillopacity": 100, "fulllabel": "Vertical bar", "groupwidth": 0.7, - "icon": [Function], "icontype": "visBarVerticalStacked", "id": "bar", "label": "Vertical bar", diff --git a/public/components/event_analytics/explorer/visualizations/shared_components/__tests__/__snapshots__/shared_components.test.tsx.snap b/public/components/event_analytics/explorer/visualizations/shared_components/__tests__/__snapshots__/shared_components.test.tsx.snap index adb37ebb1..29dcef545 100644 --- a/public/components/event_analytics/explorer/visualizations/shared_components/__tests__/__snapshots__/shared_components.test.tsx.snap +++ b/public/components/event_analytics/explorer/visualizations/shared_components/__tests__/__snapshots__/shared_components.test.tsx.snap @@ -1,95 +1,5 @@ // Jest Snapshot v1, https://goo.gl/fbAQLP -exports[`Shared components Renders empty placeholder component 1`] = ` - - -
- -
- -
- - - - - - - - - -
- -

- - - No data found - - -

-
- -
- -
- - -`; - exports[`Shared components Renders tool bar button component 1`] = ` { configure({ adapter: new Adapter() }); - it('Renders empty placeholder component', async () => { - const wrapper = mount(); - - wrapper.update(); - - await waitFor(() => { - expect(wrapper).toMatchSnapshot(); - }); - }); - it('Renders tool bar button component', async () => { const handleClick = jest.fn(); const WrappedComponent = () =>
testing
; diff --git a/public/components/event_analytics/explorer/visualizations/shared_components/plotly_viz_editor/index.tsx b/public/components/event_analytics/explorer/visualizations/shared_components/plotly_viz_editor/index.tsx index 1238af2fe..088751a9f 100644 --- a/public/components/event_analytics/explorer/visualizations/shared_components/plotly_viz_editor/index.tsx +++ b/public/components/event_analytics/explorer/visualizations/shared_components/plotly_viz_editor/index.tsx @@ -1 +1,2 @@ export { PlotlyVizEditor } from './plotly_viz_editor'; +export { PlotlyEditorActionsMenu } from './plotly_actions'; diff --git a/public/components/event_analytics/explorer/visualizations/visualization_specs/default_spec.ts b/public/components/event_analytics/explorer/visualizations/visualization_specs/default_spec.ts index 310f6613f..e71363c94 100644 --- a/public/components/event_analytics/explorer/visualizations/visualization_specs/default_spec.ts +++ b/public/components/event_analytics/explorer/visualizations/visualization_specs/default_spec.ts @@ -1,6 +1,4 @@ // @ts-ignore - -import hjson from 'hjson'; import defaultLayoutSpec from '!!raw-loader!./default.layout.spec.hjson'; import defaultDataSpec from '!!raw-loader!./default.data.spec.hjson'; diff --git a/public/components/visualizations/assets/__tests__/__snapshots__/assets.test.tsx.snap b/public/components/visualizations/assets/__tests__/__snapshots__/assets.test.tsx.snap deleted file mode 100644 index 4e61b4176..000000000 --- a/public/components/visualizations/assets/__tests__/__snapshots__/assets.test.tsx.snap +++ /dev/null @@ -1,64 +0,0 @@ -// Jest Snapshot v1, https://goo.gl/fbAQLP - -exports[`Assets components Renders lens icon of bar component 1`] = ` - - - - - - -`; - -exports[`Assets components Renders lens icon of horizontal bar component 1`] = ` - - - - - - -`; - -exports[`Assets components Renders lens icon of line component 1`] = ` - - - - - - -`; diff --git a/public/components/visualizations/assets/__tests__/assets.test.tsx b/public/components/visualizations/assets/__tests__/assets.test.tsx deleted file mode 100644 index 3b3263533..000000000 --- a/public/components/visualizations/assets/__tests__/assets.test.tsx +++ /dev/null @@ -1,56 +0,0 @@ -/* - * Copyright OpenSearch Contributors - * SPDX-License-Identifier: Apache-2.0 - */ - -import { configure, mount } from 'enzyme'; -import Adapter from 'enzyme-adapter-react-16'; -import React from 'react'; -import { waitFor } from '@testing-library/react'; -import { LensIconChartBar } from '../chart_bar'; -import { LensIconChartBarHorizontal } from '../chart_bar_horizontal'; -import { LensIconChartLine } from '../chart_line'; - - -describe('Assets components', () => { - configure({ adapter: new Adapter() }); - - it('Renders lens icon of bar component', async () => { - - const wrapper = mount( - - ); - - wrapper.update(); - - await waitFor(() => { - expect(wrapper).toMatchSnapshot(); - }); - }); - - it('Renders lens icon of horizontal bar component', async () => { - - const wrapper = mount( - - ); - - wrapper.update(); - - await waitFor(() => { - expect(wrapper).toMatchSnapshot(); - }); - }); - - it('Renders lens icon of line component', async () => { - - const wrapper = mount( - - ); - - wrapper.update(); - - await waitFor(() => { - expect(wrapper).toMatchSnapshot(); - }); - }); -}); \ No newline at end of file diff --git a/public/components/visualizations/assets/chart_bar.tsx b/public/components/visualizations/assets/chart_bar.tsx deleted file mode 100644 index 3ee43ea22..000000000 --- a/public/components/visualizations/assets/chart_bar.tsx +++ /dev/null @@ -1,29 +0,0 @@ -/* - * Copyright OpenSearch Contributors - * SPDX-License-Identifier: Apache-2.0 - */ - -import React from 'react'; -import { EuiIconProps } from '@elastic/eui'; - -export const LensIconChartBar = ({ title, titleId, ...props }: Omit) => ( - - {title ? {title} : null} - - - -); diff --git a/public/components/visualizations/assets/chart_bar_horizontal.tsx b/public/components/visualizations/assets/chart_bar_horizontal.tsx deleted file mode 100644 index 0ccfa6a6c..000000000 --- a/public/components/visualizations/assets/chart_bar_horizontal.tsx +++ /dev/null @@ -1,33 +0,0 @@ -/* - * Copyright OpenSearch Contributors - * SPDX-License-Identifier: Apache-2.0 - */ - -import React from 'react'; -import { EuiIconProps } from '@elastic/eui'; - -export const LensIconChartBarHorizontal = ({ - title, - titleId, - ...props -}: Omit) => ( - - {title ? {title} : null} - - - -); diff --git a/public/components/visualizations/assets/chart_datatable.tsx b/public/components/visualizations/assets/chart_datatable.tsx deleted file mode 100644 index d6a5cb733..000000000 --- a/public/components/visualizations/assets/chart_datatable.tsx +++ /dev/null @@ -1,35 +0,0 @@ -/* - * Copyright Elasticsearch B.V. and/or licensed to Elasticsearch B.V. under one - * or more contributor license agreements. Licensed under the Elastic License - * 2.0; you may not use this file except in compliance with the Elastic License - * 2.0. - */ - -import React from 'react'; -import { EuiIconProps } from '@elastic/eui'; - -export const LensIconChartDatatable = ({ - title, - titleId, - ...props -}: Omit) => ( - - {title ? {title} : null} - - - -); diff --git a/public/components/visualizations/assets/chart_line.tsx b/public/components/visualizations/assets/chart_line.tsx deleted file mode 100644 index af6b1294c..000000000 --- a/public/components/visualizations/assets/chart_line.tsx +++ /dev/null @@ -1,29 +0,0 @@ -/* - * Copyright OpenSearch Contributors - * SPDX-License-Identifier: Apache-2.0 - */ - -import React from 'react'; -import { EuiIconProps } from '@elastic/eui'; - -export const LensIconChartLine = ({ title, titleId, ...props }: Omit) => ( - - {title ? {title} : null} - - - -); diff --git a/public/components/visualizations/assets/chart_pie.tsx b/public/components/visualizations/assets/chart_pie.tsx deleted file mode 100644 index 56a18b9d6..000000000 --- a/public/components/visualizations/assets/chart_pie.tsx +++ /dev/null @@ -1,31 +0,0 @@ -/* - * Copyright Elasticsearch B.V. and/or licensed to Elasticsearch B.V. under one - * or more contributor license agreements. Licensed under the Elastic License - * 2.0; you may not use this file except in compliance with the Elastic License - * 2.0. - */ - -import React from 'react'; -import { EuiIconProps } from '@elastic/eui'; - -export const LensIconChartPie = ({ title, titleId, ...props }: Omit) => ( - - {title ? {title} : null} - - - -); diff --git a/public/components/visualizations/charts/__tests__/__snapshots__/bar.test.tsx.snap b/public/components/visualizations/charts/__tests__/__snapshots__/bar.test.tsx.snap index 4b8f4303a..9c5fa0f37 100644 --- a/public/components/visualizations/charts/__tests__/__snapshots__/bar.test.tsx.snap +++ b/public/components/visualizations/charts/__tests__/__snapshots__/bar.test.tsx.snap @@ -632,7 +632,6 @@ exports[`Veritcal Bar component Renders veritcal bar component 1`] = ` "fillopacity": 100, "fulllabel": "Vertical bar", "groupwidth": 0.7, - "icon": [Function], "icontype": "visBarVerticalStacked", "id": "bar", "label": "Vertical bar", diff --git a/public/components/visualizations/charts/__tests__/__snapshots__/gauge.test.tsx.snap b/public/components/visualizations/charts/__tests__/__snapshots__/gauge.test.tsx.snap index 893fbd762..021c679e5 100644 --- a/public/components/visualizations/charts/__tests__/__snapshots__/gauge.test.tsx.snap +++ b/public/components/visualizations/charts/__tests__/__snapshots__/gauge.test.tsx.snap @@ -501,7 +501,6 @@ exports[`Gauge component Renders gauge component 1`] = ` ], }, "fulllabel": "Gauge", - "icon": [Function], "icontype": "visGauge", "id": "gauge", "label": "Gauge", diff --git a/public/components/visualizations/charts/__tests__/__snapshots__/heatmap.test.tsx.snap b/public/components/visualizations/charts/__tests__/__snapshots__/heatmap.test.tsx.snap index 59917f21b..040fffaf5 100644 --- a/public/components/visualizations/charts/__tests__/__snapshots__/heatmap.test.tsx.snap +++ b/public/components/visualizations/charts/__tests__/__snapshots__/heatmap.test.tsx.snap @@ -636,7 +636,6 @@ exports[`Heatmap component Renders heatmap component 1`] = ` "fillopacity": 100, "fulllabel": "Vertical bar", "groupwidth": 0.7, - "icon": [Function], "icontype": "visBarVerticalStacked", "id": "bar", "label": "Vertical bar", diff --git a/public/components/visualizations/charts/__tests__/__snapshots__/histogram.test.tsx.snap b/public/components/visualizations/charts/__tests__/__snapshots__/histogram.test.tsx.snap index 3b55f2c74..ea0daf6b3 100644 --- a/public/components/visualizations/charts/__tests__/__snapshots__/histogram.test.tsx.snap +++ b/public/components/visualizations/charts/__tests__/__snapshots__/histogram.test.tsx.snap @@ -632,7 +632,6 @@ exports[`Histogram component Renders histogram component 1`] = ` "fillopacity": 100, "fulllabel": "Vertical bar", "groupwidth": 0.7, - "icon": [Function], "icontype": "visBarVerticalStacked", "id": "bar", "label": "Vertical bar", diff --git a/public/components/visualizations/charts/__tests__/__snapshots__/horizontal_bar.test.tsx.snap b/public/components/visualizations/charts/__tests__/__snapshots__/horizontal_bar.test.tsx.snap index 113f9b68b..a9007af59 100644 --- a/public/components/visualizations/charts/__tests__/__snapshots__/horizontal_bar.test.tsx.snap +++ b/public/components/visualizations/charts/__tests__/__snapshots__/horizontal_bar.test.tsx.snap @@ -632,7 +632,6 @@ exports[`Horizontal bar component Renders horizontal bar component 1`] = ` "fillopacity": 100, "fulllabel": "Horizontal bar", "groupwidth": 0.7, - "icon": [Function], "icontype": "visBarHorizontalStacked", "id": "horizontal_bar", "label": "Horizontal bar", diff --git a/public/components/visualizations/charts/__tests__/__snapshots__/line.test.tsx.snap b/public/components/visualizations/charts/__tests__/__snapshots__/line.test.tsx.snap index f5cd8f9cb..fce2a2ea0 100644 --- a/public/components/visualizations/charts/__tests__/__snapshots__/line.test.tsx.snap +++ b/public/components/visualizations/charts/__tests__/__snapshots__/line.test.tsx.snap @@ -632,7 +632,6 @@ exports[`Line component Renders line component 1`] = ` "fillopacity": 100, "fulllabel": "Vertical bar", "groupwidth": 0.7, - "icon": [Function], "icontype": "visBarVerticalStacked", "id": "bar", "label": "Vertical bar", diff --git a/public/components/visualizations/charts/__tests__/__snapshots__/metrics.test.tsx.snap b/public/components/visualizations/charts/__tests__/__snapshots__/metrics.test.tsx.snap index 87fa48258..ff5c84b86 100644 --- a/public/components/visualizations/charts/__tests__/__snapshots__/metrics.test.tsx.snap +++ b/public/components/visualizations/charts/__tests__/__snapshots__/metrics.test.tsx.snap @@ -599,7 +599,6 @@ exports[`Metrics component Renders Metrics component 1`] = ` ], }, "fulllabel": "Metrics", - "icon": [Function], "icontype": "stats", "id": "metrics", "label": "Metrics", diff --git a/public/components/visualizations/charts/__tests__/__snapshots__/pie.test.tsx.snap b/public/components/visualizations/charts/__tests__/__snapshots__/pie.test.tsx.snap index 4465f321e..5545e2bab 100644 --- a/public/components/visualizations/charts/__tests__/__snapshots__/pie.test.tsx.snap +++ b/public/components/visualizations/charts/__tests__/__snapshots__/pie.test.tsx.snap @@ -632,7 +632,6 @@ exports[`Pie component Renders pie component 1`] = ` "fillopacity": 100, "fulllabel": "Vertical bar", "groupwidth": 0.7, - "icon": [Function], "icontype": "visBarVerticalStacked", "id": "bar", "label": "Vertical bar", diff --git a/public/components/visualizations/charts/__tests__/__snapshots__/text.test.tsx.snap b/public/components/visualizations/charts/__tests__/__snapshots__/text.test.tsx.snap index 6c40f4dfe..8c7f67a8d 100644 --- a/public/components/visualizations/charts/__tests__/__snapshots__/text.test.tsx.snap +++ b/public/components/visualizations/charts/__tests__/__snapshots__/text.test.tsx.snap @@ -615,7 +615,6 @@ exports[`Text component Renders text component 1`] = ` "fillopacity": 100, "fulllabel": "Vertical bar", "groupwidth": 0.7, - "icon": [Function], "icontype": "visBarVerticalStacked", "id": "bar", "label": "Vertical bar", diff --git a/public/components/visualizations/charts/__tests__/__snapshots__/treemap.test.tsx.snap b/public/components/visualizations/charts/__tests__/__snapshots__/treemap.test.tsx.snap index 4081a0419..bf22d5b9e 100644 --- a/public/components/visualizations/charts/__tests__/__snapshots__/treemap.test.tsx.snap +++ b/public/components/visualizations/charts/__tests__/__snapshots__/treemap.test.tsx.snap @@ -632,7 +632,6 @@ exports[`Treemap component Renders treemap component 1`] = ` "fillopacity": 100, "fulllabel": "Vertical bar", "groupwidth": 0.7, - "icon": [Function], "icontype": "visBarVerticalStacked", "id": "bar", "label": "Vertical bar", diff --git a/public/components/visualizations/charts/bar/bar_type.ts b/public/components/visualizations/charts/bar/bar_type.ts index 8fc51ce22..1d51fc0dc 100644 --- a/public/components/visualizations/charts/bar/bar_type.ts +++ b/public/components/visualizations/charts/bar/bar_type.ts @@ -5,7 +5,6 @@ import { Bar } from './bar'; import { getPlotlySharedConfigs, getPlotlyCategory } from '../shared/shared_configs'; -import { LensIconChartBar } from '../../assets/chart_bar'; import { VizDataPanel } from '../../../event_analytics/explorer/visualizations/config_panel/config_panes/default_vis_editor'; import { ConfigEditor } from '../../../event_analytics/explorer/visualizations/config_panel/config_panes/json_editor'; import { @@ -41,7 +40,6 @@ export const createBarTypeDefinition = (params: any) => ({ dataLoss: 'nothing', }, category: VIS_CATEGORY.BASICS, - icon: LensIconChartBar, categoryaxis: 'xaxis', seriesaxis: 'yaxis', orientation: isHorizontalBar(params.type) ? BarOrientation.horizontal : BarOrientation.vertical, diff --git a/public/components/visualizations/charts/bubble/bubble_type.ts b/public/components/visualizations/charts/bubble/bubble_type.ts index b4cac8ded..a5ed32f2e 100644 --- a/public/components/visualizations/charts/bubble/bubble_type.ts +++ b/public/components/visualizations/charts/bubble/bubble_type.ts @@ -5,7 +5,6 @@ import { Bubble } from './bubble'; import { getPlotlySharedConfigs, getPlotlyCategory } from '../shared/shared_configs'; -import { LensIconChartPie } from '../../assets/chart_pie'; import { VizDataPanel } from '../../../event_analytics/explorer/visualizations/config_panel/config_panes/default_vis_editor'; import { ConfigEditor } from '../../../event_analytics/explorer/visualizations/config_panel/config_panes/json_editor'; import { ConfigValueOptions } from '../../../event_analytics/explorer/visualizations/config_panel/config_panes/config_controls'; @@ -62,7 +61,6 @@ export const createBubbleVisDefinition = () => ({ }, ], }, - icon: LensIconChartPie, visconfig: { layout: { ...sharedConfigs.layout, diff --git a/public/components/visualizations/charts/data_table/data_table_type.ts b/public/components/visualizations/charts/data_table/data_table_type.ts index 7ad582ccd..eacb37be3 100644 --- a/public/components/visualizations/charts/data_table/data_table_type.ts +++ b/public/components/visualizations/charts/data_table/data_table_type.ts @@ -5,9 +5,6 @@ import { DataTable } from './data_table'; import { getPlotlySharedConfigs, getPlotlyCategory } from '../shared/shared_configs'; -import { LensIconChartDatatable } from '../../assets/chart_datatable'; -import { ConfigEditor } from '../../../event_analytics/explorer/visualizations/config_panel/config_panes/json_editor'; -import { ConfigAvailability } from '../../../event_analytics/explorer/visualizations/config_panel/config_panes/config_controls/config_availability'; import { VizDataPanel } from '../../../event_analytics/explorer/visualizations/config_panel/config_panes/default_vis_editor'; import { InputFieldItem, @@ -28,7 +25,6 @@ export const createDatatableTypeDefinition = (params: any = {}) => ({ fulllabel: 'Table View', icontype: 'visTable', category: VIS_CATEGORY.BASICS, - icon: LensIconChartDatatable, showtableheader: true, enablepagination: true, colunmfilter: false, diff --git a/public/components/visualizations/charts/financial/gauge/gauge_type.ts b/public/components/visualizations/charts/financial/gauge/gauge_type.ts index 2da38fefd..d273b19c5 100644 --- a/public/components/visualizations/charts/financial/gauge/gauge_type.ts +++ b/public/components/visualizations/charts/financial/gauge/gauge_type.ts @@ -5,7 +5,6 @@ import { Gauge } from './gauge'; import { getPlotlySharedConfigs, getPlotlyCategory } from '../../shared/shared_configs'; -import { LensIconChartLine } from '../../../assets/chart_line'; import { VizDataPanel } from '../../../../event_analytics/explorer/visualizations/config_panel/config_panes/default_vis_editor'; import { ConfigEditor } from '../../../../event_analytics/explorer/visualizations/config_panel/config_panes/json_editor'; import { @@ -32,7 +31,6 @@ export const createGaugeTypeDefinition = (params: any = {}) => ({ selection: { dataLoss: 'nothing', }, - icon: LensIconChartLine, valueseries: 'yaxis', editorconfig: { panelTabs: [ diff --git a/public/components/visualizations/charts/lines/line_type.ts b/public/components/visualizations/charts/lines/line_type.ts index b82e44eb0..aa145ea86 100644 --- a/public/components/visualizations/charts/lines/line_type.ts +++ b/public/components/visualizations/charts/lines/line_type.ts @@ -5,15 +5,12 @@ import { Line } from './line'; import { getPlotlySharedConfigs, getPlotlyCategory } from '../shared/shared_configs'; -import { LensIconChartLine } from '../../assets/chart_line'; import { VizDataPanel } from '../../../event_analytics/explorer/visualizations/config_panel/config_panes/default_vis_editor'; -import { ConfigEditor } from '../../../event_analytics/explorer/visualizations/config_panel/config_panes/json_editor'; import { ConfigThresholds, ConfigLineChartStyles, ConfigLegend, InputFieldItem, - ConfigColorTheme, } from '../../../event_analytics/explorer/visualizations/config_panel/config_panes/config_controls'; import { ConfigAvailability } from '../../../event_analytics/explorer/visualizations/config_panel/config_panes/config_controls/config_availability'; import { @@ -49,7 +46,6 @@ export const createLineTypeDefinition = (params: any = {}) => ({ selection: { dataLoss: 'nothing', }, - icon: LensIconChartLine, categoryaxis: 'xaxis', seriesaxis: 'yaxis', editorconfig: { diff --git a/public/components/visualizations/charts/maps/heatmap.tsx b/public/components/visualizations/charts/maps/heatmap.tsx index 9bca96d8f..42e2e8555 100644 --- a/public/components/visualizations/charts/maps/heatmap.tsx +++ b/public/components/visualizations/charts/maps/heatmap.tsx @@ -4,7 +4,7 @@ */ import React, { useMemo } from 'react'; import { colorPalette } from '@elastic/eui'; -import { forEach, isEmpty, isEqual, map } from 'lodash'; +import { forEach, isEmpty, map } from 'lodash'; import Plotly from 'plotly.js-dist'; import { HEATMAP_PALETTE_COLOR, @@ -18,7 +18,6 @@ import { getPropName, } from '../../../../components/event_analytics/utils/utils'; import { IVisualizationContainerProps } from '../../../../../common/types/explorer'; -import { VisCanvassPlaceholder } from '../../../event_analytics/explorer/visualizations/shared_components/vis_canvass_placeholder'; import { Plt } from '../../plotly/plot'; import { AGGREGATIONS, GROUPBY } from '../../../../../common/constants/explorer'; import { PLOT_MARGIN } from '../../../../../common/constants/shared'; diff --git a/public/components/visualizations/charts/maps/heatmap_type.ts b/public/components/visualizations/charts/maps/heatmap_type.ts index 6501aa04a..ff9e7abd7 100644 --- a/public/components/visualizations/charts/maps/heatmap_type.ts +++ b/public/components/visualizations/charts/maps/heatmap_type.ts @@ -5,11 +5,8 @@ import { HeatMap } from './heatmap'; import { getPlotlySharedConfigs, getPlotlyCategory } from '../shared/shared_configs'; -import { LensIconChartPie } from '../../assets/chart_pie'; import { VizDataPanel } from '../../../event_analytics/explorer/visualizations/config_panel/config_panes/default_vis_editor'; -import { ConfigEditor } from '../../../event_analytics/explorer/visualizations/config_panel/config_panes/json_editor'; import { - ConfigValueOptions, HeatmapColorPalettePicker, ConfigChartOptions, PanelItem, @@ -37,7 +34,6 @@ export const createMapsVisDefinition = () => ({ selection: { dataLoss: 'nothing', }, - icon: LensIconChartPie, editorconfig: { panelTabs: [ { diff --git a/public/components/visualizations/charts/maps/treemap_type.ts b/public/components/visualizations/charts/maps/treemap_type.ts index 1a87a428b..788a75e4b 100644 --- a/public/components/visualizations/charts/maps/treemap_type.ts +++ b/public/components/visualizations/charts/maps/treemap_type.ts @@ -5,7 +5,6 @@ import { TreeMap } from './treemaps'; import { getPlotlySharedConfigs, getPlotlyCategory } from '../shared/shared_configs'; -import { LensIconChartBar } from '../../assets/chart_bar'; import { VizDataPanel } from '../../../event_analytics/explorer/visualizations/config_panel/config_panes/default_vis_editor'; import { ConfigEditor } from '../../../event_analytics/explorer/visualizations/config_panel/config_panes/json_editor'; import { @@ -24,9 +23,7 @@ const VIS_CATEGORY = getPlotlyCategory(); const { SortSectors } = DEFAULT_CHART_STYLES; -export interface BarTypeParams {} - -export const createTreeMapDefinition = (params: BarTypeParams = {}) => ({ +export const createTreeMapDefinition = (params: any) => ({ name: 'tree_map', type: 'tree_map', id: 'tree_map', @@ -37,7 +34,6 @@ export const createTreeMapDefinition = (params: BarTypeParams = {}) => ({ }, category: VIS_CATEGORY.BASICS, icontype: 'heatmap', - icon: LensIconChartBar, categoryaxis: 'xaxis', seriesaxis: 'yaxis', orientation: 'v', diff --git a/public/components/visualizations/charts/metrics/metrics_type.ts b/public/components/visualizations/charts/metrics/metrics_type.ts index bc8f6b546..e13df7744 100644 --- a/public/components/visualizations/charts/metrics/metrics_type.ts +++ b/public/components/visualizations/charts/metrics/metrics_type.ts @@ -5,7 +5,6 @@ import { Metrics } from './metrics'; import { getPlotlySharedConfigs, getPlotlyCategory } from '../shared/shared_configs'; -import { LensIconChartLine } from '../../assets/chart_line'; import { PLOTLY_COLOR } from '../../../../../common/constants/shared'; import { VizDataPanel } from '../../../event_analytics/explorer/visualizations/config_panel/config_panes/default_vis_editor'; import { ConfigEditor } from '../../../event_analytics/explorer/visualizations/config_panel/config_panes/json_editor'; @@ -17,7 +16,6 @@ import { TextInputFieldItem, } from '../../../event_analytics/explorer/visualizations/config_panel/config_panes/config_controls'; import { fetchConfigObject } from '../../../event_analytics/utils/utils'; -import { ConfigAvailability } from '../../../event_analytics/explorer/visualizations/config_panel/config_panes/config_controls/config_availability'; import { DEFAULT_METRICS_CHART_PARAMETERS } from '../../../../../common/constants/explorer'; const sharedConfigs = getPlotlySharedConfigs(); @@ -40,7 +38,6 @@ export const createMetricsTypeDefinition = (params: any = {}) => ({ fulllabel: 'Metrics', icontype: 'stats', category: VIS_CATEGORY.BASICS, - icon: LensIconChartLine, categoryaxis: 'xaxis', seriesaxis: 'yaxis', charttype: DefaultChartType, diff --git a/public/components/visualizations/charts/pie/pie_type.ts b/public/components/visualizations/charts/pie/pie_type.ts index 2ec2dc1e7..b161c515f 100644 --- a/public/components/visualizations/charts/pie/pie_type.ts +++ b/public/components/visualizations/charts/pie/pie_type.ts @@ -5,18 +5,14 @@ import { Pie } from './pie'; import { getPlotlySharedConfigs, getPlotlyCategory } from '../shared/shared_configs'; -import { LensIconChartPie } from '../../assets/chart_pie'; import { VizDataPanel } from '../../../event_analytics/explorer/visualizations/config_panel/config_panes/default_vis_editor'; -import { ConfigEditor } from '../../../event_analytics/explorer/visualizations/config_panel/config_panes/json_editor'; import { - ColorPalettePicker, ConfigChartOptions, ConfigLegend, InputFieldItem, ButtonGroupItem, } from '../../../event_analytics/explorer/visualizations/config_panel/config_panes/config_controls'; import { fetchConfigObject } from '../../../../components/event_analytics/utils/utils'; -import { DEFAULT_PALETTE, PIE_PALETTES } from '../../../../../common/constants/colors'; import { PLOTLY_COLOR, DEFAULT_CHART_STYLES } from '../../../../../common/constants/shared'; import { DEFAULT_PIE_CHART_PARAMETERS } from '../../../../../common/constants/explorer'; @@ -39,7 +35,6 @@ export const createPieTypeDefinition = (params: any) => ({ categoryaxis: 'xaxis', seriesaxis: 'yaxis', mode: DefaultMode, - icon: LensIconChartPie, editorconfig: { panelTabs: [ { diff --git a/public/components/visualizations/charts/text/text_type.ts b/public/components/visualizations/charts/text/text_type.ts index 183fe9235..3e096ac5a 100644 --- a/public/components/visualizations/charts/text/text_type.ts +++ b/public/components/visualizations/charts/text/text_type.ts @@ -5,7 +5,6 @@ import { Text } from './text'; import { getPlotlySharedConfigs, getPlotlyCategory } from '../shared/shared_configs'; -import { LensIconChartLine } from '../../assets/chart_line'; import { PLOTLY_COLOR } from '../../../../../common/constants/shared'; import { VizDataPanel } from '../../../event_analytics/explorer/visualizations/config_panel/config_panes/default_vis_editor'; import { ConfigText } from '../../../event_analytics/explorer/visualizations/config_panel/config_panes/config_controls'; @@ -24,7 +23,6 @@ export const createTextTypeDefinition = (params: any = {}) => ({ selection: { dataLoss: 'nothing', }, - icon: LensIconChartLine, categoryaxis: 'xaxis', seriesaxis: 'yaxis', editorconfig: {