FilesGarbageCollectorTest.php 13 KB

123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156157158159160161162163164165166167168169170171172173174175176177178179180181182183184185186187188189190191192193194195196197198199200201202203204205206207208209210211212213214215216217218219220221222223224225226227228229230231232233234235236237238239240241242243244245246247248249250251252253254255256257258259260261262263264265266267268269270271272273274275276277278279280281282283284285286287288289290291292293294295296297298299300301302303304305306307308309310311312313314315316317318319320321322323324325326327328329330331332333334335336337338339340341342343344345346347348349350351352353354355356357358359360361362363364365366367368369370371372373374375376377378379380381382383384385386387388389390391392393394395396397398399400401402403404405406407408409410411412413414415416417418
  1. <?php
  2. namespace App\Tests\Unit\Service\Cron\Job;
  3. use App\Entity\Core\File;
  4. use App\Enum\Core\FileHostEnum;
  5. use App\Enum\Core\FileStatusEnum;
  6. use App\Repository\Core\FileRepository;
  7. use App\Service\Cron\Job\FilesGarbageCollector;
  8. use App\Service\Cron\UI\CronUIInterface;
  9. use App\Service\File\Storage\LocalStorage;
  10. use Doctrine\DBAL\Connection;
  11. use Doctrine\ORM\Query;
  12. use Doctrine\ORM\QueryBuilder;
  13. use Doctrine\ORM\Query\Expr;
  14. use PHPUnit\Framework\MockObject\MockObject;
  15. use PHPUnit\Framework\TestCase;
  16. use Psr\Log\LoggerInterface;
  17. class TestableFilesGarbageCollector extends FilesGarbageCollector
  18. {
  19. public function listFilesToDelete(): array
  20. {
  21. return parent::listFilesToDelete();
  22. }
  23. public function deleteFiles(array $files): void
  24. {
  25. parent::deleteFiles($files);
  26. }
  27. public function getQueryConditions(QueryBuilder $queryBuilder): void
  28. {
  29. parent::getQueryConditions($queryBuilder);
  30. }
  31. }
  32. class FilesGarbageCollectorTest extends TestCase
  33. {
  34. private CronUIInterface|MockObject $ui;
  35. private MockObject|LoggerInterface $logger;
  36. private Connection|MockObject $connection;
  37. private FileRepository|MockObject $fileRepository;
  38. private LocalStorage|MockObject $storage;
  39. public function setUp(): void
  40. {
  41. $this->ui = $this->getMockBuilder(CronUIInterface::class)->disableOriginalConstructor()->getMock();
  42. $this->logger = $this->getMockBuilder(LoggerInterface::class)->disableOriginalConstructor()->getMock();
  43. $this->connection = $this->getMockBuilder(Connection::class)->disableOriginalConstructor()->getMock();
  44. $this->fileRepository = $this->getMockBuilder(FileRepository::class)->disableOriginalConstructor()->getMock();
  45. $this->storage = $this->getMockBuilder(LocalStorage::class)->disableOriginalConstructor()->getMock();
  46. }
  47. private function getFilesGarbageCollectorMockFor(string $methodName): TestableFilesGarbageCollector|MockObject
  48. {
  49. $filesGarbageCollector = $this->getMockBuilder(TestableFilesGarbageCollector::class)
  50. ->setConstructorArgs([$this->connection, $this->fileRepository, $this->storage])
  51. ->setMethodsExcept([$methodName, 'setUI', 'setLoggerInterface'])
  52. ->getMock();
  53. $filesGarbageCollector->setUI($this->ui);
  54. $filesGarbageCollector->setLoggerInterface($this->logger);
  55. return $filesGarbageCollector;
  56. }
  57. /**
  58. * @see FilesGarbageCollector::preview()
  59. */
  60. public function testPreview(): void
  61. {
  62. $filesGarbageCollector = $this->getFilesGarbageCollectorMockFor('preview');
  63. $file1 = $this->getMockBuilder(File::class)->getMock();
  64. $file1->method('getSlug')->willReturn('file1.txt');
  65. $file1->method('getStatus')->willReturn(FileStatusEnum::DELETION_REQUESTED);
  66. $file2 = $this->getMockBuilder(File::class)->getMock();
  67. $file2->method('getSlug')->willReturn('file2.txt');
  68. $file2->method('getStatus')->willReturn(FileStatusEnum::DELETION_REQUESTED);
  69. $files = [$file1, $file2];
  70. $filesGarbageCollector->expects(self::once())
  71. ->method('listFilesToDelete')
  72. ->willReturn($files);
  73. $this->storage->expects(self::exactly(2))
  74. ->method('exists')
  75. ->willReturnMap([
  76. [$file1, true],
  77. [$file2, false]
  78. ]);
  79. $this->ui->expects(self::exactly(4))
  80. ->method('print')
  81. ->withConsecutive(
  82. ['2 files to be removed'],
  83. ['> Printing the first and last 50 :'],
  84. [' * file1.txt (Status: DELETION_REQUESTED - Exists: Yes)'],
  85. [' * file2.txt (Status: DELETION_REQUESTED - Exists: No)']
  86. );
  87. $filesGarbageCollector->preview();
  88. }
  89. /**
  90. * @see FilesGarbageCollector::execute()
  91. */
  92. public function testExecute(): void
  93. {
  94. $filesGarbageCollector = $this->getFilesGarbageCollectorMockFor('execute');
  95. $file1 = $this->getMockBuilder(File::class)->getMock();
  96. $file2 = $this->getMockBuilder(File::class)->getMock();
  97. $files = [$file1, $file2];
  98. $filesGarbageCollector->expects(self::once())
  99. ->method('listFilesToDelete')
  100. ->willReturn($files);
  101. $filesGarbageCollector->expects(self::once())
  102. ->method('deleteFiles')
  103. ->with($files);
  104. $this->logger->expects(self::once())
  105. ->method('info')
  106. ->with('2 files to be removed');
  107. $filesGarbageCollector->execute();
  108. }
  109. /**
  110. * @see FilesGarbageCollector::listFilesToDelete()
  111. */
  112. public function testListFilesToDelete(): void
  113. {
  114. $filesGarbageCollector = $this->getFilesGarbageCollectorMockFor('listFilesToDelete');
  115. $queryBuilder = $this->getMockBuilder(QueryBuilder::class)
  116. ->disableOriginalConstructor()
  117. ->getMock();
  118. $query = $this->getMockBuilder(Query::class)
  119. ->disableOriginalConstructor()
  120. ->getMock();
  121. $file1 = $this->getMockBuilder(File::class)->getMock();
  122. $file2 = $this->getMockBuilder(File::class)->getMock();
  123. $expectedResult = [$file1, $file2];
  124. $this->fileRepository->expects(self::once())
  125. ->method('createQueryBuilder')
  126. ->with('f')
  127. ->willReturn($queryBuilder);
  128. $queryBuilder->expects(self::once())
  129. ->method('select')
  130. ->willReturnSelf();
  131. $filesGarbageCollector->expects(self::once())
  132. ->method('getQueryConditions')
  133. ->with($queryBuilder);
  134. $queryBuilder->expects(self::once())
  135. ->method('getQuery')
  136. ->willReturn($query);
  137. $query->expects(self::once())
  138. ->method('getResult')
  139. ->willReturn($expectedResult);
  140. $this->ui->expects(self::once())
  141. ->method('print')
  142. ->with('List files with host ap2i and status DELETION_REQUESTED');
  143. $result = $filesGarbageCollector->listFilesToDelete();
  144. $this->assertEquals($expectedResult, $result);
  145. }
  146. /**
  147. * @see FilesGarbageCollector::deleteFiles()
  148. */
  149. public function testDeleteFiles(): void
  150. {
  151. $filesGarbageCollector = $this->getFilesGarbageCollectorMockFor('deleteFiles');
  152. $file1 = $this->getMockBuilder(File::class)->getMock();
  153. $file1->method('getId')->willReturn(1);
  154. $file2 = $this->getMockBuilder(File::class)->getMock();
  155. $file2->method('getId')->willReturn(2);
  156. $files = [$file1, $file2];
  157. $queryBuilder = $this->getMockBuilder(QueryBuilder::class)
  158. ->disableOriginalConstructor()
  159. ->getMock();
  160. $query = $this->getMockBuilder(Query::class)
  161. ->disableOriginalConstructor()
  162. ->getMock();
  163. $this->fileRepository->expects(self::once())
  164. ->method('createQueryBuilder')
  165. ->with('f')
  166. ->willReturn($queryBuilder);
  167. $this->storage->expects(self::exactly(2))
  168. ->method('exists')
  169. ->willReturnMap([
  170. [$file1, true],
  171. [$file2, false]
  172. ]);
  173. $this->storage->expects(self::once())
  174. ->method('hardDelete')
  175. ->with($file1);
  176. $queryBuilder->expects(self::exactly(2))
  177. ->method('delete')
  178. ->willReturnSelf();
  179. $queryBuilder->expects(self::exactly(2))
  180. ->method('where')
  181. ->with('f.id = :id')
  182. ->willReturnSelf();
  183. $queryBuilder->expects(self::exactly(2))
  184. ->method('setParameter')
  185. ->withConsecutive(
  186. ['id', 1],
  187. ['id', 2]
  188. )
  189. ->willReturnSelf();
  190. $queryBuilder->expects(self::exactly(2))
  191. ->method('getQuery')
  192. ->willReturn($query);
  193. $query->expects(self::exactly(2))
  194. ->method('execute');
  195. $this->logger->expects(self::exactly(3))
  196. ->method('info')
  197. ->withConsecutive(
  198. ['2 files to be removed'],
  199. ['Deleting files...'],
  200. ['2 files deleted']
  201. );
  202. $this->ui->expects(self::exactly(3))
  203. ->method('progress');
  204. $filesGarbageCollector->deleteFiles($files);
  205. }
  206. /**
  207. * @see FilesGarbageCollector::deleteFiles()
  208. */
  209. public function testDeleteFilesWithNonBlockingErrors(): void
  210. {
  211. $filesGarbageCollector = $this->getFilesGarbageCollectorMockFor('deleteFiles');
  212. $file1 = $this->getMockBuilder(File::class)->getMock();
  213. $file1->method('getId')->willReturn(1);
  214. $file2 = $this->getMockBuilder(File::class)->getMock();
  215. $file2->method('getId')->willReturn(2);
  216. $files = [$file1, $file2];
  217. $queryBuilder = $this->getMockBuilder(QueryBuilder::class)
  218. ->disableOriginalConstructor()
  219. ->getMock();
  220. $this->fileRepository->expects(self::once())
  221. ->method('createQueryBuilder')
  222. ->with('f')
  223. ->willReturn($queryBuilder);
  224. $this->storage->expects(self::exactly(2))
  225. ->method('exists')
  226. ->willReturnMap([
  227. [$file1, true],
  228. [$file2, true]
  229. ]);
  230. $this->storage->expects(self::exactly(2))
  231. ->method('hardDelete')
  232. ->willReturnOnConsecutiveCalls(
  233. $this->throwException(new \RuntimeException('Storage error')),
  234. $this->throwException(new \InvalidArgumentException('Invalid file'))
  235. );
  236. $this->logger->expects(self::exactly(3))
  237. ->method('info')
  238. ->withConsecutive(
  239. ['2 files to be removed'],
  240. ['Deleting files...'],
  241. ['0 files deleted']
  242. );
  243. $this->logger->expects(self::exactly(2))
  244. ->method('error')
  245. ->withConsecutive(
  246. ['ERROR : Storage error'],
  247. ['ERROR : Invalid file']
  248. );
  249. $this->ui->expects(self::exactly(3))
  250. ->method('progress');
  251. $filesGarbageCollector->deleteFiles($files);
  252. }
  253. /**
  254. * @see FilesGarbageCollector::deleteFiles()
  255. */
  256. public function testDeleteFilesWithBlockingError(): void
  257. {
  258. $filesGarbageCollector = $this->getFilesGarbageCollectorMockFor('deleteFiles');
  259. $file = $this->getMockBuilder(File::class)->getMock();
  260. $file->method('getId')->willReturn(1);
  261. $files = [$file];
  262. $queryBuilder = $this->getMockBuilder(QueryBuilder::class)
  263. ->disableOriginalConstructor()
  264. ->getMock();
  265. $this->fileRepository->expects(self::once())
  266. ->method('createQueryBuilder')
  267. ->with('f')
  268. ->willReturn($queryBuilder);
  269. $this->storage->expects(self::once())
  270. ->method('exists')
  271. ->with($file)
  272. ->willReturn(true);
  273. $this->storage->expects(self::once())
  274. ->method('hardDelete')
  275. ->with($file)
  276. ->willThrowException(new \Exception('Critical storage error'));
  277. $this->logger->expects(self::exactly(2))
  278. ->method('info')
  279. ->withConsecutive(
  280. ['1 files to be removed'],
  281. ['Deleting files...']
  282. );
  283. $this->ui->expects(self::exactly(2))
  284. ->method('progress');
  285. $this->expectException(\Exception::class);
  286. $this->expectExceptionMessage('Critical storage error');
  287. $filesGarbageCollector->deleteFiles($files);
  288. }
  289. /**
  290. * @see FilesGarbageCollector::getQueryConditions()
  291. */
  292. public function testGetQueryConditions(): void
  293. {
  294. $filesGarbageCollector = $this->getFilesGarbageCollectorMockFor('getQueryConditions');
  295. $queryBuilder = $this->getMockBuilder(QueryBuilder::class)
  296. ->disableOriginalConstructor()
  297. ->getMock();
  298. $expr = $this->getMockBuilder(Expr::class)
  299. ->disableOriginalConstructor()
  300. ->getMock();
  301. $queryBuilder->expects(self::atLeastOnce())
  302. ->method('expr')
  303. ->willReturn($expr);
  304. $hostComparison = $this->getMockBuilder(Expr\Comparison::class)
  305. ->disableOriginalConstructor()
  306. ->getMock();
  307. $statusComparison = $this->getMockBuilder(Expr\Comparison::class)
  308. ->disableOriginalConstructor()
  309. ->getMock();
  310. $expr->expects(self::exactly(2))
  311. ->method('eq')
  312. ->willReturnMap([
  313. ['f.host', ':host', $hostComparison],
  314. ['f.status', ':status', $statusComparison]
  315. ]);
  316. $queryBuilder->expects(self::exactly(2))
  317. ->method('andWhere')
  318. ->withConsecutive(
  319. [$hostComparison],
  320. [$statusComparison]
  321. )
  322. ->willReturnSelf();
  323. $queryBuilder->expects(self::exactly(2))
  324. ->method('setParameter')
  325. ->withConsecutive(
  326. ['host', FileHostEnum::AP2I],
  327. ['status', FileStatusEnum::DELETION_REQUESTED]
  328. )
  329. ->willReturnSelf();
  330. $filesGarbageCollector->getQueryConditions($queryBuilder);
  331. }
  332. }