createController: story and test

Author: slax57

packages/ra-core/src/controller/create/useCreateController.security.stories.tsx

@@ -13,6 +13,7 @@ import {
     CreateControllerProps,
     useCreateController,
 } from './useCreateController';
+import { useAuthState } from '../../auth';
 
 export default {
     title: 'ra-core/controller/useCreateController',
@@ -39,6 +40,16 @@ const defaultDataProvider = fakeDataProvider(
     process.env.NODE_ENV === 'development'
 );
 
+const PostList = () => {
+    useAuthState();
+    return (
+        <div style={styles.mainContainer}>
+            <div>List view</div>
+            <Link to="/posts/create">Create</Link>
+        </div>
+    );
+};
+
 const CreatePost = (props: Partial<CreateControllerProps>) => {
     const params = useCreateController({
         resource: 'posts',
@@ -47,6 +58,7 @@ const CreatePost = (props: Partial<CreateControllerProps>) => {
     return (
         <div style={styles.mainContainer}>
             {params.isPending ? <p>Loading...</p> : <div>Create view</div>}
+            <Link to="/posts">List</Link>
         </div>
     );
 };
@@ -93,16 +105,33 @@ export const DisableAuthentication = ({
                 dataProvider={dataProvider}
                 authProvider={authProvider}
             >
-                <CoreAdminUI>
+                <CoreAdminUI accessDenied={AccessDenied}>
                     <Resource
                         name="posts"
+                        list={<PostList />}
                         create={<CreatePost disableAuthentication />}
                     />
                 </CoreAdminUI>
             </CoreAdminContext>
         </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/create/useCreateController.spec.tsx

@@ -10,6 +10,7 @@ import React from 'react';
 import { Route, Routes } from 'react-router-dom';
 
 import {
+    AuthProvider,
     CreateContextProvider,
     DataProvider,
     Form,
@@ -28,7 +29,10 @@ import {
 import { CreateController } from './CreateController';
 
 import { TestMemoryRouter } from '../../routing';
-import { CanAccess } from './useCreateController.security.stories';
+import {
+    CanAccess,
+    DisableAuthentication,
+} from './useCreateController.security.stories';
 
 describe('useCreateController', () => {
     const defaultProps = {
@@ -70,8 +74,8 @@ describe('useCreateController', () => {
         let saveCallback;
         const dataProvider = testDataProvider({
             getOne: () => Promise.resolve({ data: { id: 12 } } as any),
-            // @ts-ignore
             create: (_, { data }) =>
+                // @ts-ignore
                 Promise.resolve({ data: { id: 123, ...data } }),
         });
 
@@ -224,8 +228,8 @@ describe('useCreateController', () => {
         let saveCallback;
         const dataProvider = testDataProvider({
             getOne: () => Promise.resolve({ data: { id: 12 } } as any),
-            // @ts-ignore
             create: (_, { data }) =>
+                // @ts-ignore
                 Promise.resolve({ data: { id: 123, ...data } }),
         });
         const onSuccess = jest.fn();
@@ -262,8 +266,8 @@ describe('useCreateController', () => {
         let saveCallback;
         const dataProvider = testDataProvider({
             getOne: () => Promise.resolve({ data: { id: 12 } } as any),
-            // @ts-ignore
             create: (_, { data }) =>
+                // @ts-ignore
                 Promise.resolve({ data: { id: 123, ...data } }),
         });
         const onSuccess = jest.fn();
@@ -692,5 +696,39 @@ describe('useCreateController', () => {
             await screen.findByText('Loading...');
             await screen.findByText('Create view');
         });
+
+        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('Create view');
+            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('Create view');
+            fireEvent.click(await screen.findByText('List'));
+            await screen.findByText('List view');
+            fireEvent.click(await screen.findByText('Create'));
+            await screen.findByText('Create view');
+            expect(authProvider.checkAuth).toHaveBeenCalledTimes(1);
+            expect(authProvider.canAccess).not.toHaveBeenCalled();
+        });
     });
 });