From 421943fb67730989d9e2e927d4900ad69acb16a8 Mon Sep 17 00:00:00 2001 From: Harsh Vador Date: Tue, 31 Mar 2026 16:03:52 +0530 Subject: [PATCH 1/2] fix eslint warnings --- .../FeedEditor/FeedEditor.test.tsx | 95 +++--- .../ActivityFeed/FeedEditor/FeedEditor.tsx | 4 +- .../components/AppBar/Suggestions.test.tsx | 10 +- .../AppRouter/SettingsRouter.test.tsx | 4 +- .../Auth/AuthProviders/AuthProvider.tsx | 8 +- .../DataAssetSummaryPanelV1.test.tsx | 12 +- .../ContractDetailFormTab.test.tsx | 2 +- .../ContractDetailTab/ContractDetail.test.tsx | 16 +- .../ContractExecutionChart.test.tsx | 22 +- .../ContractQualityFormTab.test.tsx | 18 +- .../ContractSLACard/ContractSLA.test.tsx | 2 +- .../ContractSchemaFormTab.test.tsx | 37 ++- .../ContractSemanticFormTab.test.tsx | 4 +- .../ContractTab/ContractTab.test.tsx | 4 +- .../ContractYaml/ContractYaml.test.tsx | 2 +- .../DataProductsPage.component.tsx | 56 ++-- .../TestCaseStatusModal.test.tsx | 4 +- .../QueryCardExtraOption.test.tsx | 4 +- .../DomainListing/DomainListPage.tsx | 14 +- .../components/DomainTreeView.tsx | 7 +- .../DirectoryVersion.test.tsx | 2 +- .../DriveService/File/FileDetails.test.tsx | 2 +- .../File/FileVersion/FileVersion.test.tsx | 2 +- .../SpreadsheetVersion.test.tsx | 2 +- .../Worksheet/WorksheetDetails.test.tsx | 2 +- .../WorksheetVersion.test.tsx | 2 +- .../NodeSuggestions.component.tsx | 3 +- .../AdvanceSearchProvider.test.tsx | 2 +- .../WorkFlowTab/WorkFlowHistory.interface.ts | 2 +- .../GlossaryUpdateConfirmationModal.test.tsx | 2 +- .../KnowledgeGraph/KnowledgeGraph.tsx | 2 +- .../src/components/Lineage/Lineage.test.tsx | 2 +- .../useLineagetTableState.test.ts | 13 +- .../MlModelDetail.component.test.tsx | 2 +- .../ChangeParentHierarchy.test.tsx | 2 +- .../Modals/TourEndModal/TourEndModal.test.tsx | 2 +- .../WhatsNewAlert/WhatsNewAlert.test.tsx | 6 +- .../MyData/FeedWidget/FeedWidget.test.tsx | 11 +- .../MyData/RightSidebar/FollowingWidget.tsx | 2 +- .../SelectAssetTypeField.test.tsx | 4 +- .../DataProductsWidget.test.tsx | 11 +- .../DomainsWidget/DomainsWidget.test.tsx | 2 +- .../TotalDataAssetsWidget.test.tsx | 4 +- .../ui/src/components/NavBar/NavBar.test.tsx | 4 +- .../NotificationFeedCard.test.tsx | 4 +- .../FilterConfiguration.tsx | 16 +- .../SearchPreview/SearchPreview.test.tsx | 2 +- .../SearchSettings/TermBoost/TermBoost.tsx | 2 +- .../PlatformInsightsWidget.test.tsx | 6 +- .../MarketPlaceAppDetails.test.tsx | 2 +- .../ServiceConnectionDetails.component.tsx | 2 +- .../TeamsSelectable/TeamsSelectableNew.tsx | 6 +- .../Settings/Users/Users.component.test.tsx | 3 +- .../UserProfileRoles.component.tsx | 4 +- .../SettingsSso/SettingsSso.test.tsx | 22 +- .../TestDefinitionForm.test.tsx | 3 +- .../components/UploadFile/UploadFile.test.tsx | 2 +- .../common/AsyncSelect/AsyncSelect.test.tsx | 2 +- .../TableTypePropertyEditTable.test.tsx | 5 +- .../DomainDisplay/DomainDisplay.test.tsx | 6 +- .../common/DomainLabel/DomainLabel.test.tsx | 11 +- .../ErrorPlaceHolderES.test.tsx | 2 +- .../ErrorPlaceHolderIngestion.test.tsx | 4 +- .../CodeWidget/CodeWidget.test.tsx | 2 +- .../common/ListView/ListView.component.tsx | 2 +- .../NextPreviousWithOffset.test.tsx | 2 +- .../CommonEntitySummaryInfoV1.test.tsx | 7 +- .../QueryBuilderWidgetV1.test.tsx | 26 +- .../CustomHtmlRederer.interface.ts | 2 +- .../CustomHtmlRederer/CustomHtmlRederer.tsx | 2 +- .../SuccessScreen/SuccessScreen.test.tsx | 4 +- .../atoms/asyncTreeSelect/useTreeData.tsx | 46 +-- .../LineageProvider/LineageProvider.tsx | 96 +++--- .../useCurrentUserStore.test.ts | 13 +- .../resources/ui/src/hooks/useFqn.test.ts | 2 +- .../ui/src/hooks/useLineageStore.test.ts | 3 +- .../main/resources/ui/src/hooks/usePubSub.ts | 6 +- .../src/interface/data-insight.interface.ts | 7 +- .../src/pages/AppInstall/AppInstall.test.tsx | 9 +- .../ForgotPassword/ForgotPassword.test.tsx | 2 +- .../AddPolicyPage/AddPolicyPage.test.tsx | 2 +- .../ResetPassword/ResetPassword.test.tsx | 2 +- .../AddRolePage/AddRolePage.test.tsx | 2 +- .../ui/src/pages/SwaggerPage/RapiDocReact.tsx | 20 +- .../ui/src/pages/UserPage/UserPage.test.tsx | 2 +- .../ui/src/utils/BlockEditorUtils.ts | 88 +++--- .../ui/src/utils/CSV/CSVUtilsClassBase.tsx | 24 +- .../resources/ui/src/utils/CanvasUtils.ts | 18 +- .../ui/src/utils/ClassificationUtils.test.tsx | 2 +- .../ui/src/utils/CuratedAssetsUtils.test.tsx | 2 +- .../ui/src/utils/CuratedAssetsUtils.tsx | 2 +- .../CustomizableLandingPageUtils.test.tsx | 2 +- .../utils/DocumentationLinksClassBase.test.ts | 2 +- .../ui/src/utils/EntityLineageUtils.test.tsx | 59 ++-- .../ui/src/utils/EntityLineageUtils.tsx | 288 +++++++++--------- .../src/utils/EntitySummaryPanelUtilsV1.tsx | 4 +- .../ui/src/utils/EntityUtils.test.tsx | 4 +- .../ui/src/utils/ExtensionPointRegistry.ts | 2 +- .../utils/GlossaryTerm/GlossaryTermUtil.tsx | 22 +- .../resources/ui/src/utils/IngestionUtils.tsx | 2 +- .../ui/src/utils/OidcTokenStorage.test.ts | 2 +- .../ui/src/utils/QueryBuilderUtils.tsx | 4 +- .../resources/ui/src/utils/SchedularUtils.tsx | 38 +-- .../utils/ServiceConnectionDetailsUtils.tsx | 167 +++++----- 104 files changed, 799 insertions(+), 709 deletions(-) diff --git a/openmetadata-ui/src/main/resources/ui/src/components/ActivityFeed/FeedEditor/FeedEditor.test.tsx b/openmetadata-ui/src/main/resources/ui/src/components/ActivityFeed/FeedEditor/FeedEditor.test.tsx index 18083991d522..1fbdf2461706 100644 --- a/openmetadata-ui/src/main/resources/ui/src/components/ActivityFeed/FeedEditor/FeedEditor.test.tsx +++ b/openmetadata-ui/src/main/resources/ui/src/components/ActivityFeed/FeedEditor/FeedEditor.test.tsx @@ -19,47 +19,26 @@ import { FeedEditor } from './FeedEditor'; const onSave = jest.fn(); const onChangeHandler = jest.fn(); -const onKeyDownHandler = jest.fn().mockImplementation((e: KeyboardEvent) => { - if (e.key === 'Enter') { - if (e.nativeEvent.isComposing || e.keyCode === 229) { - return; - } - if (!e.shiftKey) { - onSave(); - } - } -}); - const mockFeedEditorProp = { onChangeHandler: onChangeHandler, onSave: onSave, }; -jest.mock('quilljs-markdown', () => { - class MockQuillMarkdown { - constructor() { - // eslint-disable-next-line no-console - console.log('Markdown constructor'); - } - } +jest.unmock('./FeedEditor'); - const instance = new MockQuillMarkdown(); +jest.mock('@windmillcode/quill-emoji', () => ({ + TextAreaEmoji: jest.fn(), +})); - return instance; -}); +jest.mock('quill', () => ({ + Parchment: {}, +})); + +jest.mock('quill-mention/autoregister', () => ({})); jest.mock('react-quill-new', () => { class MockQuill { - constructor() { - // eslint-disable-next-line no-console - console.log('Quill constructor'); - } - - register(val: string) { - // eslint-disable-next-line no-console - console.log(`Register ${val} module`); - } - + register() {} import(val: string) { return val; } @@ -68,21 +47,53 @@ jest.mock('react-quill-new', () => { return { __esModule: true, Quill: new MockQuill(), - default: jest.fn().mockImplementation(() => { - return ( -
- editor -
- ); - }), + default: jest.fn().mockImplementation( + (props: { + onKeyDown?: (e: KeyboardEvent) => void; + }) => { + return ( +
+ editor +
+ ); + } + ), }; }); -jest.mock('../../../utils/QuillLink/QuillLink', () => { - return jest.fn(); -}); +jest.mock('../../../utils/QuillLink/QuillLink', () => ({ + LinkBlot: jest.fn(), +})); + +jest.mock('../../../hooks/useApplicationStore', () => ({ + useApplicationStore: jest.fn().mockReturnValue({ userProfilePics: {} }), +})); + +jest.mock('../../../rest/userAPI', () => ({ + getUserByName: jest.fn(), +})); + +jest.mock('../../../utils/FeedUtils', () => ({ + HTMLToMarkdown: { turndown: jest.fn() }, + suggestions: jest.fn(), + userMentionItemWithAvatar: jest.fn(), +})); + +jest.mock('../../../utils/QuillUtils', () => ({ + insertMention: jest.fn(), + insertRef: jest.fn(), +})); + +jest.mock('../../../utils/sanitize.utils', () => ({ + getSanitizeContent: jest.fn().mockImplementation((v: string) => v), +})); + +jest.mock('../../../utils/SearchClassBase', () => ({ + __esModule: true, + default: { getEntityIcon: jest.fn() }, +})); -describe.skip('Test FeedEditor Component', () => { +describe('Test FeedEditor Component', () => { it('Should render FeedEditor Component', async () => { const { container } = render(, { wrapper: MemoryRouter, diff --git a/openmetadata-ui/src/main/resources/ui/src/components/ActivityFeed/FeedEditor/FeedEditor.tsx b/openmetadata-ui/src/main/resources/ui/src/components/ActivityFeed/FeedEditor/FeedEditor.tsx index 8bebde552234..f4b21841828a 100644 --- a/openmetadata-ui/src/main/resources/ui/src/components/ActivityFeed/FeedEditor/FeedEditor.tsx +++ b/openmetadata-ui/src/main/resources/ui/src/components/ActivityFeed/FeedEditor/FeedEditor.tsx @@ -224,9 +224,9 @@ export const FeedEditor = forwardRef( toggleMentionList(true); }, onSelect: ( - item: Record, + item: Record, - insertItem: (item: Record) => void + insertItem: (item: Record) => void ) => { toggleMentionList(true); insertItem(item); diff --git a/openmetadata-ui/src/main/resources/ui/src/components/AppBar/Suggestions.test.tsx b/openmetadata-ui/src/main/resources/ui/src/components/AppBar/Suggestions.test.tsx index a783bf0343ee..df9bbc1cabec 100644 --- a/openmetadata-ui/src/main/resources/ui/src/components/AppBar/Suggestions.test.tsx +++ b/openmetadata-ui/src/main/resources/ui/src/components/AppBar/Suggestions.test.tsx @@ -26,7 +26,7 @@ jest.mock('../../context/TourProvider/TourProvider'); jest.mock('../../utils/SearchUtils', () => ({ filterOptionsByIndex: jest.fn((options, index) => { return options.filter( - (option: any) => option._source?.entityType === index + (option: { _source?: { entityType?: string } }) => option._source?.entityType === index ); }), getGroupLabel: jest.fn((index) => `Group ${index}`), @@ -40,7 +40,7 @@ jest.mock('../../utils/SearchClassBase', () => ({ getEntitiesSuggestions: jest.fn(() => []), })); jest.mock('../../utils/CommonUtils', () => ({ - Transi18next: ({ i18nKey, values }: { i18nKey: string; values: any }) => ( + Transi18next: ({ i18nKey, values }: { i18nKey: string; values: Record }) => ( {i18nKey} {values?.keyword || ''} @@ -74,12 +74,12 @@ describe('Suggestions Component', () => { mockUseTranslation.mockReturnValue({ t: jest.fn((key: string) => key), i18n: { language: 'en' }, - } as any); + } as ReturnType); mockUseTourProvider.mockReturnValue({ isTourOpen: false, updateTourPage: jest.fn(), updateTourSearch: jest.fn(), - } as any); + } as ReturnType); }); describe('AI Query Suggestions', () => { @@ -147,7 +147,7 @@ describe('Suggestions Component', () => { isTourOpen: true, updateTourPage: jest.fn(), updateTourSearch: jest.fn(), - } as any); + } as ReturnType); render(); diff --git a/openmetadata-ui/src/main/resources/ui/src/components/AppRouter/SettingsRouter.test.tsx b/openmetadata-ui/src/main/resources/ui/src/components/AppRouter/SettingsRouter.test.tsx index 684bf596c618..811273f84870 100644 --- a/openmetadata-ui/src/main/resources/ui/src/components/AppRouter/SettingsRouter.test.tsx +++ b/openmetadata-ui/src/main/resources/ui/src/components/AppRouter/SettingsRouter.test.tsx @@ -180,7 +180,7 @@ jest.mock('./AdminProtectedRoute', () => ({ default: jest.fn().mockImplementation(({ children }) => children), })); -describe.skip('SettingsRouter', () => { +describe('SettingsRouter', () => { it('should render GlobalSettingPage component for exact settings route', async () => { render( @@ -326,7 +326,7 @@ describe.skip('SettingsRouter', () => { ).toBeInTheDocument(); }); - it.skip('should render CustomPageSettings component for custom page settings route', async () => { + it('should render CustomPageSettings component for custom page settings route', async () => { render( diff --git a/openmetadata-ui/src/main/resources/ui/src/components/Auth/AuthProviders/AuthProvider.tsx b/openmetadata-ui/src/main/resources/ui/src/components/Auth/AuthProviders/AuthProvider.tsx index e4e620c68a54..54c15d51b5ad 100644 --- a/openmetadata-ui/src/main/resources/ui/src/components/Auth/AuthProviders/AuthProvider.tsx +++ b/openmetadata-ui/src/main/resources/ui/src/components/Auth/AuthProviders/AuthProvider.tsx @@ -109,7 +109,11 @@ const isEmailVerifyField = 'isEmailVerified'; let requestInterceptor: number | null = null; let responseInterceptor: number | null = null; -let pendingRequests: any[] = []; +let pendingRequests: { + resolve: (value: unknown) => void; + reject: (reason?: unknown) => void; + config: InternalAxiosRequestConfig | undefined; +}[] = []; type AuthContextType = { onLoginHandler: () => void; @@ -461,7 +465,7 @@ export const AuthProvider = ({ } requestInterceptor = axiosClient.interceptors.request.use(async function ( - config: InternalAxiosRequestConfig + config: InternalAxiosRequestConfig ) { // Need to read token from local storage as it might have been updated with refresh const token: string = await getOidcToken(); diff --git a/openmetadata-ui/src/main/resources/ui/src/components/DataAssetSummaryPanelV1/DataAssetSummaryPanelV1.test.tsx b/openmetadata-ui/src/main/resources/ui/src/components/DataAssetSummaryPanelV1/DataAssetSummaryPanelV1.test.tsx index f4caff39344e..8e9855bbb745 100644 --- a/openmetadata-ui/src/main/resources/ui/src/components/DataAssetSummaryPanelV1/DataAssetSummaryPanelV1.test.tsx +++ b/openmetadata-ui/src/main/resources/ui/src/components/DataAssetSummaryPanelV1/DataAssetSummaryPanelV1.test.tsx @@ -206,7 +206,7 @@ jest.mock('../common/DescriptionSection/DescriptionSection', () => { jest.mock('../common/OverviewSection/OverviewSection', () => { return jest.fn().mockImplementation(({ entityInfoV1 }) => (
- {(entityInfoV1 || []).map((item: any, index: number) => ( + {(entityInfoV1 || []).map((item: { name: string; value: string }, index: number) => (
@@ -221,7 +221,7 @@ jest.mock('../common/DataQualitySection/DataQualitySection', () => { return jest.fn().mockImplementation(({ tests, totalTests }) => (
{totalTests}
- {tests.map((test: any, index: number) => ( + {tests.map((test: { type: string; count: number }, index: number) => (
{test.type}: {test.count}
@@ -382,7 +382,7 @@ describe('DataAssetSummaryPanelV1', () => { const mockOnDescriptionUpdate = jest.fn(); const defaultProps: DataAssetSummaryPanelProps = { - dataAsset: mockDataAsset as any, + dataAsset: mockDataAsset as DataAssetSummaryPanelProps['dataAsset'], entityType: EntityType.TABLE, isLoading: false, onOwnerUpdate: mockOnOwnerUpdate, @@ -415,7 +415,7 @@ describe('DataAssetSummaryPanelV1', () => { ); (listTestCases as jest.Mock).mockResolvedValue({ data: mockTestCaseData }); (getEntityOverview as jest.Mock).mockImplementation( - (_entityType: any, _dataAsset: any, additionalInfo: any) => [ + (_entityType: string, _dataAsset: DataAssetSummaryPanelProps['dataAsset'], additionalInfo?: Record) => [ { name: 'Type', value: 'Table', visible: ['explore'] }, { name: 'Rows', value: 1000, visible: ['explore'] }, { name: 'Columns', value: 15, visible: ['explore'] }, @@ -596,7 +596,7 @@ describe('DataAssetSummaryPanelV1', () => { dataAsset: { ...mockDataAsset, deleted: true, - } as any, + } as DataAssetSummaryPanelProps['dataAsset'], }; await act(async () => { @@ -904,7 +904,7 @@ describe('DataAssetSummaryPanelV1', () => { name: 'new-table', displayName: 'New Table', fullyQualifiedName: 'new.fqn', - } as any; + } as DataAssetSummaryPanelProps['dataAsset']; await act(async () => { rerender( diff --git a/openmetadata-ui/src/main/resources/ui/src/components/DataContract/ContractDetailFormTab/ContractDetailFormTab.test.tsx b/openmetadata-ui/src/main/resources/ui/src/components/DataContract/ContractDetailFormTab/ContractDetailFormTab.test.tsx index b16389e66a2b..f579d46e0cbc 100644 --- a/openmetadata-ui/src/main/resources/ui/src/components/DataContract/ContractDetailFormTab/ContractDetailFormTab.test.tsx +++ b/openmetadata-ui/src/main/resources/ui/src/components/DataContract/ContractDetailFormTab/ContractDetailFormTab.test.tsx @@ -18,7 +18,7 @@ import { ContractDetailFormTab } from './ContractDetailFormTab'; jest.mock('../../../utils/formUtils', () => ({ generateFormFields: jest.fn((fields) => - fields.map((field: any) => ( + fields.map((field: { name: string; label: string; props?: Record }) => (
diff --git a/openmetadata-ui/src/main/resources/ui/src/components/DataContract/ContractDetailTab/ContractDetail.test.tsx b/openmetadata-ui/src/main/resources/ui/src/components/DataContract/ContractDetailTab/ContractDetail.test.tsx index 18faeeace742..8cc302801dcb 100644 --- a/openmetadata-ui/src/main/resources/ui/src/components/DataContract/ContractDetailTab/ContractDetail.test.tsx +++ b/openmetadata-ui/src/main/resources/ui/src/components/DataContract/ContractDetailTab/ContractDetail.test.tsx @@ -75,13 +75,13 @@ jest.mock('../../common/OwnerLabel/OwnerLabel.component', () => ({ })); jest.mock('../../AlertBar/AlertBar', () => { - return function MockAlertBar({ message }: any) { + return function MockAlertBar({ message }: { message: string }) { return
{message}
; }; }); jest.mock('../../common/ErrorWithPlaceholder/ErrorPlaceHolder', () => { - return function MockErrorPlaceHolder({ type, children }: any) { + return function MockErrorPlaceHolder({ type, children }: { type: string; children: React.ReactNode }) { return (
{children} @@ -91,7 +91,7 @@ jest.mock('../../common/ErrorWithPlaceholder/ErrorPlaceHolder', () => { }); jest.mock('../ContractExecutionChart/ContractExecutionChart.component', () => { - return function MockContractExecutionChart({ contract }: any) { + return function MockContractExecutionChart({ contract }: { contract: DataContract | null }) { return (
Chart for {contract?.name} @@ -105,7 +105,7 @@ jest.mock('../ContractQualityCard/ContractQualityCard.component', () => { }); jest.mock('../ContractSecurity/ContractSecurityCard.component', () => { - return function MockContractSecurityCard({ security }: any) { + return function MockContractSecurityCard({ security }: { security?: { dataClassification?: string } }) { return (
ContractSecurityCard - {security?.dataClassification} @@ -115,7 +115,7 @@ jest.mock('../ContractSecurity/ContractSecurityCard.component', () => { }); jest.mock('../ContractViewSwitchTab/ContractViewSwitchTab.component', () => { - return function MockContractViewSwitchTab({ handleModeChange }: any) { + return function MockContractViewSwitchTab({ handleModeChange }: { handleModeChange: (e: { target: { value: string } }) => void }) { return (
-
- ))} + {dataSource?.map( + (item: { id: string; name: string; [key: string]: unknown }) => ( +
+ +
+ ) + )}
); }; @@ -91,7 +99,11 @@ jest.mock( drawerProps, onCancel, onFormSubmit, - }: { drawerProps: { open: boolean }; onCancel: () => void; onFormSubmit: () => void }) { + }: { + drawerProps: { open: boolean }; + onCancel: () => void; + onFormSubmit: () => void; + }) { if (!drawerProps.open) { return null; } diff --git a/openmetadata-ui/src/main/resources/ui/src/components/DataContract/ContractSLACard/ContractSLA.test.tsx b/openmetadata-ui/src/main/resources/ui/src/components/DataContract/ContractSLACard/ContractSLA.test.tsx index 740e01c3a96b..ef90fd881668 100644 --- a/openmetadata-ui/src/main/resources/ui/src/components/DataContract/ContractSLACard/ContractSLA.test.tsx +++ b/openmetadata-ui/src/main/resources/ui/src/components/DataContract/ContractSLACard/ContractSLA.test.tsx @@ -23,7 +23,13 @@ import { mockTableData } from '../../../mocks/TableVersion.mock'; import ContractSLA from './ContractSLA.component'; jest.mock('../../../utils/CommonUtils', () => ({ - Transi18next: ({ i18nKey, values }: { i18nKey: string; values?: { label?: string; data?: string } }) => ( + Transi18next: ({ + i18nKey, + values, + }: { + i18nKey: string; + values?: { label?: string; data?: string }; + }) => ( {i18nKey} - {values?.label}: {values?.data} diff --git a/openmetadata-ui/src/main/resources/ui/src/components/DataContract/ContractSchemaFormTab/ContractSchemaFormTab.test.tsx b/openmetadata-ui/src/main/resources/ui/src/components/DataContract/ContractSchemaFormTab/ContractSchemaFormTab.test.tsx index c67c2dad2eb9..24678398ea78 100644 --- a/openmetadata-ui/src/main/resources/ui/src/components/DataContract/ContractSchemaFormTab/ContractSchemaFormTab.test.tsx +++ b/openmetadata-ui/src/main/resources/ui/src/components/DataContract/ContractSchemaFormTab/ContractSchemaFormTab.test.tsx @@ -85,18 +85,24 @@ jest.mock('../../common/Table/Table', () => {
Data Source Length: {dataSource?.length || 0}
Columns: {columns?.length || 0}
Pagination: {pagination ? 'enabled' : 'disabled'}
- {dataSource?.map((item: { name: string; fullyQualifiedName: string; [key: string]: unknown }) => ( -
- {item.name} - -
- ))} + {dataSource?.map( + (item: { + name: string; + fullyQualifiedName: string; + [key: string]: unknown; + }) => ( +
+ {item.name} + +
+ ) + )}
); }; diff --git a/openmetadata-ui/src/main/resources/ui/src/components/DataContract/ContractSemanticFormTab/ContractSemanticFormTab.test.tsx b/openmetadata-ui/src/main/resources/ui/src/components/DataContract/ContractSemanticFormTab/ContractSemanticFormTab.test.tsx index d53a57954858..efb9cb674ddb 100644 --- a/openmetadata-ui/src/main/resources/ui/src/components/DataContract/ContractSemanticFormTab/ContractSemanticFormTab.test.tsx +++ b/openmetadata-ui/src/main/resources/ui/src/components/DataContract/ContractSemanticFormTab/ContractSemanticFormTab.test.tsx @@ -37,7 +37,12 @@ jest.mock('../../common/QueryBuilderWidgetV1/QueryBuilderWidgetV1', () => { getQueryActions, value, readonly, - }: { onChange?: (query: string, tree: JsonTree) => void; getQueryActions?: (actions: Actions) => void; value?: string; readonly?: boolean }) { + }: { + onChange?: (query: string, tree: JsonTree) => void; + getQueryActions?: (actions: Actions) => void; + value?: string; + readonly?: boolean; + }) { return (
Query Builder Widget diff --git a/openmetadata-ui/src/main/resources/ui/src/components/DataContract/ContractTab/ContractTab.test.tsx b/openmetadata-ui/src/main/resources/ui/src/components/DataContract/ContractTab/ContractTab.test.tsx index 6b90bee24a37..8953136c9b55 100644 --- a/openmetadata-ui/src/main/resources/ui/src/components/DataContract/ContractTab/ContractTab.test.tsx +++ b/openmetadata-ui/src/main/resources/ui/src/components/DataContract/ContractTab/ContractTab.test.tsx @@ -75,7 +75,13 @@ jest.mock('../../common/DeleteWidget/DeleteWidgetModal', () => { onDelete, entityName, entityType, - }: { visible: boolean; onCancel: () => void; onDelete: () => void; entityName: string; entityType: string }) { + }: { + visible: boolean; + onCancel: () => void; + onDelete: () => void; + entityName: string; + entityType: string; + }) { if (!visible) { return null; } @@ -97,7 +103,15 @@ jest.mock('../../common/DeleteWidget/DeleteWidgetModal', () => { }); jest.mock('../AddDataContract/AddDataContract', () => { - return function MockAddDataContract({ onCancel, onSave, contract }: { onCancel: () => void; onSave: () => void; contract?: DataContract | null }) { + return function MockAddDataContract({ + onCancel, + onSave, + contract, + }: { + onCancel: () => void; + onSave: () => void; + contract?: DataContract | null; + }) { return (
Contract: {contract?.name || 'New Contract'}
diff --git a/openmetadata-ui/src/main/resources/ui/src/components/DataContract/ContractYaml/ContractYaml.test.tsx b/openmetadata-ui/src/main/resources/ui/src/components/DataContract/ContractYaml/ContractYaml.test.tsx index 5f7dc0fe9367..a340a48ec2a2 100644 --- a/openmetadata-ui/src/main/resources/ui/src/components/DataContract/ContractYaml/ContractYaml.test.tsx +++ b/openmetadata-ui/src/main/resources/ui/src/components/DataContract/ContractYaml/ContractYaml.test.tsx @@ -37,7 +37,12 @@ jest.mock('../../Database/SchemaEditor/SchemaEditor', () => { editorClass, mode, value, - }: { className?: string; editorClass?: string; mode?: { name: string }; value?: string }) { + }: { + className?: string; + editorClass?: string; + mode?: { name: string }; + value?: string; + }) { return (
{editorClass}
diff --git a/openmetadata-ui/src/main/resources/ui/src/components/DriveService/Directory/DirectoryVersion/DirectoryVersion.test.tsx b/openmetadata-ui/src/main/resources/ui/src/components/DriveService/Directory/DirectoryVersion/DirectoryVersion.test.tsx index eabfe5a9c8e8..7bff11b5bf1f 100644 --- a/openmetadata-ui/src/main/resources/ui/src/components/DriveService/Directory/DirectoryVersion/DirectoryVersion.test.tsx +++ b/openmetadata-ui/src/main/resources/ui/src/components/DriveService/Directory/DirectoryVersion/DirectoryVersion.test.tsx @@ -481,7 +481,9 @@ describe('DirectoryVersion', () => { const permissionsWithUndefinedViewCustomFields = { ...ENTITY_PERMISSIONS, }; - delete (permissionsWithUndefinedViewCustomFields as Record).ViewCustomFields; + delete ( + permissionsWithUndefinedViewCustomFields as Record + ).ViewCustomFields; renderDirectoryVersion({ entityPermissions: permissionsWithUndefinedViewCustomFields, diff --git a/openmetadata-ui/src/main/resources/ui/src/components/DriveService/File/FileVersion/FileVersion.test.tsx b/openmetadata-ui/src/main/resources/ui/src/components/DriveService/File/FileVersion/FileVersion.test.tsx index 251d92863222..fcb307643243 100644 --- a/openmetadata-ui/src/main/resources/ui/src/components/DriveService/File/FileVersion/FileVersion.test.tsx +++ b/openmetadata-ui/src/main/resources/ui/src/components/DriveService/File/FileVersion/FileVersion.test.tsx @@ -460,7 +460,9 @@ describe('FileVersion', () => { const permissionsWithUndefinedViewCustomFields = { ...ENTITY_PERMISSIONS, }; - delete (permissionsWithUndefinedViewCustomFields as Record).ViewCustomFields; + delete ( + permissionsWithUndefinedViewCustomFields as Record + ).ViewCustomFields; renderFileVersion({ entityPermissions: permissionsWithUndefinedViewCustomFields, diff --git a/openmetadata-ui/src/main/resources/ui/src/components/DriveService/Spreadsheet/SpreadsheetVersion/SpreadsheetVersion.test.tsx b/openmetadata-ui/src/main/resources/ui/src/components/DriveService/Spreadsheet/SpreadsheetVersion/SpreadsheetVersion.test.tsx index 9efe604fcb83..0f5ae62e1202 100644 --- a/openmetadata-ui/src/main/resources/ui/src/components/DriveService/Spreadsheet/SpreadsheetVersion/SpreadsheetVersion.test.tsx +++ b/openmetadata-ui/src/main/resources/ui/src/components/DriveService/Spreadsheet/SpreadsheetVersion/SpreadsheetVersion.test.tsx @@ -648,7 +648,9 @@ describe('SpreadsheetVersion', () => { const permissionsWithUndefinedViewCustomFields = { ...ENTITY_PERMISSIONS, }; - delete (permissionsWithUndefinedViewCustomFields as Record).ViewCustomFields; + delete ( + permissionsWithUndefinedViewCustomFields as Record + ).ViewCustomFields; renderSpreadsheetVersion({ entityPermissions: permissionsWithUndefinedViewCustomFields, diff --git a/openmetadata-ui/src/main/resources/ui/src/components/DriveService/Worksheet/WorksheetVersion/WorksheetVersion.test.tsx b/openmetadata-ui/src/main/resources/ui/src/components/DriveService/Worksheet/WorksheetVersion/WorksheetVersion.test.tsx index e5b41bb808cc..a242f7d2302e 100644 --- a/openmetadata-ui/src/main/resources/ui/src/components/DriveService/Worksheet/WorksheetVersion/WorksheetVersion.test.tsx +++ b/openmetadata-ui/src/main/resources/ui/src/components/DriveService/Worksheet/WorksheetVersion/WorksheetVersion.test.tsx @@ -650,7 +650,9 @@ describe('WorksheetVersion', () => { const permissionsWithUndefinedViewCustomFields = { ...ENTITY_PERMISSIONS, }; - delete (permissionsWithUndefinedViewCustomFields as Record).ViewCustomFields; + delete ( + permissionsWithUndefinedViewCustomFields as Record + ).ViewCustomFields; renderWorksheetVersion({ entityPermissions: permissionsWithUndefinedViewCustomFields, diff --git a/openmetadata-ui/src/main/resources/ui/src/components/Entity/EntityLineage/NodeSuggestions.component.tsx b/openmetadata-ui/src/main/resources/ui/src/components/Entity/EntityLineage/NodeSuggestions.component.tsx index c298134b6758..d945cc2d1880 100644 --- a/openmetadata-ui/src/main/resources/ui/src/components/Entity/EntityLineage/NodeSuggestions.component.tsx +++ b/openmetadata-ui/src/main/resources/ui/src/components/Entity/EntityLineage/NodeSuggestions.component.tsx @@ -11,8 +11,8 @@ * limitations under the License. */ -import { Button, Col, Row, Select } from 'antd'; import type { RefSelectProps } from 'antd'; +import { Button, Col, Row, Select } from 'antd'; import { AxiosError } from 'axios'; import { capitalize, debounce, get } from 'lodash'; import { diff --git a/openmetadata-ui/src/main/resources/ui/src/components/Lineage/Lineage.test.tsx b/openmetadata-ui/src/main/resources/ui/src/components/Lineage/Lineage.test.tsx index c977e40ca357..85a125cac927 100644 --- a/openmetadata-ui/src/main/resources/ui/src/components/Lineage/Lineage.test.tsx +++ b/openmetadata-ui/src/main/resources/ui/src/components/Lineage/Lineage.test.tsx @@ -423,7 +423,10 @@ describe('Lineage Component', () => { fireEvent(reactFlow, dragOverEvent); expect(preventDefaultSpy).toHaveBeenCalled(); - expect((dragOverEvent as unknown as { dataTransfer: { dropEffect: string } }).dataTransfer.dropEffect).toBe('move'); + expect( + (dragOverEvent as unknown as { dataTransfer: { dropEffect: string } }) + .dataTransfer.dropEffect + ).toBe('move'); }); it('should handle node drop event', () => { diff --git a/openmetadata-ui/src/main/resources/ui/src/components/LineageTable/useLineagetTableState.test.ts b/openmetadata-ui/src/main/resources/ui/src/components/LineageTable/useLineagetTableState.test.ts index 200a18ff6d3e..cd7e3c9290dd 100644 --- a/openmetadata-ui/src/main/resources/ui/src/components/LineageTable/useLineagetTableState.test.ts +++ b/openmetadata-ui/src/main/resources/ui/src/components/LineageTable/useLineagetTableState.test.ts @@ -132,13 +132,17 @@ describe('useLineageTableState Hook', () => { const downstreamNodes = [{ id: 'downstream1' }]; act(() => { - result.current.setUpstreamColumnLineageNodes(upstreamNodes as unknown as LineageNode[]); + result.current.setUpstreamColumnLineageNodes( + upstreamNodes as unknown as LineageNode[] + ); }); expect(result.current.upstreamColumnLineageNodes).toEqual(upstreamNodes); act(() => { - result.current.setDownstreamColumnLineageNodes(downstreamNodes as unknown as LineageNode[]); + result.current.setDownstreamColumnLineageNodes( + downstreamNodes as unknown as LineageNode[] + ); }); expect(result.current.downstreamColumnLineageNodes).toEqual( @@ -210,7 +214,9 @@ describe('useLineageTableState Hook', () => { // Set some values first act(() => { result.current.setSearchValue('test'); - result.current.setFilterNodes([{ id: 'test' }] as unknown as LineageNode[]); + result.current.setFilterNodes([ + { id: 'test' }, + ] as unknown as LineageNode[]); result.current.setFilterSelectionActive(true); }); diff --git a/openmetadata-ui/src/main/resources/ui/src/components/MyData/FeedWidget/FeedWidget.test.tsx b/openmetadata-ui/src/main/resources/ui/src/components/MyData/FeedWidget/FeedWidget.test.tsx index 23425edbb9e9..da2775fc98a4 100644 --- a/openmetadata-ui/src/main/resources/ui/src/components/MyData/FeedWidget/FeedWidget.test.tsx +++ b/openmetadata-ui/src/main/resources/ui/src/components/MyData/FeedWidget/FeedWidget.test.tsx @@ -80,7 +80,13 @@ jest.mock('../../../hooks/useApplicationStore', () => ({ jest.mock( '../Widgets/Common/WidgetWrapper/WidgetWrapper', () => - ({ children, header }: { children: React.ReactNode; header: React.ReactNode }) => + ({ + children, + header, + }: { + children: React.ReactNode; + header: React.ReactNode; + }) => (
{header} @@ -92,7 +98,15 @@ jest.mock( jest.mock( '../Widgets/Common/WidgetHeader/WidgetHeader', () => - ({ onSortChange, handleRemoveWidget, widgetKey }: { onSortChange: (value: string) => void; handleRemoveWidget: (key: string) => void; widgetKey: string }) => + ({ + onSortChange, + handleRemoveWidget, + widgetKey, + }: { + onSortChange: (value: string) => void; + handleRemoveWidget: (key: string) => void; + widgetKey: string; + }) => (