diff --git a/models/repo/upload.go b/models/repo/upload.go index 18834f6b83..c8e45bd3dd 100644 --- a/models/repo/upload.go +++ b/models/repo/upload.go @@ -104,17 +104,6 @@ func GetUploadByUUID(ctx context.Context, uuid string) (*Upload, error) { return upload, nil } -// GetUploadsByUUIDs returns multiple uploads by UUIDS -func GetUploadsByUUIDs(ctx context.Context, uuids []string) ([]*Upload, error) { - if len(uuids) == 0 { - return []*Upload{}, nil - } - - // Silently drop invalid uuids. - uploads := make([]*Upload, 0, len(uuids)) - return uploads, db.GetEngine(ctx).In("uuid", uuids).Find(&uploads) -} - // DeleteUploads deletes multiple uploads func DeleteUploads(ctx context.Context, uploads ...*Upload) (err error) { if len(uploads) == 0 { diff --git a/tests/e2e/repo-files.test.e2e.ts b/tests/e2e/repo-files.test.e2e.ts index ab8813eff9..b9d098c926 100644 --- a/tests/e2e/repo-files.test.e2e.ts +++ b/tests/e2e/repo-files.test.e2e.ts @@ -7,40 +7,40 @@ /// -import { expect } from '@playwright/test'; -import { test, dynamic_id, save_visual } from './utils_e2e.ts'; +import {expect} from '@playwright/test'; +import {test, dynamic_id, save_visual} from './utils_e2e.ts'; -test.use({ user: 'user2' }); +test.use({user: 'user2'}); -test('drap and drop upload', async ({ page }, workerInfo) => { +test('drap and drop upload', async ({page}) => { const response = await page.goto(`/user2/file-uploads/_upload/main/`); expect(response?.status()).toBe(200); // Status OK const testID = dynamic_id(); - const dropzone = page.getByRole('button', { name: 'Drop files or click here to upload.' }); + const dropzone = page.getByRole('button', {name: 'Drop files or click here to upload.'}); // create the virtual files const dataTransfer = await page.evaluateHandle(() => { const dt = new DataTransfer(); // add items in different folders - dt.items.add(new File(["Filecontent (dir1/file1.txt)"], 'dir1/file1.txt', { type: 'text/plain' })); - dt.items.add(new File(["Another file's content (double/nested/file.txt)"], 'double/nested/file.txt', { type: 'text/plain' })); - dt.items.add(new File(["Root file (root_file.txt)"], 'root_file.txt', { type: 'text/plain' })); + dt.items.add(new File(['Filecontent (dir1/file1.txt)'], 'dir1/file1.txt', {type: 'text/plain'})); + dt.items.add(new File(["Another file's content (double/nested/file.txt)"], 'double/nested/file.txt', {type: 'text/plain'})); + dt.items.add(new File(['Root file (root_file.txt)'], 'root_file.txt', {type: 'text/plain'})); return dt; }); // and drop them to the upload area - await dropzone.dispatchEvent('drop', { dataTransfer }); + await dropzone.dispatchEvent('drop', {dataTransfer}); await page.getByText('new branch').click(); await save_visual(page); - await page.getByRole('textbox', { name: 'Name the new branch for this' }).fill(testID); - await page.getByRole('button', { name: 'Propose file change' }).click(); + await page.getByRole('textbox', {name: 'Name the new branch for this'}).fill(testID); + await page.getByRole('button', {name: 'Propose file change'}).click(); // check that nested file structure is preserved - await expect(page.getByRole('link', { name: 'dir1/file1.txt' })).toBeVisible(); - await expect(page.getByRole('link', { name: 'double/nested/file.txt' })).toBeVisible(); - await expect(page.locator('#diff-file-boxes').getByRole('link', { name: 'root_file.txt', exact: true })).toBeVisible(); + await expect(page.getByRole('link', {name: 'dir1/file1.txt'})).toBeVisible(); + await expect(page.getByRole('link', {name: 'double/nested/file.txt'})).toBeVisible(); + await expect(page.locator('#diff-file-boxes').getByRole('link', {name: 'root_file.txt', exact: true})).toBeVisible(); await save_visual(page); });