Skip to content

Commit 94db4d3

Browse files
committed
Fix tests
1 parent f22af22 commit 94db4d3

24 files changed

+70
-62
lines changed

web_ui/src/features/dataset-export/components/dataset-statistics.test.tsx

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -8,15 +8,15 @@ import { useDatasetIdentifier } from '../../../pages/annotator/hooks/use-dataset
88
import { useMedia } from '../../../pages/media/providers/media-provider.component';
99
import { ExportDatasetStatistics } from './export-dataset-dialog.component';
1010

11-
jest.mock('../../../../media/providers/media-provider.component', () => ({
11+
jest.mock('../../../pages/media/providers/media-provider.component', () => ({
1212
useMedia: jest.fn(),
1313
}));
1414

15-
jest.mock('../../../../annotator/hooks/use-dataset-identifier.hook', () => ({
15+
jest.mock('../../../pages/annotator/hooks/use-dataset-identifier.hook', () => ({
1616
useDatasetIdentifier: jest.fn(),
1717
}));
1818

19-
jest.mock('../../../../../core/statistics/hooks/use-dataset-statistics.hook', () => ({
19+
jest.mock('../../../core/statistics/hooks/use-dataset-statistics.hook', () => ({
2020
useDatasetStatistics: jest.fn(),
2121
}));
2222

web_ui/src/features/dataset-export/components/export-dataset-dialog.component.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -29,9 +29,9 @@ import { ExportFormats } from '../../../core/projects/dataset.interface';
2929
import { useDatasetStatistics } from '../../../core/statistics/hooks/use-dataset-statistics.hook';
3030
import { useDatasetIdentifier } from '../../../pages/annotator/hooks/use-dataset-identifier.hook';
3131
import { useMedia } from '../../../pages/media/providers/media-provider.component';
32-
import { useExportDataset } from '../../../pages/project-details/hooks/use-export-dataset.hook';
3332
import { useProject } from '../../../pages/project-details/providers/project-provider/project-provider.component';
3433
import { isNotCropDomain, openNewTab, pluralize } from '../../../shared/utils';
34+
import { useExportDataset } from '../hooks/use-export-dataset.hook';
3535
import { AVAILABLE_FORMATS, ExportFormatDetails, isDatumaroFormat } from '../utils/utils';
3636
import { ExportDatasetMessage } from './export-dataset-message.component';
3737

web_ui/src/features/dataset-export/components/export-dataset-dialog.test.tsx

Lines changed: 11 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -15,27 +15,27 @@ import { getMockedProject } from '../../../test-utils/mocked-items-factory/mocke
1515
import { RequiredProviders } from '../../../test-utils/required-providers-render';
1616
import { ExportDatasetDialog } from './export-dataset-dialog.component';
1717

18-
jest.mock('../../../providers/project-provider/project-provider.component', () => ({
19-
...jest.requireActual('../../../providers/project-provider/project-provider.component'),
18+
jest.mock('../../../pages/project-details/providers/project-provider/project-provider.component', () => ({
19+
...jest.requireActual('../../../pages/project-details/providers/project-provider/project-provider.component'),
2020
useProject: jest.fn(),
2121
}));
2222

2323
const mockDatasetId = '321';
2424
const mockWorkspaceId = '123';
25-
jest.mock('../../../../annotator/hooks/use-dataset-identifier.hook', () => ({
26-
...jest.requireActual('../../../../annotator/hooks/use-dataset-identifier.hook'),
25+
jest.mock('../../../pages/annotator/hooks/use-dataset-identifier.hook', () => ({
26+
...jest.requireActual('../../../pages/annotator/hooks/use-dataset-identifier.hook'),
2727
useDatasetIdentifier: () => ({ workspaceId: mockWorkspaceId, datasetId: mockDatasetId }),
2828
}));
2929

3030
const mockAddNotification = jest.fn();
31-
jest.mock('../../../../../notification/notification.component', () => ({
32-
...jest.requireActual('../../../../../notification/notification.component'),
31+
jest.mock('../../../notification/notification.component', () => ({
32+
...jest.requireActual('../../../notification/notification.component'),
3333
useNotification: () => ({ addNotification: mockAddNotification }),
3434
}));
3535

3636
const mockPrepareExportDataset = jest.fn();
37-
jest.mock('../../../hooks/use-export-dataset.hook', () => ({
38-
...jest.requireActual('../../../hooks/use-export-dataset.hook'),
37+
jest.mock('../hooks/use-export-dataset.hook', () => ({
38+
...jest.requireActual('../hooks/use-export-dataset.hook'),
3939
useExportDataset: () => ({
4040
prepareExportDatasetJob: {
4141
mutate: mockPrepareExportDataset,
@@ -61,7 +61,9 @@ describe('ExportDatasetDialog', () => {
6161
mediaService?: MediaService;
6262
}): Promise<RenderResult> => {
6363
jest.mocked(useProject).mockReturnValue({
64-
...jest.requireActual('../../../providers/project-provider/project-provider.component'),
64+
...jest.requireActual(
65+
'../../../pages/project-details/providers/project-provider/project-provider.component'
66+
),
6567
project: getMockedProject({ domains }),
6668
isTaskChainProject,
6769
});

web_ui/src/features/dataset-export/components/export-dataset-download.test.tsx

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -9,12 +9,12 @@ import { downloadFile, getDownloadNotificationMessage } from '../../../shared/ut
99
import { providersRender as render } from '../../../test-utils/required-providers-render';
1010
import { ExportDatasetDownload } from './export-dataset-download.component';
1111

12-
jest.mock('../../../hooks/use-export-dataset.hook', () => ({
13-
...jest.requireActual('../../../hooks/use-export-dataset.hook'),
12+
jest.mock('../hooks/use-export-dataset.hook', () => ({
13+
...jest.requireActual('../hooks/use-export-dataset.hook'),
1414
useExportDataset: jest.fn(() => ({ exportDatasetStatus: {} })),
1515
}));
16-
jest.mock('../../../../../shared/utils', () => ({
17-
...jest.requireActual('../../../../../shared/utils'),
16+
jest.mock('../../../shared/utils', () => ({
17+
...jest.requireActual('../../../shared/utils'),
1818
downloadFile: jest.fn(),
1919
}));
2020

web_ui/src/features/dataset-export/components/export-dataset-notification.component.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@ import { View } from '@geti/ui';
77
import { OverlayTriggerState, useOverlayTriggerState } from '@react-stately/overlays';
88

99
import { DatasetIdentifier, ExportDatasetLSData } from '../../../core/projects/dataset.interface';
10-
import { useLocalStorageExportDataset } from '../../../pages/project-details/hooks/use-local-storage-export-dataset.hook';
10+
import { useLocalStorageExportDataset } from '../hooks/use-local-storage-export-dataset.hook';
1111
import { ExportDatasetDownload } from './export-dataset-download.component';
1212
import { ExportDatasetStatusJob } from './export-dataset-status-job.component';
1313

web_ui/src/features/dataset-export/components/export-dataset-notification.test.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -30,7 +30,7 @@ const mockExportDatasetLSData = {
3030
const mockUpdateLsExportDataset = jest.fn();
3131
const mockRemoveDatasetLsByDatasetId = jest.fn();
3232
const mockGetDatasetLsByDatasetId = jest.fn();
33-
jest.mock('../../../hooks/use-local-storage-export-dataset.hook', () => ({
33+
jest.mock('../hooks/use-local-storage-export-dataset.hook', () => ({
3434
useLocalStorageExportDataset: () => ({
3535
getDatasetLsByDatasetId: mockGetDatasetLsByDatasetId,
3636
updateLsExportDataset: mockUpdateLsExportDataset,

web_ui/src/features/dataset-export/components/export-dataset-status-job.component.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,10 +9,10 @@ import { getJobActiveStep } from '../../../core/jobs/utils';
99
import { DatasetIdentifier, ExportDatasetLSData, ExportFormats } from '../../../core/projects/dataset.interface';
1010
import { NOTIFICATION_TYPE } from '../../../notification/notification-toast/notification-type.enum';
1111
import { useNotification } from '../../../notification/notification.component';
12-
import { useExportDataset } from '../../../pages/project-details/hooks/use-export-dataset.hook';
1312
import { JobProgress } from '../../../shared/components/header/jobs-management/job-progress.component';
1413
import { ThinProgressBar } from '../../../shared/components/thin-progress-bar/thin-progress-bar.component';
1514
import { formatDownloadUrl } from '../../../shared/utils';
15+
import { useExportDataset } from '../hooks/use-export-dataset.hook';
1616

1717
import classes from './dataset-export.module.scss';
1818

web_ui/src/features/dataset-export/components/export-import-dataset-buttons.test.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@ import { ThemeProvider } from '../../../theme/theme-provider.component';
99
import { ExportImportDatasetButtons } from './export-import-dataset-buttons.component';
1010
import { ExportImportDatasetDialogProvider } from './export-import-dataset-dialog-provider.component';
1111

12-
jest.mock('../../../providers/project-provider/project-provider.component', () => ({
12+
jest.mock('../../../pages/project-details/providers/project-provider/project-provider.component', () => ({
1313
useProject: jest.fn(),
1414
}));
1515

web_ui/src/features/dataset-import/components/dataset-import-panel/dataset-import-panel.component-menu.test.tsx

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -37,8 +37,8 @@ const mockDatasetImportItem: DatasetImportItem = {
3737
warnings: [],
3838
};
3939

40-
jest.mock('../../../notification/notification.component', () => ({
41-
...jest.requireActual('../../../notification/notification.component'),
40+
jest.mock('../../../../notification/notification.component', () => ({
41+
...jest.requireActual('../../../../notification/notification.component'),
4242
useNotification: () => ({ addNotification: jest.fn() }),
4343
}));
4444

@@ -48,8 +48,8 @@ jest.mock('react-router-dom', () => ({
4848
}));
4949

5050
const mockAbortActiveUpload = jest.fn();
51-
jest.mock('../../../providers/tus-upload-provider/tus-upload-provider.component', () => ({
52-
...jest.requireActual('../../../providers/tus-upload-provider/tus-upload-provider.component'),
51+
jest.mock('../../../../providers/tus-upload-provider/tus-upload-provider.component', () => ({
52+
...jest.requireActual('../../../../providers/tus-upload-provider/tus-upload-provider.component'),
5353
useTusUpload: () => ({ abortActiveUpload: mockAbortActiveUpload }),
5454
}));
5555

web_ui/src/features/dataset-import/components/dataset-import-panel/dataset-import-panel.component.test.tsx

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -61,8 +61,8 @@ const mockDatasetImportItem: DatasetImportItem = {
6161
};
6262

6363
const mockedAddNotification = jest.fn();
64-
jest.mock('../../../notification/notification.component', () => ({
65-
...jest.requireActual('../../../notification/notification.component'),
64+
jest.mock('../../../../notification/notification.component', () => ({
65+
...jest.requireActual('../../../../notification/notification.component'),
6666
useNotification: () => ({ addNotification: mockedAddNotification }),
6767
}));
6868

@@ -72,8 +72,8 @@ jest.mock('react-router-dom', () => ({
7272
}));
7373

7474
const mockAbortActiveUpload = jest.fn();
75-
jest.mock('../../../providers/tus-upload-provider/tus-upload-provider.component', () => ({
76-
...jest.requireActual('../../../providers/tus-upload-provider/tus-upload-provider.component'),
75+
jest.mock('../../../../providers/tus-upload-provider/tus-upload-provider.component', () => ({
76+
...jest.requireActual('../../../../providers/tus-upload-provider/tus-upload-provider.component'),
7777
useTusUpload: () => ({ abortActiveUpload: mockAbortActiveUpload }),
7878
}));
7979

web_ui/src/features/dataset-import/components/dataset-import-to-existing-project/dataset-import-to-existing-project-dialog-buttons.test.tsx

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -53,16 +53,16 @@ const mockPatchActiveDatasetImport = jest.fn();
5353
let mockIsReadyValue = false;
5454

5555
const mockAbortActiveUpload = jest.fn();
56-
jest.mock('../../../../../providers/tus-upload-provider/tus-upload-provider.component', () => ({
57-
...jest.requireActual('../../../../../providers/tus-upload-provider/tus-upload-provider.component'),
56+
jest.mock('../../../../providers/tus-upload-provider/tus-upload-provider.component', () => ({
57+
...jest.requireActual('../../../../providers/tus-upload-provider/tus-upload-provider.component'),
5858
useTusUpload: () => ({ abortActiveUpload: mockAbortActiveUpload }),
5959
}));
6060

6161
jest.mock(
62-
'../../../../../providers/dataset-import-to-existing-project-provider/dataset-import-to-existing-project-provider.component',
62+
'../../../../providers/dataset-import-to-existing-project-provider/dataset-import-to-existing-project-provider.component',
6363
() => ({
6464
...jest.requireActual(
65-
'../../../../../providers/dataset-import-to-existing-project-provider/dataset-import-to-existing-project-provider.component'
65+
'../../../../providers/dataset-import-to-existing-project-provider/dataset-import-to-existing-project-provider.component'
6666
),
6767
useDatasetImportToExistingProject: jest.fn(() => ({
6868
abortDatasetImportAction: mockAbortDatasetImportAction,

web_ui/src/features/dataset-import/components/dataset-import-to-existing-project/dataset-import-to-existing-project-dialog.test.tsx

Lines changed: 11 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -50,13 +50,13 @@ const mockDatasetImportDeleteDialogState = {
5050
close: jest.fn(),
5151
};
5252

53-
jest.mock('../../../../../notification/notification.component', () => ({
54-
...jest.requireActual('../../../../../notification/notification.component'),
53+
jest.mock('../../../../notification/notification.component', () => ({
54+
...jest.requireActual('../../../../notification/notification.component'),
5555
useNotification: () => ({ addNotification: jest.fn() }),
5656
}));
5757

58-
jest.mock('../../../../project-details/providers/project-provider/project-provider.component', () => ({
59-
...jest.requireActual('../../../../project-details/providers/project-provider/project-provider.component'),
58+
jest.mock('../../../../pages/project-details/providers/project-provider/project-provider.component', () => ({
59+
...jest.requireActual('../../../../pages/project-details/providers/project-provider/project-provider.component'),
6060
useProject: jest.fn(() => ({
6161
project: {
6262
domains: [],
@@ -67,10 +67,10 @@ jest.mock('../../../../project-details/providers/project-provider/project-provid
6767
}));
6868

6969
jest.mock(
70-
'../../../../../providers/dataset-import-to-existing-project-provider/dataset-import-to-existing-project-provider.component',
70+
'../../../../providers/dataset-import-to-existing-project-provider/dataset-import-to-existing-project-provider.component',
7171
() => ({
7272
...jest.requireActual(
73-
'../../../../../providers/dataset-import-to-existing-project-provider/dataset-import-to-existing-project-provider.component'
73+
'../../../../providers/dataset-import-to-existing-project-provider/dataset-import-to-existing-project-provider.component'
7474
),
7575
useDatasetImportToExistingProject: jest.fn(() => ({
7676
isReady: () => jest.fn(),
@@ -97,7 +97,7 @@ describe(DatasetImportToExistingProjectDialog, () => {
9797
const mockedSetActiveDatasetImportId = jest.fn();
9898
jest.mocked(useDatasetImportToExistingProject).mockReturnValue({
9999
...jest.requireActual(
100-
'../../../../../providers/dataset-import-to-existing-project-provider/dataset-import-to-existing-project-provider.component'
100+
'../../../../providers/dataset-import-to-existing-project-provider/dataset-import-to-existing-project-provider.component'
101101
),
102102
isReady: jest.fn(),
103103
setActiveDatasetImportId: mockedSetActiveDatasetImportId,
@@ -127,7 +127,7 @@ describe(DatasetImportToExistingProjectDialog, () => {
127127
async ({ status }) => {
128128
jest.mocked(useDatasetImportToExistingProject).mockReturnValue({
129129
...jest.requireActual(
130-
'../../../../../providers/dataset-import-to-existing-project-provider/dataset-import-to-existing-project-provider.component'
130+
'../../../../providers/dataset-import-to-existing-project-provider/dataset-import-to-existing-project-provider.component'
131131
),
132132
isReady: jest.fn(),
133133
activeDatasetImport: { ...mockDatasetImportItem, status },
@@ -148,7 +148,7 @@ describe(DatasetImportToExistingProjectDialog, () => {
148148
async ({ status }) => {
149149
jest.mocked(useDatasetImportToExistingProject).mockReturnValue({
150150
...jest.requireActual(
151-
'../../../../../providers/dataset-import-to-existing-project-provider/dataset-import-to-existing-project-provider.component'
151+
'../../../../providers/dataset-import-to-existing-project-provider/dataset-import-to-existing-project-provider.component'
152152
),
153153
isReady: jest.fn(),
154154
activeDatasetImport: { ...mockDatasetImportItem, labels: ['a', 'z'], status },
@@ -168,7 +168,7 @@ describe(DatasetImportToExistingProjectDialog, () => {
168168
it('Display proper information when there is no label to map in dataset importing', async () => {
169169
jest.mocked(useDatasetImportToExistingProject).mockReturnValue({
170170
...jest.requireActual(
171-
'../../../../../providers/dataset-import-to-existing-project-provider/dataset-import-to-existing-project-provider.component'
171+
'../../../../providers/dataset-import-to-existing-project-provider/dataset-import-to-existing-project-provider.component'
172172
),
173173
isReady: jest.fn(),
174174
activeDatasetImport: {
@@ -187,7 +187,7 @@ describe(DatasetImportToExistingProjectDialog, () => {
187187
const mockedImportDatasetJob = jest.fn();
188188
jest.mocked(useDatasetImportToExistingProject).mockReturnValue({
189189
...jest.requireActual(
190-
'../../../../../providers/dataset-import-to-existing-project-provider/dataset-import-to-existing-project-provider.component'
190+
'../../../../providers/dataset-import-to-existing-project-provider/dataset-import-to-existing-project-provider.component'
191191
),
192192
isReady: jest.fn(() => true),
193193
importDataset: jest.fn(),

web_ui/src/features/dataset-import/components/dataset-import-to-existing-project/dataset-import-to-existing-project-map-labels.test.tsx

Lines changed: 9 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,7 @@
11
// Copyright (C) 2022-2025 Intel Corporation
22
// LIMITED EDGE SOFTWARE DISTRIBUTION LICENSE
33

4+
import { defaultTheme, Provider } from '@geti/ui';
45
import { fireEvent, render, screen } from '@testing-library/react';
56
import { userEvent } from '@testing-library/user-event';
67

@@ -37,10 +38,10 @@ const mockProjectLabels: Label[] = [
3738

3839
const mockPatchActiveDatasetImport = jest.fn();
3940
jest.mock(
40-
'../../../../../providers/dataset-import-to-existing-project-provider/dataset-import-to-existing-project-provider.component',
41+
'../../../../providers/dataset-import-to-existing-project-provider/dataset-import-to-existing-project-provider.component',
4142
() => ({
4243
...jest.requireActual(
43-
'../../../../../providers/dataset-import-to-existing-project-provider/dataset-import-to-existing-project-provider.component'
44+
'../../../../providers/dataset-import-to-existing-project-provider/dataset-import-to-existing-project-provider.component'
4445
),
4546
useDatasetImportToExistingProject: jest.fn(() => ({ patchActiveDatasetImport: mockPatchActiveDatasetImport })),
4647
})
@@ -73,10 +74,12 @@ describe(DatasetImportToExistingProjectMapLabels, () => {
7374

7475
it('should properly interact on labels mapping change', async () => {
7576
render(
76-
<DatasetImportToExistingProjectMapLabels
77-
projectLabels={mockProjectLabels}
78-
activeDatasetImport={{ ...mockDatasetImportItem, labels: ['cat'] }}
79-
/>
77+
<Provider theme={defaultTheme}>
78+
<DatasetImportToExistingProjectMapLabels
79+
projectLabels={mockProjectLabels}
80+
activeDatasetImport={{ ...mockDatasetImportItem, labels: ['cat'] }}
81+
/>
82+
</Provider>
8083
);
8184

8285
expect(screen.getByRole('textbox', { name: 'Select label' })).toBeVisible();

web_ui/src/features/dataset-import/components/dataset-import-to-new-project/dataset-import-to-new-project-dialog-buttons.test.tsx

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -34,8 +34,8 @@ const mockDeleteTemporallyDatasetImport = jest.fn();
3434
let mockIsReadyValue = false;
3535

3636
const mockAbortActiveUpload = jest.fn();
37-
jest.mock('../../../../../providers/tus-upload-provider/tus-upload-provider.component', () => ({
38-
...jest.requireActual('../../../../../providers/tus-upload-provider/tus-upload-provider.component'),
37+
jest.mock('../../../../providers/tus-upload-provider/tus-upload-provider.component', () => ({
38+
...jest.requireActual('../../../../providers/tus-upload-provider/tus-upload-provider.component'),
3939
useTusUpload: () => ({ abortActiveUpload: mockAbortActiveUpload }),
4040
}));
4141

web_ui/src/features/dataset-import/components/dataset-import-to-new-project/dataset-import-to-new-project-dialog-content.test.tsx

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -9,6 +9,7 @@ import {
99
DATASET_IMPORT_TO_NEW_PROJECT_STEP,
1010
} from '../../../../core/datasets/dataset.enum';
1111
import { DatasetImportToNewProjectItem, DatasetImportWarning } from '../../../../core/datasets/dataset.interface';
12+
import { useNotification } from '../../../../notification/notification.component';
1213
import { ProjectProvider } from '../../../../pages/project-details/providers/project-provider/project-provider.component';
1314
import { getMockedProjectIdentifier } from '../../../../test-utils/mocked-items-factory/mocked-identifiers';
1415
import { providersRender as render } from '../../../../test-utils/required-providers-render';
@@ -40,8 +41,8 @@ const mockDatasetImportItem: DatasetImportToNewProjectItem = {
4041
completedSteps: [],
4142
};
4243

43-
jest.mock('../../../../../notification/notification.component', () => ({
44-
...jest.requireActual('../../../../../notification/notification.component'),
44+
jest.mock('../../../../notification/notification.component', () => ({
45+
...jest.requireActual('../../../../notification/notification.component'),
4546
useNotification: () => ({ addNotification: jest.fn() }),
4647
}));
4748

web_ui/src/features/dataset-import/components/dataset-import-to-new-project/dataset-import-to-new-project-labels.test.tsx

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -42,8 +42,8 @@ const MOCKED_LABELS_DEEP_HIERARCHY: DatasetImportLabel[] = [
4242
];
4343

4444
const mockedAddNotifications = jest.fn();
45-
jest.mock('../../../../../notification/notification.component', () => ({
46-
...jest.requireActual('../../../../../notification/notification.component'),
45+
jest.mock('../../../../notification/notification.component', () => ({
46+
...jest.requireActual('../../../../notification/notification.component'),
4747
useNotification: jest.fn(() => ({
4848
addNotification: mockedAddNotifications,
4949
})),

0 commit comments

Comments
 (0)