test: cover detailed file sizes in file list service

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

Author: vitormattos

tests/php/Unit/Service/File/FileListServiceTest.php

@@ -17,6 +17,9 @@
 use OCA\Libresign\Service\FileElementService;
 use OCA\Libresign\Service\IdentifyMethodService;
 use OCA\Libresign\Tests\Unit\TestCase;
+use OCP\Files\File as NodeFile;
+use OCP\Files\Folder;
+use OCP\Files\IRootFolder;
 use OCP\IAppConfig;
 use OCP\IL10N;
 use OCP\IURLGenerator;
@@ -34,6 +37,8 @@ final class FileListServiceTest extends TestCase {
 	private IAppConfig&MockObject $appConfig;
 	private IL10N&MockObject $l10n;
 	private IUserManager&MockObject $userManager;
+	private IRootFolder&MockObject $rootFolder;
+	private Folder&MockObject $userFolder;
 	private IUser&MockObject $user;
 
 	public function setUp(): void {
@@ -47,8 +52,11 @@ public function setUp(): void {
 		$this->appConfig = $this->createMock(IAppConfig::class);
 		$this->l10n = $this->createMock(IL10N::class);
 		$this->userManager = $this->createMock(IUserManager::class);
+		$this->rootFolder = $this->createMock(IRootFolder::class);
+		$this->userFolder = $this->createMock(Folder::class);
 
 		$this->user = $this->createMock(IUser::class);
+		$this->rootFolder->method('getUserFolder')->willReturn($this->userFolder);
 	}
 
 	private function getService(): FileListService {
@@ -60,7 +68,8 @@ private function getService(): FileListService {
 			$this->urlGenerator,
 			$this->appConfig,
 			$this->l10n,
-			$this->userManager
+			$this->userManager,
+			$this->rootFolder,
 		);
 	}
 
@@ -492,6 +501,55 @@ public function testDocMdpLevelIncludedInChildFilesResponse(): void {
 		$this->assertEquals(2, $result['files'][0]['docmdpLevel']);
 	}
 
+	public function testDetailedFileIncludesSize(): void {
+		$file = self::createFileEntity(1, 'file', 'doc.pdf');
+
+		$this->signRequestMapper->method('getByMultipleFileId')->willReturn([]);
+		$this->signRequestMapper->method('getIdentifyMethodsFromSigners')->willReturn([]);
+		$this->signRequestMapper->method('getVisibleElementsFromSigners')->willReturn([]);
+
+		$fileNode = $this->createMock(NodeFile::class);
+		$fileNode->method('getSize')->willReturn(4096);
+		$this->userFolder->method('getFirstNodeById')->with(100)->willReturn($fileNode);
+
+		$service = $this->getService();
+		$result = $service->formatSingleFile($this->user, $file);
+
+		$this->assertSame(4096, $result['size']);
+		$this->assertSame(4096, $result['files'][0]['size']);
+	}
+
+	public function testEnvelopeDetailedFileIncludesAggregateSize(): void {
+		$main = self::createFileEntity(1, 'envelope', 'envelope.pdf', ['filesCount' => 2]);
+		$childA = self::createFileEntity(2, 'file', 'child-a.pdf');
+		$childB = self::createFileEntity(3, 'file', 'child-b.pdf');
+
+		$this->signRequestMapper->method('getByFileId')->with(1)->willReturn([]);
+		$this->signRequestMapper->method('getByMultipleFileId')->with([2, 3])->willReturn([]);
+		$this->signRequestMapper->method('getIdentifyMethodsFromSigners')->willReturn([]);
+		$this->fileMapper->method('getTextOfStatus')->willReturn('Draft');
+
+		$fileNodeA = $this->createMock(NodeFile::class);
+		$fileNodeA->method('getSize')->willReturn(1024);
+		$fileNodeB = $this->createMock(NodeFile::class);
+		$fileNodeB->method('getSize')->willReturn(2048);
+		$this->userFolder->method('getFirstNodeById')->willReturnMap([
+			[200, $fileNodeA],
+			[300, $fileNodeB],
+		]);
+
+		$mockUser = $this->createMock(IUser::class);
+		$mockUser->method('getDisplayName')->willReturn('Requester');
+		$this->userManager->method('get')->willReturn($mockUser);
+
+		$service = $this->getService();
+		$result = $service->formatFileWithChildren($main, [$childA, $childB], $this->user);
+
+		$this->assertSame(3072, $result['size']);
+		$this->assertSame(1024, $result['files'][0]['size']);
+		$this->assertSame(2048, $result['files'][1]['size']);
+	}
+
 	private static function createFileEntity(
 		int $id,
 		string $nodeType,