Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: saved query list view + sort/filters #11005

Merged
merged 13 commits into from
Sep 29, 2020
Merged
Original file line number Diff line number Diff line change
Expand Up @@ -19,15 +19,68 @@
import React from 'react';
import thunk from 'redux-thunk';
import configureStore from 'redux-mock-store';
import fetchMock from 'fetch-mock';
import { styledMount as mount } from 'spec/helpers/theming';
import SavedQueryList from 'src/views/CRUD/data/savedquery/SavedQueryList';
import SubMenu from 'src/components/Menu/SubMenu';
import ListView from 'src/components/ListView';
import Filters from 'src/components/ListView/Filters';
import waitForComponentToPaint from 'spec/helpers/waitForComponentToPaint';
import { act } from 'react-dom/test-utils';

// store needed for withToasts(DatabaseList)
const mockStore = configureStore([thunk]);
const store = mockStore({});

const queriesInfoEndpoint = 'glob:*/api/v1/saved_query/_info*';
const queriesEndpoint = 'glob:*/api/v1/saved_query/?*';
const queriesRelatedEndpoint = 'glob:*/api/v1/saved_query/related/database?*';
const queriesDistinctEndpoint = 'glob:*/api/v1/saved_query/distinct/schema?*';

const mockqueries = [...new Array(3)].map((_, i) => ({
created_by: {
id: i,
first_name: `user`,
last_name: `${i}`,
},
created_on: `${i}-2020`,
database: {
database_name: `db ${i}`,
id: i,
},
changed_on_delta_humanized: '1 day ago',
db_id: i,
description: `SQL for ${i}`,
label: `query ${i}`,
schema: 'public',
sql: `SELECT ${i} FROM table`,
sql_tables: [
{
catalog: null,
schema: null,
table: `${i}`,
},
],
}));

fetchMock.get(queriesInfoEndpoint, {
permissions: ['can_delete'],
});
fetchMock.get(queriesEndpoint, {
result: mockqueries,
count: 3,
});

fetchMock.get(queriesRelatedEndpoint, {
count: 0,
result: [],
});

fetchMock.get(queriesDistinctEndpoint, {
count: 0,
result: [],
});

