diff --git a/packages/twenty-server/src/modules/messaging/message-folder-manager/services/sync-message-folders.service.spec.ts b/packages/twenty-server/src/modules/messaging/message-folder-manager/services/sync-message-folders.service.spec.ts index 0475e63efe6..7b2eeb522c9 100644 --- a/packages/twenty-server/src/modules/messaging/message-folder-manager/services/sync-message-folders.service.spec.ts +++ b/packages/twenty-server/src/modules/messaging/message-folder-manager/services/sync-message-folders.service.spec.ts @@ -370,7 +370,6 @@ describe('SyncMessageFoldersService', () => { id: 'folder-1', externalId: 'inbox-ext', name: 'INBOX', - isSynced: true, isSentFolder: false, parentFolderId: null, }); @@ -378,7 +377,6 @@ describe('SyncMessageFoldersService', () => { createMockDiscoveredFolder({ externalId: 'inbox-ext', name: 'INBOX', - isSynced: true, isSentFolder: false, parentFolderId: null, }), @@ -476,7 +474,6 @@ describe('SyncMessageFoldersService', () => { createMockDiscoveredFolder({ externalId: 'unchanged-ext', name: 'Unchanged', - isSynced: true, }), createMockDiscoveredFolder({ externalId: 'new-ext', @@ -537,7 +534,6 @@ describe('SyncMessageFoldersService', () => { createMockDiscoveredFolder({ externalId: 'inbox-ext', name: 'INBOX', - isSynced: true, }), ]; const messageChannel = createMockMessageChannel({ diff --git a/packages/twenty-server/src/modules/messaging/message-folder-manager/utils/__tests__/compute-folders-to-update.util.spec.ts b/packages/twenty-server/src/modules/messaging/message-folder-manager/utils/__tests__/compute-folders-to-update.util.spec.ts index 78280196ac5..e3cbe2ce232 100644 --- a/packages/twenty-server/src/modules/messaging/message-folder-manager/utils/__tests__/compute-folders-to-update.util.spec.ts +++ b/packages/twenty-server/src/modules/messaging/message-folder-manager/utils/__tests__/compute-folders-to-update.util.spec.ts @@ -93,72 +93,6 @@ describe('computeFoldersToUpdate', () => { expect(result.size).toBe(0); }); - it('should detect isSynced change from false to true', () => { - const discoveredFolders = [ - { - name: 'Inbox', - externalId: 'INBOX', - isSynced: true, - isSentFolder: false, - parentFolderId: null, - }, - ]; - - const existingFolders = [ - { - id: 'folder-id', - name: 'Inbox', - externalId: 'INBOX', - isSynced: false, - isSentFolder: false, - parentFolderId: null, - syncCursor: 'cursor', - pendingSyncAction: MessageFolderPendingSyncAction.NONE, - }, - ]; - - const result = computeFoldersToUpdate({ - discoveredFolders, - existingFolders, - }); - - expect(result.size).toBe(1); - expect(result.get('folder-id')?.isSynced).toBe(true); - }); - - it('should detect isSynced change from true to false', () => { - const discoveredFolders = [ - { - name: 'Promotions', - externalId: 'promo-1', - isSynced: false, - isSentFolder: false, - parentFolderId: null, - }, - ]; - - const existingFolders = [ - { - id: 'folder-id', - name: 'Promotions', - externalId: 'promo-1', - isSynced: true, - isSentFolder: false, - parentFolderId: null, - syncCursor: 'cursor', - pendingSyncAction: MessageFolderPendingSyncAction.NONE, - }, - ]; - - const result = computeFoldersToUpdate({ - discoveredFolders, - existingFolders, - }); - - expect(result.size).toBe(1); - expect(result.get('folder-id')?.isSynced).toBe(false); - }); - it('should treat empty string parentFolderId same as null', () => { const discoveredFolders = [ { diff --git a/packages/twenty-server/src/modules/messaging/message-folder-manager/utils/compute-folders-to-update.util.ts b/packages/twenty-server/src/modules/messaging/message-folder-manager/utils/compute-folders-to-update.util.ts index 784d23e1a52..3af55f1adcb 100644 --- a/packages/twenty-server/src/modules/messaging/message-folder-manager/utils/compute-folders-to-update.util.ts +++ b/packages/twenty-server/src/modules/messaging/message-folder-manager/utils/compute-folders-to-update.util.ts @@ -14,24 +14,12 @@ export const computeFoldersToUpdate = ({ }: { discoveredFolders: DiscoveredMessageFolder[]; existingFolders: MessageFolder[]; -}): Map< - string, - Pick< - MessageFolderEntity, - 'name' | 'isSynced' | 'isSentFolder' | 'parentFolderId' - > -> => { +}): Map> => { const existingFoldersByExternalId = new Map( existingFolders.map((folder) => [folder.externalId, folder]), ); - const foldersToUpdate = new Map< - string, - Pick< - MessageFolderEntity, - 'name' | 'isSynced' | 'isSentFolder' | 'parentFolderId' - > - >(); + const foldersToUpdate = new Map>(); for (const discoveredFolder of discoveredFolders) { const existingFolder = existingFoldersByExternalId.get( @@ -48,14 +36,12 @@ export const computeFoldersToUpdate = ({ const discoveredFolderData = { name: discoveredFolder.name, - isSynced: discoveredFolder.isSynced, isSentFolder: discoveredFolder.isSentFolder, parentFolderId, }; const existingFolderData = { name: existingFolder.name, - isSynced: existingFolder.isSynced, isSentFolder: existingFolder.isSentFolder, parentFolderId: isNonEmptyString(existingFolder.parentFolderId) ? existingFolder.parentFolderId