Fix ProcessFileJobTest by removing deprecated OCP\Snowflake\IGenerator dependency (#346)

* Initial plan

* Fix ProcessFileJobTest by removing deprecated IGenerator dependency

The test was failing because it tried to mock OCP\Snowflake\IGenerator which no longer exists in recent Nextcloud versions. The JobList constructor signature has changed and no longer requires this parameter.

Changes:
- Removed import of OCP\Snowflake\IGenerator
- Changed from concrete JobList class to IJobList interface
- Removed unnecessary mocks for IConfig, ITimeFactory, IDBConnection, IQueryBuilder, IExpressionBuilder
- Simplified test setup to just create a mock of IJobList
- Removed IGenerator property and related setup code

This aligns the test with how other Nextcloud server tests handle JobList mocking.

Co-authored-by: R0Wi <19730957+R0Wi@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: R0Wi <19730957+R0Wi@users.noreply.github.com>

Author: Copilot

tests/Unit/BackgroundJobs/ProcessFileJobTest.php

@@ -23,15 +23,10 @@
 
 namespace OCA\WorkflowOcr\Tests\Unit\BackgroundJobs;
 
-use OC\BackgroundJob\JobList;
 use OCA\WorkflowOcr\BackgroundJobs\ProcessFileJob;
 use OCA\WorkflowOcr\Service\IOcrService;
 use OCP\AppFramework\Utility\ITimeFactory;
-use OCP\DB\QueryBuilder\IExpressionBuilder;
-use OCP\DB\QueryBuilder\IQueryBuilder;
-use OCP\IConfig;
-use OCP\IDBConnection;
-use OCP\Snowflake\IGenerator;
+use OCP\BackgroundJob\IJobList;
 use PHPUnit\Framework\MockObject\MockObject;
 use PHPUnit\Framework\TestCase;
 use Psr\Log\LoggerInterface;
@@ -43,10 +38,7 @@ class ProcessFileJobTest extends TestCase {
 	/** @var IOcrService|MockObject */
 	private $ocrService;
 
-	/** @var IGenerator|MockObject */
-	private $generator;
-
-	/** @var JobList */
+	/** @var IJobList|MockObject */
 	private $jobList;
 
 	/** @var ProcessFileJob */
@@ -71,40 +63,7 @@ public function setUp() : void {
 		);
 		$this->processFileJob->setId('111');
 
-		/** @var IConfig */
-		$configMock = $this->createMock(IConfig::class);
-		/** @var ITimeFactory */
-		$timeFactoryMock = $this->createMock(ITimeFactory::class);
-		/** @var MockObject|IDbConnection */
-		$connectionMock = $this->createMock(IDBConnection::class);
-		/** @var MockObject|IQueryBuilder */
-		$queryBuilderMock = $this->createMock(IQueryBuilder::class);
-		$expressionBuilderMock = $this->createMock(IExpressionBuilder::class);
-		$this->generator = $this->createMock(IGenerator::class);
-
-		$queryBuilderMock->method('delete')
-			->withAnyParameters()
-			->willReturn($queryBuilderMock);
-		$queryBuilderMock->method('set')
-			->withAnyParameters()
-			->willReturn($queryBuilderMock);
-		$queryBuilderMock->method('update')
-			->withAnyParameters()
-			->willReturn($queryBuilderMock);
-		$queryBuilderMock->method('expr')
-			->withAnyParameters()
-			->willReturn($expressionBuilderMock);
-		$connectionMock->method('getQueryBuilder')
-			->withAnyParameters()
-			->willReturn($queryBuilderMock);
-
-		$this->jobList = new JobList(
-			$connectionMock,
-			$configMock,
-			$timeFactoryMock,
-			$this->logger,
-			$this->generator
-		);
+		$this->jobList = $this->createMock(IJobList::class);
 
 		$this->processFileJob->setArgument($this->argument);
 	}