Cavemanon/piratepoet
7
Fork 0
Code Pull Requests Actions Releases 16 Activity

Allow running tasks to be cancelled

Browse Source 
[MAILPOET-5755]
This commit is contained in:
 Ján Mikláš
2024-06-18 22:26:46 +02:00
committed by Ján Mikláš
parent b8ca544596
commit ef2101ef46
2 changed files with 24 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

9
mailpoet/lib/Newsletter/Sending/ScheduledTasksRepository.php
View File

@@ -21,6 +21,11 @@ use MailPoetVendor\Doctrine\ORM\Query\Expr\Join;
*/
class ScheduledTasksRepository extends Repository {
const TASK_BATCH_SIZE = 20;
const CANCELLABLE_STATUSES = [
ScheduledTaskEntity::STATUS_SCHEDULED,
ScheduledTaskEntity::VIRTUAL_STATUS_RUNNING,
null,
];
/** @var WPFunctions */
private $wp;
@@ -345,8 +350,8 @@ class ScheduledTasksRepository extends Repository {
}
public function cancelTask(ScheduledTaskEntity $task): void {
if ($task->getStatus() !== ScheduledTaskEntity::STATUS_SCHEDULED) {
throw new \Exception(__('Only scheduled tasks can be cancelled', 'mailpoet'), 400);
if (!in_array($task->getStatus(), self::CANCELLABLE_STATUSES)) {
throw new \Exception(__('Only scheduled and running tasks can be cancelled', 'mailpoet'), 400);
}
$task->setStatus(ScheduledTaskEntity::STATUS_CANCELLED);
$task->setCancelledAt(Carbon::createFromTimestamp($this->wp->currentTime('timestamp')));

19
mailpoet/tests/integration/API/JSON/v1/HelpTest.php
View File

@@ -51,7 +51,7 @@ class HelpTest extends \MailPoetTest {
$response = $this->endpoint->cancelTask(['id' => $task->getId()]);
verify($response)->instanceOf('\MailPoet\API\JSON\ErrorResponse');
verify($response->status)->equals(400);
verify($response->errors[0]['message'])->equals('Only scheduled tasks can be cancelled');
verify($response->errors[0]['message'])->equals('Only scheduled and running tasks can be cancelled');
}
public function testItReturnsErrorWhenReschedulingCompletedTask() {
@@ -62,7 +62,7 @@ class HelpTest extends \MailPoetTest {
verify($response->errors[0]['message'])->equals('Only cancelled tasks can be rescheduled');
}
public function testItCanCancelTask() {
public function testItCanCancelScheduledTask() {
$task = (new ScheduledTaskFactory())->create('sending', ScheduledTaskEntity::STATUS_SCHEDULED, new \DateTime());
$response = $this->endpoint->cancelTask(['id' => $task->getId()]);
verify($response)->instanceOf(APIResponse::class);
@@ -77,6 +77,21 @@ class HelpTest extends \MailPoetTest {
}
}
public function testItCanCancelRunningTask() {
$task = (new ScheduledTaskFactory())->create('sending', null, new \DateTime());
$response = $this->endpoint->cancelTask(['id' => $task->getId()]);
verify($response)->instanceOf(APIResponse::class);
verify($response->status)->equals(200);
$task = $this->scheduledTasksRepository->findOneById($task->getId());
verify($task)->instanceOf(ScheduledTaskEntity::class);
if ($task) {
verify($task->getStatus())->equals(ScheduledTaskEntity::STATUS_CANCELLED);
verify($task->getCancelledAt())->notNull();
verify($task->getInProgress())->equals(0);
}
}
public function testItCanRescheduleTask() {
$task = (new ScheduledTaskFactory())->create('sending', ScheduledTaskEntity::STATUS_CANCELLED, new \DateTime());
$response = $this->endpoint->rescheduleTask(['id' => $task->getId()]);