Fix test warnings

marmelab · Mar 23, 2024 · 2d5a5e2 · 2d5a5e2
1 parent 9bd5179
commit 2d5a5e2
Show file tree

Hide file tree

Showing 4 changed files with 19 additions and 7 deletions.
diff --git a/packages/ra-core/src/controller/field/useReferenceArrayFieldController.spec.tsx b/packages/ra-core/src/controller/field/useReferenceArrayFieldController.spec.tsx
@@ -38,7 +38,11 @@ describe('<useReferenceArrayFieldController />', () => {
     it('should set the isLoading prop to true when related records are not yet fetched', () => {
         const children = jest.fn().mockReturnValue('child');
         render(
-            <CoreAdminContext dataProvider={testDataProvider()}>
+            <CoreAdminContext
+                dataProvider={testDataProvider({
+                    getMany: () => new Promise(() => {}),
+                })}
+            >
                 <ReferenceArrayFieldController
                     resource="foo"
                     reference="bar"

diff --git a/packages/ra-core/src/controller/field/useReferenceManyFieldController.spec.tsx b/packages/ra-core/src/controller/field/useReferenceManyFieldController.spec.tsx
@@ -288,8 +288,11 @@ describe('useReferenceManyFieldController', () => {
                 }}
             />
         );
-        const dataProvider = testDataProvider();
-        const getManyReference = jest.spyOn(dataProvider, 'getManyReference');
+        const dataProvider = testDataProvider({
+            getManyReference: jest
+                .fn()
+                .mockResolvedValue({ data: [], total: 0 }),
+        });
         render(
             <CoreAdminContext dataProvider={dataProvider}>
                 <ReferenceManyFieldController
@@ -312,8 +315,8 @@ describe('useReferenceManyFieldController', () => {
         await waitFor(() => new Promise(resolve => setTimeout(resolve, 600)));
 
         // Called twice: on load and on filter changes
-        expect(getManyReference).toHaveBeenCalledTimes(2);
-        expect(getManyReference).toHaveBeenCalledWith('books', {
+        expect(dataProvider.getManyReference).toHaveBeenCalledTimes(2);
+        expect(dataProvider.getManyReference).toHaveBeenCalledWith('books', {
             target: 'author_id',
             id: 123,
             filter: { q: 'hello' },

diff --git a/packages/ra-ui-materialui/src/layout/ResourceMenuItem.spec.tsx b/packages/ra-ui-materialui/src/layout/ResourceMenuItem.spec.tsx
@@ -8,7 +8,9 @@ import { ListGuesser } from '../list';
 
 describe('ResourceMenuItem', () => {
     it('should not throw when used with only <Resource> as <Admin> child', async () => {
-        const dataProvider = testDataProvider();
+        const dataProvider = testDataProvider({
+            getList: () => Promise.resolve({ data: [], total: 0 }),
+        });
         const CustomMenu = () => (
             <Menu>
                 <Menu.ResourceItem name="users" />
@@ -27,7 +29,9 @@ describe('ResourceMenuItem', () => {
         render(<App />);
     });
     it('should not throw when used with a Function as <Admin> child', async () => {
-        const dataProvider = testDataProvider();
+        const dataProvider = testDataProvider({
+            getList: () => Promise.resolve({ data: [], total: 0 }),
+        });
         const authProvider: any = {
             getPermissions: () => Promise.resolve([]),
             checkAuth: () => Promise.resolve(),

diff --git a/packages/ra-ui-materialui/src/list/ListGuesser.spec.tsx b/packages/ra-ui-materialui/src/list/ListGuesser.spec.tsx
@@ -25,6 +25,7 @@ describe('<ListGuesser />', () => {
                     ],
                     total: 1,
                 }),
+            getMany: () => Promise.resolve({ data: [], total: 0 }),
         });
         render(
             <ThemeProvider theme={{}}>
-Original file line number
+Diff line change
@@ Expand Up / @@ -25,6 +25,7 @@ describe('<ListGuesser />', () => { @@
                         ],
                         total: 1,
                     }),
+                getMany: () => Promise.resolve({ data: [], total: 0 }),
             });
             render(
                 <ThemeProvider theme={{}}>
@@ Expand Down @@