infiniteListController: fix

Author: slax57

packages/ra-core/src/controller/list/useInfiniteListController.spec.tsx

@@ -704,5 +704,41 @@ describe('useInfiniteListController', () => {
             // Only called once by NavigationToFirstResource
             expect(authProvider.checkAuth).toHaveBeenCalledTimes(1);
         });
+
+        it('should not call checkAuth nor canAccess when disableAuthentication is true', async () => {
+            const authProvider: AuthProvider = {
+                checkAuth: jest.fn().mockResolvedValue(true),
+                login: () => Promise.resolve(),
+                logout: () => Promise.resolve(),
+                checkError: () => Promise.resolve(),
+                getPermissions: () => Promise.resolve(),
+                canAccess: jest.fn().mockResolvedValue(false),
+            };
+            render(<DisableAuthentication authProvider={authProvider} />);
+            await screen.findByText('Post #1 - 90 votes');
+            expect(authProvider.checkAuth).not.toHaveBeenCalled();
+            expect(authProvider.canAccess).not.toHaveBeenCalled();
+        });
+
+        it('should not call checkAuth nor canAccess when disableAuthentication is true even if useAuthState was called before', async () => {
+            const authProvider: AuthProvider = {
+                checkAuth: jest.fn().mockResolvedValue(true),
+                login: () => Promise.resolve(),
+                logout: () => Promise.resolve(),
+                checkError: () => Promise.resolve(),
+                getPermissions: () => Promise.resolve(),
+                canAccess: jest.fn().mockResolvedValue(false),
+            };
+            render(<DisableAuthentication authProvider={authProvider} />);
+            await screen.findByText('Post #1 - 90 votes');
+            fireEvent.click(await screen.findByText('Dashboard'));
+            await screen.findByText('Dashboard view');
+            fireEvent.click(await screen.findByText('List'));
+            await screen.findByText('Post #1 - 90 votes');
+            // checkAuth is called twice: once by RA (with different params)
+            // and once by our custom Dashboard component
+            expect(authProvider.checkAuth).toHaveBeenCalledTimes(2);
+            expect(authProvider.canAccess).not.toHaveBeenCalled();
+        });
     });
 });

packages/ra-core/src/controller/list/useInfiniteListController.stories.tsx

@@ -10,6 +10,7 @@ import {
 } from './useInfiniteListController';
 import { Browser } from '../../storybook/FakeBrowser';
 import { TestMemoryRouter } from '../../routing';
+import { useAuthState } from '../..';
 
 export default {
     title: 'ra-core/controller/list/useInfiniteListController',
@@ -58,6 +59,17 @@ const List = params => {
                     </ul>
                 </div>
             )}
+            <Link to="/">Dashboard</Link>
+        </div>
+    );
+};
+
+const Dashboard = () => {
+    useAuthState();
+    return (
+        <div style={styles.mainContainer}>
+            <div>Dashboard view</div>
+            <Link to="/posts">List</Link>
         </div>
     );
 };
@@ -177,7 +189,7 @@ export const Authenticated = ({
     dataProvider?: DataProvider;
 }) => {
     return (
-        <TestMemoryRouter>
+        <TestMemoryRouter initialEntries={['/posts']}>
             <CoreAdminContext
                 dataProvider={dataProvider}
                 authProvider={authProvider}
@@ -198,12 +210,13 @@ export const DisableAuthentication = ({
     dataProvider?: DataProvider;
 }) => {
     return (
-        <TestMemoryRouter>
+        <TestMemoryRouter initialEntries={['/posts']}>
             <CoreAdminContext
                 dataProvider={dataProvider}
                 authProvider={authProvider}
+                dashboard={Dashboard}
             >
-                <CoreAdminUI>
+                <CoreAdminUI dashboard={Dashboard} accessDenied={AccessDenied}>
                     <Resource
                         name="posts"
                         list={<Posts disableAuthentication />}
@@ -213,6 +226,22 @@ export const DisableAuthentication = ({
         </TestMemoryRouter>
     );
 };
+DisableAuthentication.args = {
+    authProvider: undefined,
+};
+DisableAuthentication.argTypes = {
+    authProvider: {
+        options: ['default', 'canAccess'],
+        mapping: {
+            default: undefined,
+            canAccess: {
+                ...defaultAuthProvider,
+                canAccess: () => Promise.resolve(false),
+            },
+        },
+        control: { type: 'inline-radio' },
+    },
+};
 
 export const CanAccess = ({
     authProviderDelay = 300,

packages/ra-core/src/controller/list/useInfiniteListController.ts

@@ -79,8 +79,7 @@ export const useInfiniteListController = <
     const { isPending: isPendingCanAccess } = useRequireAccess<RecordType>({
         action: 'list',
         resource,
-        // If disableAuthentication is true then isPendingAuthenticated will always be true so this hook is disabled
-        enabled: !isPendingAuthenticated,
+        enabled: !disableAuthentication && !isPendingAuthenticated,
     });
 
     const translate = useTranslate();