File tree Expand file tree Collapse file tree 4 files changed +54
-0
lines changed
packages/ra-core/src/controller Expand file tree Collapse file tree 4 files changed +54
-0
lines changed Original file line number Diff line number Diff line change @@ -301,6 +301,31 @@ describe('EditBase', () => {
301301 expect ( dataProvider . getOne ) . toHaveBeenCalled ( ) ;
302302 await screen . findByText ( 'Hello' ) ;
303303 } ) ;
304+
305+ it ( 'should not wait for the authentication resolution before loading data when disableAuthentication is true' , async ( ) => {
306+ const authProvider = {
307+ login : ( ) => Promise . resolve ( ) ,
308+ logout : ( ) => Promise . resolve ( ) ,
309+ checkError : ( ) => Promise . resolve ( ) ,
310+ checkAuth : jest . fn ( ) ,
311+ } ;
312+ const dataProvider = testDataProvider ( {
313+ // @ts -ignore
314+ getOne : jest . fn ( ( ) =>
315+ Promise . resolve ( { data : { id : 12 , test : 'Hello' } } )
316+ ) ,
317+ } ) ;
318+ render (
319+ < WithAuthProviderNoAccessControl
320+ authProvider = { authProvider }
321+ dataProvider = { dataProvider }
322+ EditProps = { { disableAuthentication : true } }
323+ />
324+ ) ;
325+ await screen . findByText ( 'Hello' ) ;
326+ expect ( authProvider . checkAuth ) . not . toHaveBeenCalled ( ) ;
327+ } ) ;
328+
304329 it ( 'should wait for the authentication resolution before loading data' , async ( ) => {
305330 let resolveAuth : ( ) => void ;
306331 const authProvider = {
Original file line number Diff line number Diff line change @@ -115,13 +115,16 @@ export const WithAuthProviderNoAccessControl = ({
115115 checkAuth : ( ) => new Promise ( resolve => setTimeout ( resolve , 300 ) ) ,
116116 } ,
117117 dataProvider = defaultDataProvider ,
118+ EditProps,
118119} : {
119120 authProvider ?: AuthProvider ;
120121 dataProvider ?: DataProvider ;
122+ EditProps ?: Partial < EditBaseProps > ;
121123} ) => (
122124 < CoreAdminContext authProvider = { authProvider } dataProvider = { dataProvider } >
123125 < EditBase
124126 { ...defaultProps }
127+ { ...EditProps }
125128 loading = { < div > Authentication loading...</ div > }
126129 >
127130 < Child />
Original file line number Diff line number Diff line change @@ -24,6 +24,29 @@ describe('ShowBase', () => {
2424 expect ( dataProvider . getOne ) . toHaveBeenCalled ( ) ;
2525 await screen . findByText ( 'Hello' ) ;
2626 } ) ;
27+ it ( 'should not wait for the authentication resolution before loading data when disableAuthentication is true' , async ( ) => {
28+ const authProvider = {
29+ login : ( ) => Promise . resolve ( ) ,
30+ logout : ( ) => Promise . resolve ( ) ,
31+ checkError : ( ) => Promise . resolve ( ) ,
32+ checkAuth : jest . fn ( ) ,
33+ } ;
34+ const dataProvider = testDataProvider ( {
35+ // @ts -ignore
36+ getOne : jest . fn ( ( ) =>
37+ Promise . resolve ( { data : { id : 12 , test : 'Hello' } } )
38+ ) ,
39+ } ) ;
40+ render (
41+ < WithAuthProviderNoAccessControl
42+ authProvider = { authProvider }
43+ dataProvider = { dataProvider }
44+ ShowProps = { { disableAuthentication : true } }
45+ />
46+ ) ;
47+ await screen . findByText ( 'Hello' ) ;
48+ expect ( authProvider . checkAuth ) . not . toHaveBeenCalled ( ) ;
49+ } ) ;
2750 it ( 'should wait for the authentication resolution before loading data' , async ( ) => {
2851 let resolveAuth : ( ) => void ;
2952 const authProvider = {
Original file line number Diff line number Diff line change @@ -110,13 +110,16 @@ export const WithAuthProviderNoAccessControl = ({
110110 checkAuth : ( ) => new Promise ( resolve => setTimeout ( resolve , 300 ) ) ,
111111 } ,
112112 dataProvider = defaultDataProvider ,
113+ ShowProps,
113114} : {
114115 authProvider ?: AuthProvider ;
115116 dataProvider ?: DataProvider ;
117+ ShowProps ?: Partial < ShowBaseProps > ;
116118} ) => (
117119 < CoreAdminContext authProvider = { authProvider } dataProvider = { dataProvider } >
118120 < ShowBase
119121 { ...defaultProps }
122+ { ...ShowProps }
120123 loading = { < div > Authentication loading...</ div > }
121124 >
122125 < Child />
You can’t perform that action at this time.
0 commit comments