yujunjun/OpenMetadata
1
0
Fork 0
mirror of https://github.com/open-metadata/OpenMetadata.git synced 2026-01-08 05:26:19 +00:00
Code Issues Packages Projects Releases Wiki Activity

Bug Fixed Include Column for profiler settings disapear #7759 (#7761)

Browse Source 
* Bug Fixed Include Column for profiler settings disapear #7759

* fixed unit test
This commit is contained in:
Shailesh Parmar 2022-09-28 11:43:50 +05:30 committed by GitHub
parent 6c2f6089c7
commit f715362b7a
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 3 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
openmetadata-ui/src/main/resources/ui/src/components/TableProfiler/Component/ProfilerSettingsModal.test.tsx
View File

@ -17,11 +17,6 @@ import { MOCK_TABLE } from '../../../mocks/TableData.mock';
import { ProfilerSettingsModalProps } from '../TableProfiler.interface';
import ProfilerSettingsModal from './ProfilerSettingsModal';
jest.mock('antd/lib/form', () => {
return jest
.fn()
.mockImplementation(({ children }) => <form>{children}</form>);
});
jest.mock('antd/lib/grid', () => ({
Row: jest.fn().mockImplementation(({ children }) => <div>{children}</div>),
Col: jest

3
openmetadata-ui/src/main/resources/ui/src/components/TableProfiler/Component/ProfilerSettingsModal.tsx
View File

@ -47,6 +47,7 @@ const ProfilerSettingsModal: React.FC<ProfilerSettingsModalProps> = ({
visible,
onVisibilityChange,
}) => {
const [form] = Form.useForm();
const [data, setData] = useState<TableProfilerConfig>();
const [sqlQuery, setSqlQuery] = useState<string>('');
const [profileSample, setProfileSample] = useState<number>(100);
@ -94,6 +95,7 @@ const ProfilerSettingsModal: React.FC<ProfilerSettingsModalProps> = ({
return col;
});
form.setFieldsValue({ includeColumns: includeColValue });
setIncludeCol(includeColValue);
}
};
@ -261,6 +263,7 @@ const ProfilerSettingsModal: React.FC<ProfilerSettingsModalProps> = ({
<Col span={24}>
<Form
autoComplete="off"
form={form}
initialValues={{
includeColumns: includeCol,
}}