Skip to content

Commit

Permalink
feat: new CSV upload form and API (apache#27840)
Browse files Browse the repository at this point in the history
  • Loading branch information
dpgaspar authored Apr 15, 2024
1 parent 40e77be commit 54387b4
Show file tree
Hide file tree
Showing 30 changed files with 2,883 additions and 873 deletions.
419 changes: 209 additions & 210 deletions RESOURCES/STANDARD_ROLES.md

Large diffs are not rendered by default.

Original file line number Diff line number Diff line change
@@ -0,0 +1,355 @@
/**
* 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 fetchMock from 'fetch-mock';
import CSVUploadModal, {
validateUploadFileExtension,
} from 'src/features/databases/CSVUploadModal';
import { render, screen } from 'spec/helpers/testing-library';
import userEvent from '@testing-library/user-event';
import { waitFor } from '@testing-library/react';
import { UploadFile } from 'antd/lib/upload/interface';
import { forEach } from 'lodash';

fetchMock.post('glob:*api/v1/database/1/csv_upload/', {});

fetchMock.get(
'glob:*api/v1/database/?q=(filters:!((col:allow_file_upload,opr:eq,value:!t)),page:0,page_size:100)',
{
result: [
{
id: 1,
database_name: 'database1',
},
{
id: 2,
database_name: 'database2',
},
],
},
);

fetchMock.get('glob:*api/v1/database/1/schemas/', {
result: ['information_schema', 'public'],
});

fetchMock.get('glob:*api/v1/database/2/schemas/', {
result: ['schema1', 'schema2'],
});

const csvProps = {
show: true,
onHide: () => {},
allowedExtensions: ['csv', 'tsv'],
};

test('renders the general information elements correctly', () => {
render(<CSVUploadModal {...csvProps} />, {
useRedux: true,
});

const cancelButton = screen.getByRole('button', {
name: 'Cancel',
});
const uploadButton = screen.getByRole('button', {
name: 'Upload',
});
const selectButton = screen.getByRole('button', {
name: 'Select',
});

const title = screen.getByRole('heading', {
name: /csv upload/i,
});
const panel1 = screen.getByRole('heading', {
name: /General information/i,
});
const panel2 = screen.getByRole('heading', {
name: /file settings/i,
});
const panel3 = screen.getByRole('heading', {
name: /columns/i,
});
const panel4 = screen.getByRole('heading', {
name: /rows/i,
});
const selectDatabase = screen.getByRole('combobox', {
name: /select a database/i,
});
const selectDelimiter = screen.getByRole('combobox', {
name: /choose a delimiter/i,
});

const inputTableName = screen.getByRole('textbox', {
name: /table name/i,
});
const inputSchema = screen.getByRole('combobox', {
name: /schema/i,
});

const visibleComponents = [
cancelButton,
uploadButton,
selectButton,
title,
panel1,
panel2,
panel3,
panel4,
selectDatabase,
selectDelimiter,
inputTableName,
inputSchema,
];
visibleComponents.forEach(component => {
expect(component).toBeVisible();
});
});

test('renders the file settings elements correctly', () => {
render(<CSVUploadModal {...csvProps} />, {
useRedux: true,
});

expect(screen.queryByText('If Table Already Exists')).not.toBeInTheDocument();
const panelHeader = screen.getByRole('heading', {
name: /file settings/i,
});
userEvent.click(panelHeader);
const selectTableAlreadyExists = screen.getByRole('combobox', {
name: /choose already exists/i,
});
const switchSkipInitialSpace = screen.getByTestId('skipInitialSpace');
const switchSkipBlankLines = screen.getByTestId('skipBlankLines');
const switchDayFirst = screen.getByTestId('dayFirst');
const inputDecimalCharacter = screen.getByRole('textbox', {
name: /decimal character/i,
});
const selectColumnsDates = screen.getByRole('combobox', {
name: /choose columns to be parsed as dates/i,
});
const selectNullValues = screen.getByRole('combobox', {
name: /null values/i,
});
userEvent.click(selectColumnsDates);
userEvent.click(selectNullValues);
const visibleComponents = [
selectTableAlreadyExists,
switchSkipInitialSpace,
switchDayFirst,
switchSkipBlankLines,
inputDecimalCharacter,
selectNullValues,
];
visibleComponents.forEach(component => {
expect(component).toBeVisible();
});
});

test('renders the columns elements correctly', () => {
render(<CSVUploadModal {...csvProps} />, {
useRedux: true,
});

const panelHeader = screen.getByRole('heading', {
name: /columns/i,
});
userEvent.click(panelHeader);
const selectIndexColumn = screen.getByRole('combobox', {
name: /Choose index column/i,
});
const switchDataFrameIndex = screen.getByTestId('dataFrameIndex');
const inputColumnLabels = screen.getByRole('textbox', {
name: /Column labels/i,
});
const selectColumnsToRead = screen.getByRole('combobox', {
name: /Choose columns to read/i,
});
const switchOverwriteDuplicates = screen.getByTestId('overwriteDuplicates');
const inputColumnDataTypes = screen.getByRole('textbox', {
name: /Column data types/i,
});
userEvent.click(selectColumnsToRead);

const visibleComponents = [
selectIndexColumn,
switchDataFrameIndex,
inputColumnLabels,
selectColumnsToRead,
switchOverwriteDuplicates,
inputColumnDataTypes,
];
visibleComponents.forEach(component => {
expect(component).toBeVisible();
});
});

test('renders the rows elements correctly', () => {
render(<CSVUploadModal {...csvProps} />, {
useRedux: true,
});

const panelHeader = screen.getByRole('heading', {
name: /rows/i,
});
userEvent.click(panelHeader);
const inputHeaderRow = screen.getByRole('spinbutton', {
name: /header row/i,
});
const inputRowsToRead = screen.getByRole('spinbutton', {
name: /rows to read/i,
});
const inputSkipRows = screen.getByRole('spinbutton', {
name: /skip rows/i,
});

const visibleComponents = [inputHeaderRow, inputRowsToRead, inputSkipRows];
visibleComponents.forEach(component => {
expect(component).toBeVisible();
});
});

test('database and schema are correctly populated', async () => {
render(<CSVUploadModal {...csvProps} />, {
useRedux: true,
});

const selectDatabase = screen.getByRole('combobox', {
name: /select a database/i,
});
const selectSchema = screen.getByRole('combobox', {
name: /schema/i,
});

userEvent.click(selectDatabase);

await waitFor(() => screen.getByText('database1'));
await waitFor(() => screen.getByText('database2'));

screen.getByText('database1').click();
userEvent.click(selectSchema);
// make sure the schemas for database1 are displayed
await waitFor(() => screen.getAllByText('information_schema'));
await waitFor(() => screen.getAllByText('public'));

screen.getByText('database2').click();
userEvent.click(selectSchema);
// make sure the schemas for database2 are displayed
await waitFor(() => screen.getAllByText('schema1'));
await waitFor(() => screen.getAllByText('schema2'));
});

test('form without required fields', async () => {
render(<CSVUploadModal {...csvProps} />, {
useRedux: true,
});

const uploadButton = screen.getByRole('button', {
name: 'Upload',
});

// Submit form without filling any fields
userEvent.click(uploadButton);

await waitFor(() => screen.getByText('Uploading a file is required'));
await waitFor(() => screen.getByText('Selecting a database is required'));
await waitFor(() => screen.getByText('Table name is required'));
});
test('form post', async () => {
render(<CSVUploadModal {...csvProps} />, {
useRedux: true,
});

const selectButton = screen.getByRole('button', {
name: 'Select',
});
userEvent.click(selectButton);

// Select a file from the file dialog
const file = new File(['test'], 'test.csv', { type: 'text' });
const inputElement = document.querySelector('input[type="file"]');

if (inputElement) {
userEvent.upload(inputElement, file);
}

const selectDatabase = screen.getByRole('combobox', {
name: /select a database/i,
});
userEvent.click(selectDatabase);
await waitFor(() => screen.getByText('database1'));
await waitFor(() => screen.getByText('database2'));

screen.getByText('database1').click();
const selectSchema = screen.getByRole('combobox', {
name: /schema/i,
});
userEvent.click(selectSchema);
await waitFor(() => screen.getAllByText('public'));
screen.getAllByText('public')[1].click();

// Fill out form fields
const inputTableName = screen.getByRole('textbox', {
name: /table name/i,
});
userEvent.type(inputTableName, 'table1');
const uploadButton = screen.getByRole('button', {
name: 'Upload',
});

userEvent.click(uploadButton);
await waitFor(() => fetchMock.called('glob:*api/v1/database/1/csv_upload/'));

// Get the matching fetch calls made
const matchingCalls = fetchMock.calls('glob:*api/v1/database/1/csv_upload/');
expect(matchingCalls).toHaveLength(1);
const [_, options] = matchingCalls[0];
const formData = options?.body as FormData;
expect(formData.get('table_name')).toBe('table1');
expect(formData.get('schema')).toBe('public');
expect(formData.get('table_name')).toBe('table1');
const fileData = formData.get('file') as File;
expect(fileData.name).toBe('test.csv');
});

test('validate file extension returns false', () => {
const invalidFileNames = ['out', 'out.exe', 'out.csv.exe', '.csv'];
forEach(invalidFileNames, fileName => {
const file: UploadFile<any> = {
name: fileName,
uid: 'xp',
size: 100,
type: 'text/csv',
};
expect(validateUploadFileExtension(file, ['csv', 'tsv'])).toBe(false);
});
});

test('validate file extension returns true', () => {
const invalidFileNames = ['out.csv', 'out.tsv', 'out.exe.csv', 'out a.csv'];
forEach(invalidFileNames, fileName => {
const file: UploadFile<any> = {
name: fileName,
uid: 'xp',
size: 100,
type: 'text/csv',
};
expect(validateUploadFileExtension(file, ['csv', 'tsv'])).toBe(true);
});
});
Loading

0 comments on commit 54387b4

Please sign in to comment.