Skip to content

Commit

Permalink
feat: add modal to import charts (#11956)
Browse files Browse the repository at this point in the history
* WIP

* Splat props
  • Loading branch information
betodealmeida committed Dec 8, 2020
1 parent fbb458f commit 33325f9
Show file tree
Hide file tree
Showing 11 changed files with 401 additions and 16 deletions.
@@ -0,0 +1,106 @@
/**
* Licensed to the Apache Software Foundation (ASF) under one
* or more contributor license agreements. See the NOTICE file
* distributed with this work for additional information
* regarding copyright ownership. The ASF licenses this file
* to you under the Apache License, Version 2.0 (the
* "License"); you may not use this file except in compliance
* with the License. You may obtain a copy of the License at
*
* http://www.apache.org/licenses/LICENSE-2.0
*
* Unless required by applicable law or agreed to in writing,
* software distributed under the License is distributed on an
* "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
* KIND, either express or implied. See the License for the
* specific language governing permissions and limitations
* under the License.
*/
import React from 'react';
import thunk from 'redux-thunk';
import configureStore from 'redux-mock-store';
import { styledMount as mount } from 'spec/helpers/theming';
import { ReactWrapper } from 'enzyme';

import ImportChartModal from 'src/chart/components/ImportModal';
import Modal from 'src/common/components/Modal';

const mockStore = configureStore([thunk]);
const store = mockStore({});

const requiredProps = {
addDangerToast: () => {},
addSuccessToast: () => {},
onChartImport: () => {},
show: true,
onHide: () => {},
};

describe('ImportChartModal', () => {
let wrapper: ReactWrapper;

beforeEach(() => {
wrapper = mount(<ImportChartModal {...requiredProps} />, {
context: { store },
});
});

afterEach(() => {
jest.clearAllMocks();
});

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

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

it('renders "Import Chart" header', () => {
expect(wrapper.find('h4').text()).toEqual('Import Chart');
});

it('renders a label and a file input field', () => {
expect(wrapper.find('input[type="file"]')).toExist();
expect(wrapper.find('label')).toExist();
});

it('should attach the label to the input field', () => {
const id = 'chartFile';
expect(wrapper.find('label').prop('htmlFor')).toBe(id);
expect(wrapper.find('input').prop('id')).toBe(id);
});

it('should render the close, import and cancel buttons', () => {
expect(wrapper.find('button')).toHaveLength(3);
});

it('should render the import button initially disabled', () => {
expect(wrapper.find('button[children="Import"]').prop('disabled')).toBe(
true,
);
});

it('should render the import button enabled when a file is selected', () => {
const file = new File([new ArrayBuffer(1)], 'chart_export.zip');
wrapper.find('input').simulate('change', { target: { files: [file] } });

expect(wrapper.find('button[children="Import"]').prop('disabled')).toBe(
false,
);
});

it('should render password fields when needed for import', () => {
const wrapperWithPasswords = mount(
<ImportChartModal
{...requiredProps}
passwordFields={['databases/examples.yaml']}
/>,
{
context: { store },
},
);
expect(wrapperWithPasswords.find('input[type="password"]')).toExist();
});
});
186 changes: 186 additions & 0 deletions superset-frontend/src/chart/components/ImportModal/index.tsx
@@ -0,0 +1,186 @@
/**
* Licensed to the Apache Software Foundation (ASF) under one
* or more contributor license agreements. See the NOTICE file
* distributed with this work for additional information
* regarding copyright ownership. The ASF licenses this file
* to you under the Apache License, Version 2.0 (the
* "License"); you may not use this file except in compliance
* with the License. You may obtain a copy of the License at
*
* http://www.apache.org/licenses/LICENSE-2.0
*
* Unless required by applicable law or agreed to in writing,
* software distributed under the License is distributed on an
* "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
* KIND, either express or implied. See the License for the
* specific language governing permissions and limitations
* under the License.
*/
import React, { FunctionComponent, useEffect, useRef, useState } from 'react';
import { t } from '@superset-ui/core';

import Modal from 'src/common/components/Modal';
import {
StyledIcon,
StyledInputContainer,
} from 'src/views/CRUD/data/database/DatabaseModal';
import { useImportResource } from 'src/views/CRUD/hooks';
import { ChartObject } from 'src/views/CRUD/chart/types';

export interface ImportChartModalProps {
addDangerToast: (msg: string) => void;
addSuccessToast: (msg: string) => void;
onChartImport: () => void;
show: boolean;
onHide: () => void;
passwordFields?: string[];
setPasswordFields?: (passwordFields: string[]) => void;
}

const ImportChartModal: FunctionComponent<ImportChartModalProps> = ({
addDangerToast,
addSuccessToast,
onChartImport,
show,
onHide,
passwordFields = [],
setPasswordFields = () => {},
}) => {
const [uploadFile, setUploadFile] = useState<File | null>(null);
const [isHidden, setIsHidden] = useState<boolean>(true);
const [passwords, setPasswords] = useState<Record<string, string>>({});
const fileInputRef = useRef<HTMLInputElement>(null);

const clearModal = () => {
setUploadFile(null);
setPasswordFields([]);
setPasswords({});
if (fileInputRef && fileInputRef.current) {
fileInputRef.current.value = '';
}
};

const handleErrorMsg = (msg: string) => {
clearModal();
addDangerToast(msg);
};

const {
state: { passwordsNeeded },
importResource,
} = useImportResource<ChartObject>('chart', t('chart'), handleErrorMsg);

useEffect(() => {
setPasswordFields(passwordsNeeded);
}, [passwordsNeeded]);

// Functions
const hide = () => {
setIsHidden(true);
onHide();
};

const onUpload = () => {
if (uploadFile === null) {
return;
}

importResource(uploadFile, passwords).then(result => {
if (result) {
addSuccessToast(t('The charts have been imported'));
clearModal();
onChartImport();
}
});
};

const changeFile = (event: React.ChangeEvent<HTMLInputElement>) => {
const { files } = event.target as HTMLInputElement;
setUploadFile((files && files[0]) || null);
};

const renderPasswordFields = () => {
if (passwordFields.length === 0) {
return null;
}

return (
<>
<h5>Database passwords</h5>
<StyledInputContainer>
<div className="helper">
{t(
'The passwords for the databases below are needed in order to ' +
'import them together with the charts. Please note that the ' +
'"Secure Extra" and "Certificate" sections of ' +
'the database configuration are not present in export files, and ' +
'should be added manually after the import if they are needed.',
)}
</div>
</StyledInputContainer>
{passwordFields.map(fileName => (
<StyledInputContainer key={`password-for-${fileName}`}>
<div className="control-label">
{fileName}
<span className="required">*</span>
</div>
<input
name={`password-${fileName}`}
autoComplete="off"
type="password"
value={passwords[fileName]}
onChange={event =>
setPasswords({ ...passwords, [fileName]: event.target.value })
}
/>
</StyledInputContainer>
))}
</>
);
};

// Show/hide
if (isHidden && show) {
setIsHidden(false);
}

return (
<Modal
name="chart"
className="chart-modal"
disablePrimaryButton={uploadFile === null}
onHandledPrimaryAction={onUpload}
onHide={hide}
primaryButtonName={t('Import')}
width="750px"
show={show}
title={
<h4>
<StyledIcon name="nav-charts" />
{t('Import Chart')}
</h4>
}
>
<StyledInputContainer>
<div className="control-label">
<label htmlFor="chartFile">
{t('File')}
<span className="required">*</span>
</label>
</div>
<input
ref={fileInputRef}
data-test="chart-file-input"
name="chartFile"
id="chartFile"
type="file"
accept=".yaml,.json,.yml,.zip"
onChange={changeFile}
/>
</StyledInputContainer>
{renderPasswordFields()}
</Modal>
);
};

export default ImportChartModal;
Expand Up @@ -95,7 +95,7 @@ describe('ImportDatasetModal', () => {
const wrapperWithPasswords = mount(
<ImportDatasetModal
{...requiredProps}
passwordFields={['datasets/examples.yaml']}
passwordFields={['databases/examples.yaml']}
/>,
{
context: { store },
Expand Down

0 comments on commit 33325f9

Please sign in to comment.