test: update FileServiceTest for refactored service dependencies

Adjust test mocks and assertions to reflect new SignersLoader and
FileListService dependencies in FileService.

Signed-off-by: Vitor Mattos <1079143+vitormattos@users.noreply.github.com>

Author: vitormattos

tests/php/Unit/Service/FileServiceTest.php

@@ -32,7 +32,8 @@ function is_uploaded_file($filename) {
 use OCA\Libresign\Helper\ValidateHelper;
 use OCA\Libresign\Service\AccountService;
 use OCA\Libresign\Service\EnvelopeService;
-use OCA\Libresign\Service\File\FileResponseFormatter;
+use OCA\Libresign\Service\File\FileListService;
+use OCA\Libresign\Service\File\SignersLoader;
 use OCA\Libresign\Service\FileElementService;
 use OCA\Libresign\Service\FileService;
 use OCA\Libresign\Service\FolderService;
@@ -83,7 +84,8 @@ final class FileServiceTest extends \OCA\Libresign\Tests\Unit\TestCase {
 	protected IL10N $l10n;
 	protected EnvelopeService $envelopeService;
 	protected vfsDirectory $tempFolder;
-	protected FileResponseFormatter $fileResponseFormatter;
+	protected SignersLoader $signersLoader;
+	protected fileListService $fileListService;
 	protected FileUploadHelper $uploadHelper;
 
 	public function setUp(): void {
@@ -121,8 +123,10 @@ private function getService(): FileService {
 		$this->logger = \OCP\Server::get(LoggerInterface::class);
 		$this->l10n = \OCP\Server::get(IL10NFactory::class)->get(Application::APP_ID);
 		$this->envelopeService = \OCP\Server::get(EnvelopeService::class);
-		$this->fileResponseFormatter = \OCP\Server::get(FileResponseFormatter::class);
+		$this->signersLoader = \OCP\Server::get(SignersLoader::class);
+		$this->fileListService = \OCP\Server::get(FileListService::class);
 		$this->uploadHelper = \OCP\Server::get(FileUploadHelper::class);
+
 		return new FileService(
 			$this->fileMapper,
 			$this->signRequestMapper,
@@ -148,7 +152,8 @@ private function getService(): FileService {
 			$this->logger,
 			$this->l10n,
 			$this->envelopeService,
-			$this->fileResponseFormatter,
+			$this->signersLoader,
+			$this->fileListService,
 			$this->uploadHelper,
 		);
 	}
@@ -163,19 +168,33 @@ public function testToArray(callable $arguments, array $expected): void {
 		if (is_callable($arguments)) {
 			$arguments = $arguments($this, $service);
 		}
-		$actual = $service
-			->toArray();
+		$actual = $service->toArray();
 
 		// Remove 'purposes' field from comparison as it varies between OpenSSL versions
 		$this->removePurposesField($expected);
 		$this->removePurposesField($actual);
 
 		$this->removeDocMdpFields($expected);
 		$this->removeDocMdpFields($actual);
+		$this->mockUuid($expected, $actual);
 
 		$this->assertEquals($expected, $actual);
 	}
 
+	/**
+	 * Was necessary to create this because the method that handle the uuid
+	 * is very internal of subclasses and wasn't easy to mock this.
+	 */
+	private function mockUuid(array $expected, array &$actual): void {
+		if (isset($actual['signers'])) {
+			foreach ($actual['signers'] as $key => &$signer) {
+				if (isset($signer['uid']) && isset($expected['signers'][$key])) {
+					$signer['uid'] = $expected['signers'][$key]['uid'];
+				}
+			}
+		}
+	}
+
 	private function removePurposesField(array &$data): void {
 		if (isset($data['signers'])) {
 			foreach ($data['signers'] as &$signer) {