Skip to content

Commit

Permalink
Merge pull request opensearch-project#588 from mengweieric/feature/pr…
Browse files Browse the repository at this point in the history 
…erelease-code-cleanup
  • Loading branch information
@anirudha
anirudha authored Jun 30, 2023
2 parents 6d5bd47 + bd11c23 commit a85b716
Show file tree
Hide file tree
Showing 34 changed files with 3 additions and 456 deletions.
9 changes: 0 additions & 9 deletions public/components/custom_panels/helpers/__tests__/__snapshots__/utils.test.tsx.snap
View file
Original file line number Diff line number Diff line change
Expand Up @@ -416,7 +416,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",
Expand Down Expand Up @@ -872,7 +871,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",
Expand Down Expand Up @@ -1335,7 +1333,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",
Expand Down Expand Up @@ -2036,7 +2033,6 @@ exports[`Utils helper functions renders displayVisualization function 2`] = `
],
},
"fulllabel": "Time series",
"icon": [Function],
"icontype": "visLine",
"id": "line",
"label": "Time series",
Expand Down Expand Up @@ -2553,7 +2549,6 @@ exports[`Utils helper functions renders displayVisualization function 2`] = `
],
},
"fulllabel": "Time series",
"icon": [Function],
"icontype": "visLine",
"id": "line",
"label": "Time series",
Expand Down Expand Up @@ -3084,7 +3079,6 @@ exports[`Utils helper functions renders displayVisualization function 2`] = `
],
},
"fulllabel": "Time series",
"icon": [Function],
"icontype": "visLine",
"id": "line",
"label": "Time series",
Expand Down Expand Up @@ -3767,7 +3761,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",
Expand Down Expand Up @@ -4223,7 +4216,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",
Expand Down Expand Up @@ -4686,7 +4678,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",
Expand Down
21 changes: 0 additions & 21 deletions ...s/explorer/visualizations/config_panel/__tests__/__snapshots__/config_panel.test.tsx.snap
View file
Original file line number Diff line number Diff line change
Expand Up @@ -624,7 +624,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",
Expand Down Expand Up @@ -947,7 +946,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",
Expand Down Expand Up @@ -1253,7 +1251,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",
Expand Down Expand Up @@ -1581,7 +1578,6 @@ exports[`Config panel component Renders config panel with visualization data 1`]
],
},
"fulllabel": "Time series",
"icon": [Function],
"icontype": "visLine",
"id": "line",
"label": "Time series",
Expand Down Expand Up @@ -1822,7 +1818,6 @@ exports[`Config panel component Renders config panel with visualization data 1`]
],
},
"fulllabel": "Pie",
"icon": [Function],
"icontype": "visPie",
"id": "pie",
"label": "Pie",
Expand Down Expand Up @@ -2262,7 +2257,6 @@ exports[`Config panel component Renders config panel with visualization data 1`]
],
},
"fulllabel": "Hubble",
"icon": [Function],
"icontype": "heatmap",
"id": "heatmap",
"label": "Heatmap",
Expand Down Expand Up @@ -2315,7 +2309,6 @@ exports[`Config panel component Renders config panel with visualization data 1`]
],
},
"fulllabel": "Markdown",
"icon": [Function],
"icontype": "visText",
"id": "text",
"label": "Markdown",
Expand Down Expand Up @@ -2650,7 +2643,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",
Expand Down Expand Up @@ -2988,7 +2980,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",
Expand Down Expand Up @@ -3332,7 +3323,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"
Expand Down Expand Up @@ -3609,7 +3599,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",
Expand Down Expand Up @@ -3954,7 +3943,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"
Expand Down Expand Up @@ -4268,7 +4256,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}
Expand Down Expand Up @@ -5359,7 +5346,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",
Expand Down Expand Up @@ -6302,7 +6288,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",
Expand Down Expand Up @@ -7286,7 +7271,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",
Expand Down Expand Up @@ -7998,7 +7982,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",
Expand Down Expand Up @@ -8977,7 +8960,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",
Expand Down Expand Up @@ -10455,7 +10437,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",
Expand Down Expand Up @@ -11954,7 +11935,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",
Expand Down Expand Up @@ -15018,7 +14998,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",
Expand Down
90 changes: 0 additions & 90 deletions .../visualizations/shared_components/__tests__/__snapshots__/shared_components.test.tsx.snap
View file
Original file line number Diff line number Diff line change
@@ -1,95 +1,5 @@
// Jest Snapshot v1, https://goo.gl/fbAQLP

exports[`Shared components Renders empty placeholder component 1`] = `
<Component
icon={[Function]}
>
<EuiText
className="lnsChart__empty"
color="subdued"
data-test-subj="vizWorkspace__noData"
size="xs"
textAlign="center"
>
<div
className="euiText euiText--extraSmall lnsChart__empty"
data-test-subj="vizWorkspace__noData"
>
<EuiTextAlign
textAlign="center"
>
<div
className="euiTextAlign euiTextAlign--center"
>
<EuiTextColor
color="subdued"
component="div"
>
<div
className="euiTextColor euiTextColor--subdued"
>
<EuiIcon
color="subdued"
size="xxl"
type={[Function]}
>
<Component
aria-hidden={true}
className="euiIcon euiIcon--xxLarge euiIcon--subdued"
focusable="false"
role="img"
style={null}
>
<svg
aria-hidden={true}
className="euiIcon euiIcon--xxLarge euiIcon--subdued"
fill="none"
focusable="false"
height={22}
role="img"
style={null}
viewBox="0 0 30 22"
width={30}
xmlns="http://www.w3.org/2000/svg"
>
<path
className="lensChartIcon__subdued"
d="M5 7a1 1 0 011 1v13a1 1 0 01-1 1H1a1 1 0 01-1-1V8a1 1 0 011-1h4zm16-7a1 1 0 011 1v20a1 1 0 01-1 1h-4a1 1 0 01-1-1V1a1 1 0 011-1h4z"
/>
<path
className="lensChartIcon__accent"
d="M13 11a1 1 0 011 1v9a1 1 0 01-1 1H9a1 1 0 01-1-1v-9a1 1 0 011-1h4zm16-7a1 1 0 011 1v16a1 1 0 01-1 1h-4a1 1 0 01-1-1V5a1 1 0 011-1h4z"
/>
</svg>
</Component>
</EuiIcon>
<EuiSpacer
size="l"
>
<div
className="euiSpacer euiSpacer--l"
/>
</EuiSpacer>
<p>
<FormattedMessage
defaultMessage="No data found"
id="visualization_noData"
values={Object {}}
>
<span>
No data found
</span>
</FormattedMessage>
</p>
</div>
</EuiTextColor>
</div>
</EuiTextAlign>
</div>
</EuiText>
</Component>
`;

exports[`Shared components Renders tool bar button component 1`] = `
<Component
fontWeight="bold"
Expand Down
12 changes: 0 additions & 12 deletions ..._analytics/explorer/visualizations/shared_components/__tests__/shared_components.test.tsx
View file
Original file line number Diff line number Diff line change
Expand Up @@ -7,23 +7,11 @@ import { configure, mount } from 'enzyme';
import Adapter from 'enzyme-adapter-react-16';
import React from 'react';
import { waitFor } from '@testing-library/react';
import { EmptyPlaceholder } from '../empty_placeholder';
import { LensIconChartBar } from '../../../../../visualizations/assets/chart_bar';
import { ToolbarButton } from '../toolbar_button';

describe('Shared components', () => {
configure({ adapter: new Adapter() });

it('Renders empty placeholder component', async () => {
const wrapper = mount(<EmptyPlaceholder icon={LensIconChartBar} />);

wrapper.update();

await waitFor(() => {
expect(wrapper).toMatchSnapshot();
});
});

it('Renders tool bar button component', async () => {
const handleClick = jest.fn();
const WrappedComponent = () => <div>testing</div>;
Expand Down
1 change: 1 addition & 0 deletions ...nts/event_analytics/explorer/visualizations/shared_components/plotly_viz_editor/index.tsx
View file
Original file line number Diff line number Diff line change
@@ -1 +1,2 @@
export { PlotlyVizEditor } from './plotly_viz_editor';
export { PlotlyEditorActionsMenu } from './plotly_actions';
2 changes: 0 additions & 2 deletions ...ic/components/event_analytics/explorer/visualizations/visualization_specs/default_spec.ts
View file
Original file line number Diff line number Diff line change
@@ -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';

Expand Down
Loading

0 comments on commit a85b716

Please sign in to comment.