Skip to content

Commit 74407a3

Browse files
authored
Merge pull request #9070 from marmelab/stabilize-use-unique-tests
Stabilize useunique tests
2 parents 01338e7 + c058725 commit 74407a3

2 files changed

Lines changed: 17 additions & 22 deletions

File tree

packages/ra-core/src/form/useUnique.spec.tsx

Lines changed: 13 additions & 22 deletions
Original file line numberDiff line numberDiff line change
@@ -11,13 +11,6 @@ import { testDataProvider } from '../dataProvider';
1111
import { DataProvider } from '../types';
1212

1313
describe('useUnique', () => {
14-
beforeEach(() => {
15-
jest.useFakeTimers();
16-
});
17-
afterEach(() => {
18-
jest.useRealTimers();
19-
});
20-
2114
const baseDataProvider = (overrides?: Partial<DataProvider>) =>
2215
testDataProvider({
2316
// @ts-ignore
@@ -39,22 +32,20 @@ describe('useUnique', () => {
3932
await screen.findByDisplayValue('John Doe');
4033

4134
fireEvent.click(screen.getByText('Submit'));
42-
await waitFor(() => {
43-
expect(dataProvider.getList).toHaveBeenCalledWith('users', {
44-
filter: {
45-
name: 'John Doe',
46-
},
47-
pagination: {
48-
page: 1,
49-
perPage: 1,
50-
},
51-
sort: {
52-
field: 'id',
53-
order: 'ASC',
54-
},
55-
});
56-
});
5735
await screen.findByText('Must be unique');
36+
expect(dataProvider.getList).toHaveBeenCalledWith('users', {
37+
filter: {
38+
name: 'John Doe',
39+
},
40+
pagination: {
41+
page: 1,
42+
perPage: 1,
43+
},
44+
sort: {
45+
field: 'id',
46+
order: 'ASC',
47+
},
48+
});
5849
expect(dataProvider.create).not.toHaveBeenCalled();
5950
});
6051

packages/ra-core/src/form/useUnique.stories.tsx

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -12,6 +12,8 @@ import {
1212
mergeTranslations,
1313
useUnique,
1414
} from '..';
15+
import { createMemoryHistory } from 'history';
16+
import { QueryClient } from 'react-query';
1517

1618
export default {
1719
title: 'ra-core/form/useUnique',
@@ -79,6 +81,8 @@ const Wrapper = ({ children, dataProvider = defaultDataProvider }) => {
7981
<CoreAdminContext
8082
dataProvider={dataProvider}
8183
i18nProvider={i18nProvider}
84+
history={createMemoryHistory()}
85+
queryClient={new QueryClient()}
8286
>
8387
<CreateBase resource="users">{children}</CreateBase>
8488
</CoreAdminContext>

0 commit comments

Comments
 (0)