ahtlon/deck
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases 1 Wiki Activity

fix: update DeleteCron to remove deleted lists

Browse Source 
Signed-off-by: Luka Trovic <luka@nextcloud.com>
This commit is contained in:
Luka Trovic
2025-05-28 11:52:02 +02:00
parent 9938a6d57f
commit 324eb05598
3 changed files with 48 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

28
tests/unit/Cron/DeleteCronTest.php
View File

@@ -30,6 +30,8 @@ use OCA\Deck\Db\Board;
use OCA\Deck\Db\BoardMapper;
use OCA\Deck\Db\Card;
use OCA\Deck\Db\CardMapper;
use OCA\Deck\Db\Stack;
use OCA\Deck\Db\StackMapper;
use OCA\Deck\InvalidAttachmentType;
use OCA\Deck\Service\AttachmentService;
use OCA\Deck\Service\IAttachmentService;
@@ -49,6 +51,8 @@ class DeleteCronTest extends TestCase {
private $attachmentService;
/** @var AttachmentMapper|MockObject */
private $attachmentMapper;
/** @var StackMapper|MockObject */
private $stackMapper;
/** @var DeleteCron */
protected $deleteCron;
@@ -59,7 +63,8 @@ class DeleteCronTest extends TestCase {
$this->cardMapper = $this->createMock(CardMapper::class);
$this->attachmentService = $this->createMock(AttachmentService::class);
$this->attachmentMapper = $this->createMock(AttachmentMapper::class);
$this->deleteCron = new DeleteCron($this->timeFactory, $this->boardMapper, $this->cardMapper, $this->attachmentService, $this->attachmentMapper);
$this->stackMapper = $this->createMock(StackMapper::class);
$this->deleteCron = new DeleteCron($this->timeFactory, $this->boardMapper, $this->cardMapper, $this->attachmentService, $this->attachmentMapper, $this->stackMapper);
}
protected function getBoard($id) {
@@ -74,6 +79,12 @@ class DeleteCronTest extends TestCase {
return $card;
}
protected function getStack($id) {
$stack = new Stack();
$stack->setId($id);
return $stack;
}
public function testDeleteCron() {
$boards = [
$this->getBoard(1),
@@ -118,6 +129,21 @@ class DeleteCronTest extends TestCase {
$this->attachmentMapper->expects($this->once())
->method('delete')
->with($attachment);
$stacks = [
$this->getStack(100),
$this->getStack(101),
];
$this->stackMapper->expects($this->once())
->method('findToDelete')
->willReturn($stacks);
$this->stackMapper->expects($this->exactly(count($stacks)))
->method('delete')
->withConsecutive(
[$stacks[0]],
[$stacks[1]]
);
$this->invokePrivate($this->deleteCron, 'run', [null]);
}