test: add deprecateRedirectUri utility tests to meet 100% coverage

Author: subhankarmaiti

__tests__/auth-reducer.test.tsx

@@ -19,6 +19,7 @@ describe('reducer', () => {
   it('should initialise when not authenticated', async () => {
     const payload = {
       isAuthenticated: false,
+      user: undefined,
     };
     expect(
       reducer(initialAuthState, { type: 'INITIALISED', ...payload })

__tests__/utils.test.tsx

@@ -1,6 +1,31 @@
-import { hasAuthParams, loginError, tokenError } from '../src/utils';
+import { hasAuthParams, loginError, tokenError, deprecateRedirectUri } from '../src/utils';
 import { OAuthError } from '../src/errors';
 
+// Define interfaces for testing deprecateRedirectUri function
+interface TestOptionsWithRedirectUri {
+  redirectUri?: string;
+  authorizationParams?: {
+    redirect_uri?: string;
+    scope?: string;
+  };
+}
+
+interface TestOptionsWithAuthorizationParams {
+  authorizationParams: {
+    redirectUri?: string;
+    redirect_uri?: string;
+    scope?: string;
+  };
+}
+
+interface TestOptionsWithBothRedirectUri {
+  redirectUri?: string;
+  authorizationParams: {
+    scope: string;
+    redirect_uri?: string;
+  };
+}
+
 describe('utils hasAuthParams', () => {
   it('should not recognise only the code param', async () => {
     ['?code=1', '?foo=1&code=2', '?code=1&foo=2'].forEach((search) =>
@@ -62,3 +87,75 @@ describe('utils error', () => {
     }).toThrowError('Login failed');
   });
 });
+
+describe('utils deprecateRedirectUri', () => {
+  let consoleSpy: jest.SpyInstance;
+
+  beforeEach(() => {
+    consoleSpy = jest.spyOn(console, 'warn').mockImplementation(() => {});
+  });
+
+  afterEach(() => {
+    consoleSpy.mockRestore();
+  });
+
+  it('should handle options with redirectUri', () => {
+    const options: TestOptionsWithRedirectUri = {
+      redirectUri: 'https://example.com/callback',
+    };
+    
+    deprecateRedirectUri(options);
+    
+    expect(consoleSpy).toHaveBeenCalledWith(
+      'Using `redirectUri` has been deprecated, please use `authorizationParams.redirect_uri` instead as `redirectUri` will be no longer supported in a future version'
+    );
+    expect(options.authorizationParams?.redirect_uri).toBe('https://example.com/callback');
+    expect(options.redirectUri).toBeUndefined();
+  });
+
+  it('should handle options with authorizationParams.redirectUri', () => {
+    const options: TestOptionsWithAuthorizationParams = {
+      authorizationParams: {
+        redirectUri: 'https://example.com/callback',
+      },
+    };
+    
+    deprecateRedirectUri(options);
+    
+    expect(consoleSpy).toHaveBeenCalledWith(
+      'Using `authorizationParams.redirectUri` has been deprecated, please use `authorizationParams.redirect_uri` instead as `authorizationParams.redirectUri` will be removed in a future version'
+    );
+    expect(options.authorizationParams.redirect_uri).toBe('https://example.com/callback');
+    expect(options.authorizationParams.redirectUri).toBeUndefined();
+  });
+
+  it('should handle options with both redirectUri and existing authorizationParams', () => {
+    const options: TestOptionsWithBothRedirectUri = {
+      redirectUri: 'https://example.com/callback',
+      authorizationParams: {
+        scope: 'openid profile',
+      },
+    };
+    
+    deprecateRedirectUri(options);
+    
+    expect(options.authorizationParams.redirect_uri).toBe('https://example.com/callback');
+    expect(options.authorizationParams.scope).toBe('openid profile');
+    expect(options.redirectUri).toBeUndefined();
+  });
+
+  it('should handle undefined options', () => {
+    deprecateRedirectUri(undefined);
+    expect(consoleSpy).not.toHaveBeenCalled();
+  });
+
+  it('should handle options without redirectUri properties', () => {
+    const options = {
+      domain: 'example.auth0.com',
+      clientId: 'client-id',
+    };
+    
+    deprecateRedirectUri(options);
+    expect(consoleSpy).not.toHaveBeenCalled();
+  });
+});

src/auth0-provider.tsx

@@ -198,7 +198,7 @@ const Auth0Provider = <TUser extends User = User>(opts: Auth0ProviderOptions<TUs
       const user = await client.getUser();
       dispatch({ type: 'LOGIN_POPUP_COMPLETE', user });
     },
-    [client]
+    [client, handleError]
   );
 
   const logout = useCallback(