Jiashen/test fix

lib/msal-browser/src/app/PublicClientApplication.ts

@@ -428,6 +428,10 @@ export class PublicClientApplication implements IPublicClientApplication {
     clearCache(logoutRequest?: ClearCacheRequest): Promise<void> {
         return this.controller.clearCache(logoutRequest);
     }
+
+    closeEventHandlerChannel(): void {
+        this.controller.closeEventHandlerChannel();
+    }
 }
 
 /**

lib/msal-browser/src/controllers/IController.ts

@@ -121,4 +121,6 @@ export interface IController {
 
     /** @internal */
     getPerformanceClient(): IPerformanceClient;
+
+    closeEventHandlerChannel(): void;
 }

lib/msal-browser/src/controllers/NestedAppAuthController.ts

@@ -881,4 +881,8 @@ export class NestedAppAuthController implements IController {
         );
         return this.browserStorage.hydrateCache(result, request);
     }
+
+    closeEventHandlerChannel(): void {
+        this.eventHandler.closeBroadcastChannel();
+    }
 }

lib/msal-browser/src/controllers/StandardController.ts

@@ -2322,6 +2322,10 @@ export class StandardController implements IController {
         );
         return res;
     }
+
+    closeEventHandlerChannel(): void {
+        this.eventHandler.closeBroadcastChannel();
+    }
 }
 
 /**

lib/msal-browser/src/controllers/UnknownOperatingContextController.ts

@@ -380,4 +380,8 @@ export class UnknownOperatingContextController implements IController {
         blockAPICallsBeforeInitialize(this.initialized);
         blockNonBrowserEnvironment();
     }
+
+    closeEventHandlerChannel(): void {
+        this.eventHandler.closeBroadcastChannel();
+    }
 }

lib/msal-browser/src/event/EventHandler.ts

@@ -158,4 +158,11 @@ export class EventHandler {
             this.invokeCrossTabCallbacks
         );
     }
+
+    /**
+     * Close the broadcast channel
+     */
+    closeBroadcastChannel(): void {
+        this.broadcastChannel.close();
+    }
 }

lib/msal-custom-auth/src/CustomAuthPublicClientApplication.ts

@@ -100,6 +100,10 @@ export class CustomAuthPublicClientApplication
         return this.customAuthController.resetPassword(resetPasswordInputs);
     }
 
+    close(): void {
+        this.closeEventHandlerChannel();
+    }
+
     /**
      * Validates the configuration to ensure it is a valid CustomAuthConfiguration object.
      * @param config The configuration object for the PublicClientApplication.

lib/msal-custom-auth/src/ICustomAuthPublicClientApplication.ts

@@ -37,4 +37,6 @@ export interface ICustomAuthPublicClientApplication {
      * @returns {Promise<ResetPasswordStartResult>} A promise that resolves to ResetPasswordStartResult
      */
     resetPassword(resetPasswordInputs: ResetPasswordInputs): Promise<ResetPasswordStartResult>;
+
+    close(): void;
 }

lib/msal-custom-auth/test/CustomAuthPublicClientApplication.spec.ts

@@ -65,6 +65,8 @@ describe("CustomAuthPublicClientApplication", () => {
             const app = await CustomAuthPublicClientApplication.create(customAuthConfig);
 
             expect(app).toBeInstanceOf(CustomAuthPublicClientApplication);
+
+            app.close();
         });
     });
 

lib/msal-custom-auth/test/controller/CustomAuthStandardController.spec.ts

@@ -90,6 +90,7 @@ describe("CustomAuthStandardController", () => {
     });
 
     afterEach(() => {
+        controller.closeEventHandlerChannel();
         jest.clearAllMocks(); // Clear mocks between tests
     });
 

lib/msal-custom-auth/test/integration_tests/GetAccount.spec.ts

@@ -20,6 +20,7 @@ describe("GetAccount", () => {
     });
 
     afterEach(() => {
+        app.close();
         jest.clearAllMocks(); // Clear mocks between tests
     });
 

lib/msal-custom-auth/test/integration_tests/ResetPassword.spec.ts

@@ -55,6 +55,7 @@ describe("Reset password", () => {
     });
 
     afterEach(() => {
+        app.close(); // Close the app
         jest.clearAllMocks(); // Clear mocks between tests
     });
 

lib/msal-custom-auth/test/integration_tests/SignIn.spec.ts

@@ -52,6 +52,7 @@ describe("Sign in", () => {
     });
 
     afterEach(() => {
+        app.close();
         jest.clearAllMocks(); // Clear mocks between tests
     });
 

lib/msal-custom-auth/test/integration_tests/SignUp.spec.ts

@@ -59,6 +59,7 @@ describe("Sign up", () => {
     });
 
     afterEach(() => {
+        app.close();
         jest.clearAllMocks(); // Clear mocks between tests
     });
 

lib/msal-custom-auth/test/test_resources/CustomAuthConfig.ts

@@ -3,7 +3,6 @@
  * Licensed under the MIT License.
  */
 
-import { LogLevel } from "@azure/msal-browser";
 import { CustomAuthConfiguration } from "../../src/configuration/CustomAuthConfiguration.js";
 
 export const customAuthConfig: CustomAuthConfiguration = {
@@ -23,25 +22,7 @@ export const customAuthConfig: CustomAuthConfiguration = {
     system: {
         loggerOptions: {
             loggerCallback: (level, message, containsPii) => {
-                if (containsPii) {
-                    return;
-                }
-                switch (level) {
-                    case LogLevel.Error:
-                        console.info(`[Error] ${message}`);
-                        return;
-                    case LogLevel.Info:
-                        console.info(`[Info] ${message}`);
-                        return;
-                    case LogLevel.Verbose:
-                        console.info(`[Verbose] ${message}`);
-                        return;
-                    case LogLevel.Warning:
-                        console.info(`[Warning] ${message}`);
-                        return;
-                    default:
-                        return;
-                }
+                return;
             },
         },
     },