0269bc6d28
Deux causes racines généraient ~170 erreurs/cycle (toutes les 10 min) sur la prod : "syncFolder[...] listMessages failed: Folder ... not found". 1. Double-encodage UTF7-IMAP : listFolders() stocke le chemin brut UTF7-IMAP, mais ImapMailProvider rappelait getFolder($path) qui ré-encode UTF8->UTF7-IMAP (webklex Client::getFolderByPath, utf7=false). Le caractère de shift "&" était ré-encodé, rendant introuvables les dossiers à accents/specials. Fix : getFolder($path, null, utf7: true) partout dans ImapMailProvider. 2. Dossiers fantômes jamais purgés : syncFolderStructure() gardait en DB les dossiers disparus du serveur, re-tentés à chaque cycle. Fix : syncFolderStructure() retourne le set des chemins présents sur le serveur ; doSyncAll() skip silencieusement les dossiers DB absents (conservés en DB pour les liens messages/tâches). Fallback historique si listFolders échoue. Test : testSyncAllSkipsFoldersNoLongerPresentOnServer.
251 lines
9.2 KiB
PHP
251 lines
9.2 KiB
PHP
<?php
|
|
|
|
declare(strict_types=1);
|
|
|
|
namespace App\Tests\Unit\Service;
|
|
|
|
use App\Module\Mail\Application\Dto\MailFolderDto;
|
|
use App\Module\Mail\Application\Service\MailSyncService;
|
|
use App\Module\Mail\Domain\Entity\MailConfiguration;
|
|
use App\Module\Mail\Domain\Entity\MailFolder;
|
|
use App\Module\Mail\Domain\Provider\MailProviderInterface;
|
|
use App\Module\Mail\Domain\Repository\MailConfigurationRepositoryInterface;
|
|
use App\Module\Mail\Domain\Repository\MailFolderRepositoryInterface;
|
|
use App\Module\Mail\Domain\Repository\MailMessageRepositoryInterface;
|
|
use Doctrine\ORM\EntityManagerInterface;
|
|
use Doctrine\Persistence\ManagerRegistry;
|
|
use PHPUnit\Framework\TestCase;
|
|
use Psr\Log\NullLogger;
|
|
use Symfony\Component\Lock\LockFactory;
|
|
use Symfony\Component\Lock\SharedLockInterface;
|
|
|
|
/**
|
|
* @internal
|
|
*/
|
|
class MailSyncServiceTest extends TestCase
|
|
{
|
|
public function testSyncAllReturnsEmptyReportWhenConfigDisabled(): void
|
|
{
|
|
$config = new MailConfiguration();
|
|
$config->setEnabled(false);
|
|
|
|
$configRepo = $this->createMock(MailConfigurationRepositoryInterface::class);
|
|
$configRepo->method('findSingleton')->willReturn($config);
|
|
|
|
$provider = $this->createMock(MailProviderInterface::class);
|
|
$folderRepo = $this->createMock(MailFolderRepositoryInterface::class);
|
|
$messageRepo = $this->createMock(MailMessageRepositoryInterface::class);
|
|
$em = $this->createMock(EntityManagerInterface::class);
|
|
$lockFactory = $this->makeLockFactory();
|
|
|
|
$service = new MailSyncService(
|
|
provider: $provider,
|
|
configRepository: $configRepo,
|
|
folderRepository: $folderRepo,
|
|
messageRepository: $messageRepo,
|
|
entityManager: $em,
|
|
lockFactory: $lockFactory,
|
|
logger: new NullLogger(),
|
|
managerRegistry: $this->createMock(ManagerRegistry::class),
|
|
);
|
|
|
|
$report = $service->syncAll();
|
|
|
|
self::assertSame(0, $report->createdCount);
|
|
self::assertSame(0, $report->updatedCount);
|
|
self::assertSame(0, $report->deletedCount);
|
|
self::assertSame(0, $report->foldersScanned);
|
|
}
|
|
|
|
public function testSyncAllReturnsEmptyReportWhenLockNotAcquired(): void
|
|
{
|
|
$config = new MailConfiguration();
|
|
$config->setEnabled(true);
|
|
|
|
$configRepo = $this->createMock(MailConfigurationRepositoryInterface::class);
|
|
$configRepo->method('findSingleton')->willReturn($config);
|
|
|
|
$provider = $this->createMock(MailProviderInterface::class);
|
|
$folderRepo = $this->createMock(MailFolderRepositoryInterface::class);
|
|
$messageRepo = $this->createMock(MailMessageRepositoryInterface::class);
|
|
$em = $this->createMock(EntityManagerInterface::class);
|
|
$lockFactory = $this->makeLockFactory(false);
|
|
|
|
$service = new MailSyncService(
|
|
provider: $provider,
|
|
configRepository: $configRepo,
|
|
folderRepository: $folderRepo,
|
|
messageRepository: $messageRepo,
|
|
entityManager: $em,
|
|
lockFactory: $lockFactory,
|
|
logger: new NullLogger(),
|
|
managerRegistry: $this->createMock(ManagerRegistry::class),
|
|
);
|
|
|
|
$report = $service->syncAll();
|
|
|
|
self::assertSame(0, $report->createdCount);
|
|
self::assertContains('lock_not_acquired', $report->errors);
|
|
}
|
|
|
|
public function testSyncFolderStructureCreatesNewFolders(): void
|
|
{
|
|
$config = new MailConfiguration();
|
|
$config->setEnabled(true);
|
|
|
|
$configRepo = $this->createMock(MailConfigurationRepositoryInterface::class);
|
|
$configRepo->method('findSingleton')->willReturn($config);
|
|
|
|
$folderDto = new MailFolderDto(
|
|
path: 'INBOX',
|
|
displayName: 'Inbox',
|
|
parentPath: null,
|
|
unreadCount: 5,
|
|
totalCount: 42,
|
|
);
|
|
|
|
$provider = $this->createMock(MailProviderInterface::class);
|
|
$provider->method('listFolders')->willReturn([$folderDto]);
|
|
|
|
$folderRepo = $this->createMock(MailFolderRepositoryInterface::class);
|
|
$folderRepo->method('findByPath')->willReturn(null);
|
|
$folderRepo->method('findAllOrderedByPath')->willReturn([]);
|
|
|
|
$messageRepo = $this->createMock(MailMessageRepositoryInterface::class);
|
|
$em = $this->createMock(EntityManagerInterface::class);
|
|
$em->expects(self::once())->method('persist');
|
|
$em->expects(self::once())->method('flush');
|
|
|
|
$lockFactory = $this->makeLockFactory();
|
|
|
|
$service = new MailSyncService(
|
|
provider: $provider,
|
|
configRepository: $configRepo,
|
|
folderRepository: $folderRepo,
|
|
messageRepository: $messageRepo,
|
|
entityManager: $em,
|
|
lockFactory: $lockFactory,
|
|
logger: new NullLogger(),
|
|
managerRegistry: $this->createMock(ManagerRegistry::class),
|
|
);
|
|
|
|
$service->syncFolderStructure();
|
|
}
|
|
|
|
public function testSyncAllSkipsFoldersNoLongerPresentOnServer(): void
|
|
{
|
|
$config = new MailConfiguration();
|
|
$config->setEnabled(true);
|
|
|
|
$configRepo = $this->createMock(MailConfigurationRepositoryInterface::class);
|
|
$configRepo->method('findSingleton')->willReturn($config);
|
|
|
|
// The server only exposes INBOX; "Trash/STALE" was deleted remotely but
|
|
// still lingers in the DB.
|
|
$inboxDto = new MailFolderDto(
|
|
path: 'INBOX',
|
|
displayName: 'Inbox',
|
|
parentPath: null,
|
|
unreadCount: 0,
|
|
totalCount: 0,
|
|
);
|
|
|
|
$inboxFolder = new MailFolder();
|
|
$inboxFolder->setPath('INBOX');
|
|
|
|
$staleFolder = new MailFolder();
|
|
$staleFolder->setPath('Trash/STALE');
|
|
|
|
$provider = $this->createMock(MailProviderInterface::class);
|
|
$provider->method('listFolders')->willReturn([$inboxDto]);
|
|
// listMessages must only ever be called for INBOX, never the stale folder.
|
|
$provider->expects(self::once())
|
|
->method('listMessages')
|
|
->with('INBOX', 5000, 0)
|
|
->willReturn([])
|
|
;
|
|
|
|
$folderRepo = $this->createMock(MailFolderRepositoryInterface::class);
|
|
$folderRepo->method('findByPath')->willReturn($inboxFolder);
|
|
$folderRepo->method('findAllOrderedByPath')->willReturn([$inboxFolder, $staleFolder]);
|
|
|
|
$messageRepo = $this->createMock(MailMessageRepositoryInterface::class);
|
|
$messageRepo->method('findMaxUidInFolder')->willReturn(0);
|
|
$messageRepo->method('findAllUidsByFolder')->willReturn([]);
|
|
$messageRepo->method('findLastNByFolder')->willReturn([]);
|
|
|
|
$em = $this->createMock(EntityManagerInterface::class);
|
|
$em->method('isOpen')->willReturn(true);
|
|
$lockFactory = $this->makeLockFactory();
|
|
|
|
$service = new MailSyncService(
|
|
provider: $provider,
|
|
configRepository: $configRepo,
|
|
folderRepository: $folderRepo,
|
|
messageRepository: $messageRepo,
|
|
entityManager: $em,
|
|
lockFactory: $lockFactory,
|
|
logger: new NullLogger(),
|
|
managerRegistry: $this->createMock(ManagerRegistry::class),
|
|
);
|
|
|
|
$report = $service->syncAll();
|
|
|
|
self::assertSame(1, $report->foldersScanned);
|
|
self::assertSame([], $report->errors);
|
|
}
|
|
|
|
public function testSyncFolderAbortsSuppressionWhenOver50Percent(): void
|
|
{
|
|
$config = new MailConfiguration();
|
|
$config->setEnabled(true);
|
|
|
|
$configRepo = $this->createMock(MailConfigurationRepositoryInterface::class);
|
|
$configRepo->method('findSingleton')->willReturn($config);
|
|
|
|
$folder = new MailFolder();
|
|
$folder->setPath('INBOX');
|
|
|
|
$messageRepo = $this->createMock(MailMessageRepositoryInterface::class);
|
|
$messageRepo->method('findMaxUidInFolder')->willReturn(10);
|
|
$messageRepo->method('findAllUidsByFolder')->willReturn([1, 2, 3, 4, 5, 6, 7, 8, 9, 10]);
|
|
$messageRepo->method('findLastNByFolder')->willReturn([]);
|
|
|
|
$provider = $this->createMock(MailProviderInterface::class);
|
|
$provider->method('listMessages')->willReturn([]);
|
|
|
|
$folderRepo = $this->createMock(MailFolderRepositoryInterface::class);
|
|
$em = $this->createMock(EntityManagerInterface::class);
|
|
$em->expects(self::never())->method('remove');
|
|
|
|
$lockFactory = $this->makeLockFactory();
|
|
|
|
$service = new MailSyncService(
|
|
provider: $provider,
|
|
configRepository: $configRepo,
|
|
folderRepository: $folderRepo,
|
|
messageRepository: $messageRepo,
|
|
entityManager: $em,
|
|
lockFactory: $lockFactory,
|
|
logger: new NullLogger(),
|
|
managerRegistry: $this->createMock(ManagerRegistry::class),
|
|
);
|
|
|
|
$report = $service->syncFolder($folder);
|
|
|
|
self::assertSame(0, $report->deletedCount);
|
|
self::assertNotEmpty($report->errors);
|
|
}
|
|
|
|
private function makeLockFactory(bool $acquired = true): LockFactory
|
|
{
|
|
$lock = $this->createMock(SharedLockInterface::class);
|
|
$lock->method('acquire')->willReturn($acquired);
|
|
|
|
$factory = $this->createMock(LockFactory::class);
|
|
$factory->method('createLock')->willReturn($lock);
|
|
|
|
return $factory;
|
|
}
|
|
}
|