fix: change callback method to allow other sign in method

bjerkio · Sep 28, 2020 · c497f49 · c497f49
1 parent d05b964
commit c497f49
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/src/AuthContext.tsx b/src/AuthContext.tsx
@@ -85,7 +85,7 @@ export const AuthProvider: FC<AuthProviderProps> = ({
        * Check if the user is returning back from OIDC.
        */
       if (hasCodeInUrl(location)) {
-        await userManager.signinRedirectCallback();
+        await userManager.signinCallback();
         const user = await userManager.getUser();
         setUserData(user);
         onSignIn && onSignIn(user);

diff --git a/src/__tests__/AuthContext.test.tsx b/src/__tests__/AuthContext.test.tsx
@@ -68,7 +68,7 @@ describe('AuthContext', () => {
         getUser: async () => ({
           access_token: 'token',
         }),
-        signinRedirectCallback: jest.fn(),
+        signinCallback: jest.fn(),
       } as any;
       const { getByText } = render(
         <AuthProvider userManager={userManager}>
@@ -90,7 +90,7 @@ describe('AuthContext', () => {
   it('should login the user', async () => {
     const userManager = {
       getUser: jest.fn(),
-      signinRedirectCallback: jest.fn(),
+      signinCallback: jest.fn(),
     } as any;
     const location = {
       search: '?code=test-code',
@@ -106,7 +106,7 @@ describe('AuthContext', () => {
     );
     await waitFor(() => expect(onSignIn).toHaveBeenCalled());
     await waitFor(() =>
-      expect(userManager.signinRedirectCallback).toHaveBeenCalled(),
+      expect(userManager.signinCallback).toHaveBeenCalled(),
     );
   });