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

(cherry picked from commit a85b716)
  • Loading branch information
anirudha authored and mengweieric committed Jun 30, 2023
1 parent bf7d526 commit 3bc44b6
Show file tree
Hide file tree
Showing 34 changed files with 827 additions and 455 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -418,7 +418,6 @@ exports[`Utils helper functions renders displayVisualization function 1`] = `
"fillopacity": 70,
"fulllabel": "Vertical bar",
"groupwidth": 0.7,
"icon": [Function],
"icontype": "visBarVerticalStacked",
"id": "bar",
"label": "Vertical bar",
Expand Down Expand Up @@ -876,7 +875,6 @@ exports[`Utils helper functions renders displayVisualization function 1`] = `
"fillopacity": 70,
"fulllabel": "Vertical bar",
"groupwidth": 0.7,
"icon": [Function],
"icontype": "visBarVerticalStacked",
"id": "bar",
"label": "Vertical bar",
Expand Down Expand Up @@ -1341,7 +1339,6 @@ exports[`Utils helper functions renders displayVisualization function 1`] = `
"fillopacity": 70,
"fulllabel": "Vertical bar",
"groupwidth": 0.7,
"icon": [Function],
"icontype": "visBarVerticalStacked",
"id": "bar",
"label": "Vertical bar",
Expand Down Expand Up @@ -2051,7 +2048,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 @@ -2577,7 +2573,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 @@ -3117,7 +3112,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 @@ -3802,7 +3796,6 @@ exports[`Utils helper functions renders displayVisualization function 3`] = `
"fillopacity": 70,
"fulllabel": "Horizontal bar",
"groupwidth": 0.7,
"icon": [Function],
"icontype": "visBarHorizontalStacked",
"id": "horizontal_bar",
"label": "Horizontal bar",
Expand Down Expand Up @@ -4260,7 +4253,6 @@ exports[`Utils helper functions renders displayVisualization function 3`] = `
"fillopacity": 70,
"fulllabel": "Horizontal bar",
"groupwidth": 0.7,
"icon": [Function],
"icontype": "visBarHorizontalStacked",
"id": "horizontal_bar",
"label": "Horizontal bar",
Expand Down Expand Up @@ -4725,7 +4717,6 @@ exports[`Utils helper functions renders displayVisualization function 3`] = `
"fillopacity": 70,
"fulllabel": "Horizontal bar",
"groupwidth": 0.7,
"icon": [Function],
"icontype": "visBarHorizontalStacked",
"id": "horizontal_bar",
"label": "Horizontal bar",
Expand Down

Large diffs are not rendered by default.

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`] = `
<EmptyPlaceholder
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]}
>
<LensIconChartBar
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>
</LensIconChartBar>
</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>
</EmptyPlaceholder>
`;

exports[`Shared components Renders tool bar button component 1`] = `
<ToolbarButton
fontWeight="bold"
Expand Down
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
Original file line number Diff line number Diff line change
@@ -1 +1,2 @@
export { PlotlyVizEditor } from './plotly_viz_editor';
export { PlotlyEditorActionsMenu } from './plotly_actions';
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

This file was deleted.

56 changes: 0 additions & 56 deletions public/components/visualizations/assets/__tests__/assets.test.tsx

This file was deleted.

29 changes: 0 additions & 29 deletions public/components/visualizations/assets/chart_bar.tsx

This file was deleted.

33 changes: 0 additions & 33 deletions public/components/visualizations/assets/chart_bar_horizontal.tsx

This file was deleted.

Loading

0 comments on commit 3bc44b6

Please sign in to comment.