FilesGarbageCollectorTest.php 13 KB

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