describe('SavedQueryList', () => {
const wrapper = mount(<SavedQueryList />, { context: { store } });

Expand All @@ -42,4 +95,28 @@ describe('SavedQueryList', () => {
it('renders a SubMenu', () => {
expect(wrapper.find(SubMenu)).toExist();
});

it('renders a ListView', () => {
expect(wrapper.find(ListView)).toExist();
});

it('fetches saved queries', () => {
const callsQ = fetchMock.calls(/saved_query\/\?q/);
expect(callsQ).toHaveLength(1);
expect(callsQ[0][0]).toMatchInlineSnapshot(
`"http://localhost/api/v1/saved_query/?q=(order_column:changed_on_delta_humanized,order_direction:desc,page:0,page_size:25)"`,
);
});

it('searches', async () => {
const filtersWrapper = wrapper.find(Filters);
act(() => {
filtersWrapper.find('[name="label"]').first().props().onSubmit('fooo');
});
await waitForComponentToPaint(wrapper);

expect(fetchMock.lastCall()[0]).toMatchInlineSnapshot(
`"http://localhost/api/v1/saved_query/?q=(filters:!((col:label,opr:all_text,value:fooo)),order_column:changed_on_delta_humanized,order_direction:desc,page:0,page_size:25)"`,
);
});
});
273 changes: 272 additions & 1 deletion superset-frontend/src/views/CRUD/data/savedquery/SavedQueryList.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -17,28 +17,299 @@
* under the License.
*/

import React from 'react';
import { t, styled } from '@superset-ui/core';
import React, { useMemo } from 'react';
import moment from 'moment';
import {
createFetchRelated,
createFetchDistinct,
createErrorHandler,
} from 'src/views/CRUD/utils';
import { Popover } from 'src/common/components';
import withToasts from 'src/messageToasts/enhancers/withToasts';
import { useListViewResource } from 'src/views/CRUD/hooks';
import SubMenu, { SubMenuProps } from 'src/components/Menu/SubMenu';
import ListView, { Filters } from 'src/components/ListView';
import TooltipWrapper from 'src/components/TooltipWrapper';
import Icon from 'src/components/Icon';
import { commonMenuData } from 'src/views/CRUD/data/common';

const PAGE_SIZE = 25;

interface SavedQueryListProps {
addDangerToast: (msg: string) => void;
addSuccessToast: (msg: string) => void;
}

type SavedQueryObject = {};

const StyledTableLabel = styled.div`
.count {
margin-left: 5px;
color: ${({ theme }) => theme.colors.primary.base};
text-decoration: underline;
cursor: pointer;
}
`;

const StyledPopoverItem = styled.div`
color: ${({ theme }) => theme.colors.grayscale.dark2};
`;

function SavedQueryList({
addDangerToast,
addSuccessToast,
}: SavedQueryListProps) {
const {
state: { loading, resourceCount: queryCount, resourceCollection: queries },
hasPerm,
fetchData,
// refreshData, //TODO: add back later when editing?
} = useListViewResource<SavedQueryObject>(
'saved_query',
t('saved_queries'),
addDangerToast,
);

const canCreate = hasPerm('can_add');
const canEdit = hasPerm('can_edit');
const canDelete = hasPerm('can_delete');

const menuData: SubMenuProps = {
activeChild: 'Saved Queries',
...commonMenuData,
};

const initialSort = [{ id: 'changed_on_delta_humanized', desc: true }];
const columns = useMemo(
() => [
{
accessor: 'label',
Header: t('Name'),
},
{
accessor: 'database.database_name',
Header: t('Database'),
},
{
accessor: 'database',
hidden: true,
disableSortBy: true,
},
{
accessor: 'schema',
Header: t('Schema'),
},
{
Cell: ({
row: {
original: { sql_tables: tables },
},
}: any) => {
const names = tables.map((table: any) => table.table);
const main = names.shift();

if (names.length) {
return (
<StyledTableLabel>
<span>{main}</span>
<Popover
placement="right"
title={t('TABLES')}
trigger="click"
content={
<>
{names.map((name: string) => (
<StyledPopoverItem>{name}</StyledPopoverItem>
))}
</>
}
>
<span className="count">(+{names.length})</span>
</Popover>
</StyledTableLabel>
);
}

return main;
},
accessor: 'sql_tables',
Header: t('Tables'),
disableSortBy: true,
},
{
Cell: ({
row: {
original: { created_on: createdOn },
},
}: any) => {
const date = new Date(createdOn);
const utc = new Date(
Date.UTC(
date.getFullYear(),
date.getMonth(),
date.getDate(),
date.getHours(),
date.getMinutes(),
date.getSeconds(),
date.getMilliseconds(),
),
);

return moment(utc).fromNow();
},
Header: t('Created On'),
accessor: 'created_on',
},
{
Cell: ({
row: {
original: { changed_on_delta_humanized: changedOn },
},
}: any) => changedOn,
Header: t('Modified'),
accessor: 'changed_on_delta_humanized',
},
{
Cell: ({ row: { original } }: any) => {
const handlePreview = () => {}; // openQueryPreviewModal(original); // TODO: open preview modal
const handleEdit = () => {}; // handleQueryEdit(original); // TODO: navigate to sql editor with selected query open
const handleCopy = () => {}; // TODO: copy link to clipboard
const handleDelete = () => {}; // openQueryDeleteModal(original);

return (
<span className="actions">
<TooltipWrapper
label="preview-action"
tooltip={t('Query preview')}
placement="bottom"
>
<span
role="button"
tabIndex={0}
className="action-button"
onClick={handlePreview}
>
<Icon name="binoculars" />
</span>
</TooltipWrapper>
{canEdit && (
<TooltipWrapper
label="edit-action"
tooltip={t('Edit query')}
placement="bottom"
>
<span
role="button"
tabIndex={0}
className="action-button"
onClick={handleEdit}
>
<Icon name="edit" />
</span>
</TooltipWrapper>
)}
<TooltipWrapper
label="copy-action"
tooltip={t('Copy query URL')}
placement="bottom"
>
<span
role="button"
tabIndex={0}
className="action-button"
onClick={handleCopy}
>
<Icon name="copy" />
</span>
</TooltipWrapper>
{canDelete && (
<span
role="button"
tabIndex={0}
className="action-button"
data-test="database-delete"
onClick={handleDelete}
>
<TooltipWrapper
label="delete-action"
tooltip={t('Delete query')}
placement="bottom"
>
<Icon name="trash" />
</TooltipWrapper>
</span>
)}
</span>
);
},
Header: t('Actions'),
id: 'actions',
disableSortBy: true,
},
],
[canDelete, canCreate],
);

const filters: Filters = useMemo(
() => [
{
Header: t('Database'),
id: 'database',
input: 'select',
operator: 'rel_o_m',
unfilteredLabel: 'All',
fetchSelects: createFetchRelated(
'saved_query',
'database',
createErrorHandler(errMsg =>
t(
'An error occurred while fetching dataset datasource values: %s',
errMsg,
),
),
),
paginate: true,
},
{
Header: t('Schema'),
id: 'schema',
input: 'select',
operator: 'eq',
unfilteredLabel: 'All',
fetchSelects: createFetchDistinct(
'saved_query',
'schema',
createErrorHandler(errMsg =>
t('An error occurred while fetching schema values: %s', errMsg),
),
),
paginate: true,
},
{
Header: t('Search'),
id: 'label',
input: 'search',
operator: 'all_text',
},
],
[],
);

return (
<>
<SubMenu {...menuData} />
<ListView<SavedQueryObject>
className="saved_query-list-view"
columns={columns}
count={queryCount}
data={queries}
fetchData={fetchData}
filters={filters}
initialSort={initialSort}
loading={loading}
pageSize={PAGE_SIZE}
/>
</>
);
}
Expand Down
Loading