limit once per 10s

sandy081 · sandy081 · commit 4af9b3e29a2d · 2020-05-18T16:56:27.000+02:00
diff --git a/src/vs/platform/userDataSync/common/userDataAutoSyncService.ts b/src/vs/platform/userDataSync/common/userDataAutoSyncService.ts
@@ -126,12 +126,12 @@ export class UserDataAutoSyncService extends Disposable implements IUserDataAuto
 
 		/*
 		If sync is not triggered by sync resource (triggered by other sources like window focus etc.,)
-		then limit sync to once per minute
+		then limit sync to once per 10s
 		*/
 		const isNotTriggeredBySyncResource = ALL_SYNC_RESOURCES.every(syncResource => sources.indexOf(syncResource) === -1);
 		if (isNotTriggeredBySyncResource && this.lastSyncTriggerTime
-			&& Math.round((new Date().getTime() - this.lastSyncTriggerTime) / 1000) < 60) {
-			this.logService.debug('Auto Sync Skipped: Limited to once per minute.');
+			&& Math.round((new Date().getTime() - this.lastSyncTriggerTime) / 1000) < 10) {
+			this.logService.debug('Auto Sync Skipped: Limited to once per 10 seconds.');
 			return;
 		}
 
diff --git a/src/vs/platform/userDataSync/test/common/userDataAutoSyncService.test.ts b/src/vs/platform/userDataSync/test/common/userDataAutoSyncService.test.ts
@@ -53,12 +53,11 @@ suite('UserDataAutoSyncService', () => {
 		const testObject: UserDataAutoSyncService = client.instantiationService.createInstance(TestUserDataAutoSyncService);
 
 		// Trigger auto sync with settings change multiple times
-		for (let counter = 0; counter < 3; counter++) {
+		for (let counter = 0; counter < 2; counter++) {
 			await testObject.triggerAutoSync([SyncResource.Settings]);
 		}
 
 		assert.deepEqual(target.requests, [
-			{ type: 'GET', url: `${target.url}/v1/manifest`, headers: {} },
 			{ type: 'GET', url: `${target.url}/v1/manifest`, headers: {} },
 			{ type: 'GET', url: `${target.url}/v1/manifest`, headers: {} }
 		]);
@@ -98,7 +97,7 @@ suite('UserDataAutoSyncService', () => {
 		const testObject: UserDataAutoSyncService = client.instantiationService.createInstance(TestUserDataAutoSyncService);
 
 		// Trigger auto sync with window focus multiple times
-		for (let counter = 0; counter < 3; counter++) {
+		for (let counter = 0; counter < 2; counter++) {
 			await testObject.triggerAutoSync(['windowFocus']);
 		}
 
