opensource/cloudbeaver
3
0
Fork 0
mirror of https://github.com/dbeaver/cloudbeaver.git synced 2026-09-24 16:04:36 +08:00
Code Issues Packages Projects Releases Wiki Activity

dbeaver/pro#6113 fixes clientActivity wrong spying on SetTimeout (#3559)

Browse Source
This commit is contained in:
sergeyteleshev
2025-06-30 18:10:54 +08:00
committed by GitHub
parent 2117840e40
commit 85a242bf34
1 changed files with 12 additions and 9 deletions
Download Patch File Download Diff File Expand all files Collapse all files
webapp/packages/core-client-activity/src/ClientActivityService.test.ts
+12 -9
View File
@@ -28,9 +28,6 @@ describe('ClientActivityService', () => {
beforeEach(() => {
vitest.useFakeTimers();
clientActivityService = new ClientActivityService();
vitest.spyOn(globalThis, 'setTimeout');
vitest.spyOn(globalThis, 'clearTimeout');
});
afterEach(() => {
@@ -57,24 +54,30 @@ describe('ClientActivityService', () => {
it('should clear previous timer if updateActivity is called multiple times', () => {
clientActivityService.updateActivity();
expect(setTimeout).toHaveBeenCalledTimes(1);
expect(clientActivityService.isActive).toBe(true);
vitest.advanceTimersByTime(Math.random() * INACTIVE_PERIOD_TIME - 1);
vitest.advanceTimersByTime(INACTIVE_PERIOD_TIME - 10);
clientActivityService.updateActivity();
expect(clearTimeout).toHaveBeenCalledTimes(1);
expect(setTimeout).toHaveBeenCalledTimes(2);
expect(clientActivityService.isActive).toBe(true);
vitest.advanceTimersByTime(INACTIVE_PERIOD_TIME);
expect(clientActivityService.isActive).toBe(false);
});
it('should clear timer and reset activity when resetActivity is called', () => {
clientActivityService.updateActivity();
vitest.advanceTimersByTime(Math.random() * INACTIVE_PERIOD_TIME - 1);
vitest.advanceTimersByTime(INACTIVE_PERIOD_TIME - 10);
clientActivityService.resetActivity();
expect(clientActivityService.isActive).toBe(false);
expect(clearTimeout).toHaveBeenCalled();
vitest.advanceTimersByTime(INACTIVE_PERIOD_TIME);
expect(clientActivityService.isActive).toBe(false);
});
it('should call onActiveStateChange executor with correct value', () => {