reload page after logout if no redirect

Author: netanelgilad

src/modules/auth.ts

@@ -57,8 +57,8 @@ export function createAuthModule(
 
     /**
      * Logout the current user
-     * Removes the token from localStorage and optionally redirects to a URL
-     * @param redirectUrl - Optional URL to redirect to after logout
+     * Removes the token from localStorage and optionally redirects to a URL or reloads the page
+     * @param redirectUrl - Optional URL to redirect to after logout. Reloads the page if not provided
      * @returns {Promise<void>}
      */
     async logout(redirectUrl?: string) {
@@ -75,11 +75,13 @@ export function createAuthModule(
       }
 
       // Redirect if a URL is provided
-      if (redirectUrl && typeof window !== "undefined") {
-        window.location.href = redirectUrl;
+      if (typeof window !== "undefined") {
+        if (redirectUrl) {
+          window.location.href = redirectUrl;
+        } else {
+          window.location.reload();
+        }
       }
-
-      return Promise.resolve();
     },
 
     /**

tests/unit/auth.test.js

@@ -214,7 +214,10 @@ describe('Auth Module', () => {
       };
       const originalWindow = global.window;
       global.window = {
-        localStorage: mockLocalStorage
+        localStorage: mockLocalStorage,
+        location: {
+          reload: vi.fn()
+        }
       };
 
       // Set a token to localStorage first
@@ -241,7 +244,10 @@ describe('Auth Module', () => {
       const consoleSpy = vi.spyOn(console, 'error').mockImplementation(() => {});
       const originalWindow = global.window;
       global.window = {
-        localStorage: mockLocalStorage
+        localStorage: mockLocalStorage,
+        location: {
+          reload: vi.fn()
+        }
       };
 
       // Call logout - should not throw
@@ -272,6 +278,26 @@ describe('Auth Module', () => {
       // Restore window
       global.window = originalWindow;
     });
+    
+    test('should reload page when no redirect URL is provided', async () => {
+      // Mock window object with reload function
+      const mockReload = vi.fn();
+      const originalWindow = global.window;
+      global.window = {
+        location: {
+          reload: mockReload
+        }
+      };
+      
+      // Call logout without redirect URL
+      await base44.auth.logout();
+      
+      // Verify page reload was called
+      expect(mockReload).toHaveBeenCalledTimes(1);
+      
+      // Restore window
+      global.window = originalWindow;
+    });
   });
 
   describe('setToken()', () => {