From 0dc93bc320611a37b92175171035920e0532a207 Mon Sep 17 00:00:00 2001
From: Julien Veyssier <julien-nc@posteo.net>
Date: Wed, 6 Aug 2025 18:21:27 +0200
Subject: [PATCH] feat(taskprocessing): fix phpunit tests

Signed-off-by: Julien Veyssier <julien-nc@posteo.net>
---
 .../lib/TaskProcessing/TaskProcessingTest.php | 30 +++++++++++++++----
 1 file changed, 24 insertions(+), 6 deletions(-)

diff --git a/tests/lib/TaskProcessing/TaskProcessingTest.php b/tests/lib/TaskProcessing/TaskProcessingTest.php
index d8dea0db93b..1bd34996c61 100644
--- a/tests/lib/TaskProcessing/TaskProcessingTest.php
+++ b/tests/lib/TaskProcessing/TaskProcessingTest.php
@@ -943,11 +943,6 @@ class TaskProcessingTest extends \Test\TestCase {
 		$timeFactory->expects($this->any())->method('getDateTime')->willReturnCallback(fn () => $currentTime);
 		$timeFactory->expects($this->any())->method('getTime')->willReturnCallback(fn () => $currentTime->getTimestamp());
 
-		$this->taskMapper = new TaskMapper(
-			Server::get(IDBConnection::class),
-			$timeFactory,
-		);
-
 		$this->registrationContext->expects($this->any())->method('getTaskProcessingProviders')->willReturn([
 			new ServiceRegistration('test', SuccessfulSyncProvider::class)
 		]);
@@ -968,11 +963,34 @@ class TaskProcessingTest extends \Test\TestCase {
 
 		$task = $this->manager->getTask($task->getId());
 
+		$taskMapper = new TaskMapper(
+			Server::get(IDBConnection::class),
+			$timeFactory,
+		);
+		$manager = new Manager(
+			$this->appConfig,
+			$this->coordinator,
+			$this->serverContainer,
+			Server::get(LoggerInterface::class),
+			$taskMapper,
+			$this->jobList,
+			Server::get(IEventDispatcher::class),
+			Server::get(IAppDataFactory::class),
+			Server::get(IRootFolder::class),
+			Server::get(\OC\TextToImage\Manager::class),
+			$this->userMountCache,
+			Server::get(IClientService::class),
+			Server::get(IAppManager::class),
+			Server::get(IUserManager::class),
+			Server::get(IUserSession::class),
+			Server::get(ICacheFactory::class),
+		);
 		$currentTime = $currentTime->add(new \DateInterval('P1Y'));
 		// run background job
 		$bgJob = new RemoveOldTasksBackgroundJob(
 			$timeFactory,
-			$this->manager,
+			// use a locally defined manager to make sure the taskMapper uses the mocked timeFactory
+			$manager,
 		);
 		$bgJob->setArgument([]);
 		$bgJob->start($this->jobList);