fix(files): Exclude file size of excluded files in personal files view

Signed-off-by: Rens Groothuijsen <l.groothuijsen@alumni.maastrichtuniversity.nl>

Author: rgroothuijsen

apps/files/src/services/PersonalFiles.spec.ts

@@ -0,0 +1,46 @@
+import { Folder } from '@nextcloud/files'
+import { describe, expect, it, vi } from 'vitest'
+import { getContents as getPersonalFiles } from './PersonalFiles.ts'
+
+const getContents = vi.hoisted(() => vi.fn())
+vi.mock('./Files.ts', () => ({ getContents }))
+
+const fakeFolder = new Folder({
+	owner: 'owner',
+	source: 'https://cloud.example.com/remote.php/dav/files/owner/folder',
+	root: '/',
+	size: 3,
+})
+
+const fakeNodes = {
+	contents: [
+		{
+			attributes: {
+				'mount-type': '',
+			},
+			size: 1,
+		},
+		{
+			attributes: {
+				'mount-type': '',
+			},
+			size: 1,
+		},
+		{
+			attributes: {
+				'mount-type': 'group',
+			},
+			size: 1,
+		},
+	],
+	folder: fakeFolder,
+}
+
+describe('Personal files service', () => {
+	getContents.mockImplementationOnce(() => Promise.resolve(fakeNodes))
+
+	it('Excludes the file size of excluded files', async () => {
+		const contents = await getPersonalFiles('/', { signal: new AbortController().signal })
+		expect(contents.folder.size).toBe(2)
+	})
+})

apps/files/src/services/PersonalFiles.ts

@@ -43,6 +43,10 @@ export function getContents(path: string = '/', options: { signal: AbortSignal }
 	return getFiles(path, options)
 		.then((content) => {
 			content.contents = content.contents.filter(isPersonalFile)
+			content.folder.size = content.contents
+				.map((node) => node.size)
+				.filter((size) => size !== undefined)
+				.reduce((total, size) => total + size)
 			return content
 		})
 }