diff --git a/mailpoet/lib/Newsletter/Sending/ScheduledTasksRepository.php b/mailpoet/lib/Newsletter/Sending/ScheduledTasksRepository.php index 08b9f6bc9a..8ff1889bbc 100644 --- a/mailpoet/lib/Newsletter/Sending/ScheduledTasksRepository.php +++ b/mailpoet/lib/Newsletter/Sending/ScheduledTasksRepository.php @@ -239,7 +239,7 @@ class ScheduledTasksRepository extends Repository { /** * @return ScheduledTaskEntity[] */ - public function findScheduledSendingTasks(int $limit): array { + public function findScheduledSendingTasks(?int $limit = null): array { $now = Carbon::createFromTimestamp($this->wp->currentTime('timestamp')); return $this->doctrineRepository->createQueryBuilder('st') ->select('st') diff --git a/mailpoet/tests/integration/Newsletter/Sending/ScheduledTasksRepositoryTest.php b/mailpoet/tests/integration/Newsletter/Sending/ScheduledTasksRepositoryTest.php index 6abe4ec900..cd56b27f5e 100644 --- a/mailpoet/tests/integration/Newsletter/Sending/ScheduledTasksRepositoryTest.php +++ b/mailpoet/tests/integration/Newsletter/Sending/ScheduledTasksRepositoryTest.php @@ -198,7 +198,7 @@ class ScheduledTasksRepositoryTest extends \MailPoetTest { $task = $this->scheduledTaskFactory->create(SendingQueueWorker::TASK_TYPE, ScheduledTaskEntity::STATUS_SCHEDULED, Carbon::now()->addDay()); $this->sendingQueueFactory->create($task); - $tasks = $this->repository->findScheduledSendingTasks(5); + $tasks = $this->repository->findScheduledSendingTasks(); $this->assertSame($expectedResult, $tasks); } diff --git a/mailpoet/tests/integration/Tasks/SendingTest.php b/mailpoet/tests/integration/Tasks/SendingTest.php index 27eaeb348c..354a9ea0ee 100644 --- a/mailpoet/tests/integration/Tasks/SendingTest.php +++ b/mailpoet/tests/integration/Tasks/SendingTest.php @@ -157,7 +157,7 @@ class SendingTest extends \MailPoetTest { $this->sending->status = ScheduledTask::STATUS_SCHEDULED; $this->sending->scheduled_at = Carbon::createFromTimestamp(WPFunctions::get()->currentTime('timestamp'))->subHours(1); $this->sending->save(); - $tasks = $this->scheduledTaskRepository->findScheduledSendingTasks(5); + $tasks = $this->scheduledTaskRepository->findScheduledSendingTasks(); expect($tasks)->notEmpty(); foreach ($tasks as $task) { expect($task)->isInstanceOf(ScheduledTaskEntity::class); @@ -181,7 +181,7 @@ class SendingTest extends \MailPoetTest { public function testItDoesNotGetPaused() { $this->_after(); $this->createNewSendingTask(['status' => ScheduledTask::STATUS_PAUSED]); - expect($this->scheduledTaskRepository->findScheduledSendingTasks(5))->count(0); + expect($this->scheduledTaskRepository->findScheduledSendingTasks())->count(0); } public function testItGetsRunningQueues() {