From 61e0844bdeee505e742424cdedff922fe725be2f Mon Sep 17 00:00:00 2001 From: Juan Escalada Date: Sat, 18 Oct 2025 21:08:11 +0900 Subject: [PATCH] test: amend existing disconnectedUsers tests and add new one for unresolved isLoggedIn Signed-off-by: Juan Escalada --- .../AutoLogout/disconnectedUsers.test.ts | 22 ++++++++++++++++++- 1 file changed, 21 insertions(+), 1 deletion(-) diff --git a/packages/core-components/src/components/AutoLogout/disconnectedUsers.test.ts b/packages/core-components/src/components/AutoLogout/disconnectedUsers.test.ts index 3d097dcc7f..80da5bca93 100644 --- a/packages/core-components/src/components/AutoLogout/disconnectedUsers.test.ts +++ b/packages/core-components/src/components/AutoLogout/disconnectedUsers.test.ts @@ -35,10 +35,27 @@ const mockTimestampStore = { }; describe('useLogoutDisconnectedUserEffect', () => { - it('should not do anything if effect is not enabled', () => { + it('should not do anything if isLoggedIn has not yet resolved', () => { + const props: UseLogoutDisconnectedUserEffectProps = { + enableEffect: true, + autologoutIsEnabled: true, + isLoggedIn: null, + idleTimeoutSeconds: 300, + lastSeenOnlineStore: mockTimestampStore, + identityApi: mockIdentityApi, + }; + + renderHook(() => useLogoutDisconnectedUserEffect(props)); + + expect(mockTimestampStore.get).not.toHaveBeenCalled(); + expect(mockIdentityApi.signOut).not.toHaveBeenCalled(); + }); + + it('should not do anything if effect is not enabled and isLoggedIn is false', () => { const props: UseLogoutDisconnectedUserEffectProps = { enableEffect: false, autologoutIsEnabled: true, + isLoggedIn: false, idleTimeoutSeconds: 300, lastSeenOnlineStore: mockTimestampStore, identityApi: mockIdentityApi, @@ -54,6 +71,7 @@ describe('useLogoutDisconnectedUserEffect', () => { const props: UseLogoutDisconnectedUserEffectProps = { enableEffect: true, autologoutIsEnabled: false, + isLoggedIn: true, idleTimeoutSeconds: 300, lastSeenOnlineStore: mockTimestampStore, identityApi: mockIdentityApi, @@ -70,6 +88,7 @@ describe('useLogoutDisconnectedUserEffect', () => { const props: UseLogoutDisconnectedUserEffectProps = { enableEffect: true, autologoutIsEnabled: true, + isLoggedIn: true, idleTimeoutSeconds: 1, lastSeenOnlineStore: { ...mockTimestampStore, @@ -93,6 +112,7 @@ describe('useLogoutDisconnectedUserEffect', () => { const props: UseLogoutDisconnectedUserEffectProps = { enableEffect: true, autologoutIsEnabled: true, + isLoggedIn: true, idleTimeoutSeconds: 300, lastSeenOnlineStore: mockTimestampStore, identityApi: mockIdentityApi,