| 123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156157158159160161162163164165166167168169170171172173174175176177178179180181182183184185186187188189190191192193194195196197198199200201202203204205206207208209210211212213214215216217218219220221222223224225226227228229230231232233234235236237238239240241242243244245246247248249250251252253254255256257258259260261262263264265266267268269270271272273274275276277278279280281282283284285286287288289290291292 |
- /**
- * Tests for folder deletion permission gating in the File Manager tree (#1781).
- *
- * Users with only library:delete_own may delete empty, unlinked, non-external
- * folders; everything else stays behind library:delete_all.
- */
- import { describe, it, expect, beforeEach, afterEach } from 'vitest';
- import { screen, waitFor, within } from '@testing-library/react';
- import userEvent from '@testing-library/user-event';
- import { http, HttpResponse } from 'msw';
- import { render } from '../utils';
- import { server } from '../mocks/server';
- import { FileManagerPage } from '../../pages/FileManagerPage';
- import { setAuthToken } from '../../api/client';
- const mockFolders = [
- {
- id: 1,
- name: 'EmptyOne',
- parent_id: null,
- file_count: 0,
- project_id: null,
- archive_id: null,
- project_name: null,
- archive_name: null,
- is_external: false,
- children: [],
- },
- {
- id: 2,
- name: 'HasFiles',
- parent_id: null,
- file_count: 3,
- project_id: null,
- archive_id: null,
- project_name: null,
- archive_name: null,
- is_external: false,
- children: [],
- },
- {
- id: 3,
- name: 'LinkedEmpty',
- parent_id: null,
- file_count: 0,
- project_id: 1,
- archive_id: null,
- project_name: 'My Project',
- archive_name: null,
- is_external: false,
- children: [],
- },
- ];
- function mockAuthUser(permissions: string[]) {
- setAuthToken('test-token', 'session');
- server.use(
- http.get('*/api/v1/auth/status', () =>
- HttpResponse.json({ auth_enabled: true, requires_setup: false }),
- ),
- http.get('*/api/v1/auth/me', () =>
- HttpResponse.json({
- id: 7,
- username: 'operator1',
- is_admin: false,
- permissions,
- }),
- ),
- );
- }
- async function openFolderMenu(user: ReturnType<typeof userEvent.setup>, folderName: string) {
- // Walk up to the row itself rather than assuming the name is its direct
- // child — the name sits in a wrapper that also holds the optional
- // last-activity line (#2680).
- const row = screen.getByText(folderName).closest('div.group')!;
- const buttons = within(row).getAllByRole('button');
- // The kebab (MoreVertical) menu toggle is the last button in the row
- await user.click(buttons[buttons.length - 1]);
- return row;
- }
- describe('FileManager folder deletion gating (#1781)', () => {
- beforeEach(() => {
- localStorage.clear();
- server.use(
- http.get('/api/v1/library/folders', () => HttpResponse.json(mockFolders)),
- http.get('/api/v1/library/files', () => HttpResponse.json([])),
- http.get('/api/v1/library/stats', () =>
- HttpResponse.json({
- total_files: 3,
- total_folders: 3,
- total_size_bytes: 1024,
- disk_free_bytes: 10737418240,
- disk_total_bytes: 107374182400,
- }),
- ),
- http.get('/api/v1/projects/', () => HttpResponse.json([{ id: 1, name: 'My Project', color: '#00ae42' }])),
- http.get('/api/v1/archives/', () => HttpResponse.json([])),
- );
- });
- afterEach(() => {
- setAuthToken(null);
- });
- it('enables delete on an empty folder for a delete_own user', async () => {
- mockAuthUser(['library:read_own', 'library:delete_own']);
- render(<FileManagerPage />);
- await waitFor(() => expect(screen.getByText('EmptyOne')).toBeInTheDocument());
- const user = userEvent.setup();
- const row = await openFolderMenu(user, 'EmptyOne');
- const deleteButton = within(row).getByRole('button', { name: 'Delete' });
- expect(deleteButton).not.toBeDisabled();
- });
- it('disables delete on a non-empty folder for a delete_own user, with own-folders tooltip', async () => {
- mockAuthUser(['library:read_own', 'library:delete_own']);
- render(<FileManagerPage />);
- await waitFor(() => expect(screen.getByText('HasFiles')).toBeInTheDocument());
- const user = userEvent.setup();
- const row = await openFolderMenu(user, 'HasFiles');
- const deleteButton = within(row).getByRole('button', { name: 'Delete' });
- expect(deleteButton).toBeDisabled();
- expect(deleteButton).toHaveAttribute(
- 'title',
- 'You can only delete your own folders when everything in them is yours, or empty folders without an owner',
- );
- });
- it('disables delete on a linked folder for a delete_own user, with no-permission tooltip', async () => {
- mockAuthUser(['library:read_own', 'library:delete_own']);
- render(<FileManagerPage />);
- await waitFor(() => expect(screen.getByText('LinkedEmpty')).toBeInTheDocument());
- const user = userEvent.setup();
- const row = await openFolderMenu(user, 'LinkedEmpty');
- const deleteButton = within(row).getByRole('button', { name: 'Delete' });
- expect(deleteButton).toBeDisabled();
- expect(deleteButton).toHaveAttribute('title', 'You do not have permission to delete folders');
- });
- it('disables delete entirely for a user without any delete permission', async () => {
- mockAuthUser(['library:read_own']);
- render(<FileManagerPage />);
- await waitFor(() => expect(screen.getByText('EmptyOne')).toBeInTheDocument());
- const user = userEvent.setup();
- const row = await openFolderMenu(user, 'EmptyOne');
- const deleteButton = within(row).getByRole('button', { name: 'Delete' });
- expect(deleteButton).toBeDisabled();
- expect(deleteButton).toHaveAttribute('title', 'You do not have permission to delete folders');
- });
- it('keeps delete enabled on non-empty folders for a delete_all user', async () => {
- mockAuthUser(['library:read_all', 'library:delete_all']);
- render(<FileManagerPage />);
- await waitFor(() => expect(screen.getByText('HasFiles')).toBeInTheDocument());
- const user = userEvent.setup();
- const row = await openFolderMenu(user, 'HasFiles');
- const deleteButton = within(row).getByRole('button', { name: 'Delete' });
- expect(deleteButton).not.toBeDisabled();
- });
- });
- // #3201: the backend says per folder what the current user may do with it.
- function ownedFolder(overrides: Record<string, unknown>) {
- return {
- parent_id: null,
- file_count: 0,
- project_id: null,
- archive_id: null,
- project_name: null,
- archive_name: null,
- is_external: false,
- external_path: null,
- external_readonly: false,
- latest_activity_at: null,
- created_by_id: 7,
- shared: false,
- can_write: true,
- can_rename: true,
- can_delete: true,
- children: [],
- ...overrides,
- };
- }
- describe('FileManager folder ownership (#3201)', () => {
- const folders = [
- ownedFolder({ id: 10, name: 'MyModels', file_count: 4 }),
- ownedFolder({ id: 11, name: 'ClassProject', created_by_id: 1, shared: true, can_rename: false, can_delete: false }),
- ownedFolder({ id: 12, name: 'PassThrough', created_by_id: 1, can_write: false, can_rename: false, can_delete: false }),
- ];
- let sharedBody: unknown = null;
- beforeEach(() => {
- localStorage.clear();
- sharedBody = null;
- server.use(
- http.get('/api/v1/library/folders', () => HttpResponse.json(folders)),
- http.get('/api/v1/library/files', () => HttpResponse.json([])),
- http.get('/api/v1/library/stats', () =>
- HttpResponse.json({
- total_files: 4,
- total_folders: 3,
- total_size_bytes: 1024,
- disk_free_bytes: 10737418240,
- disk_total_bytes: 107374182400,
- }),
- ),
- http.put('/api/v1/library/folders/:id', async ({ request, params }) => {
- sharedBody = await request.json();
- return HttpResponse.json(ownedFolder({ id: Number(params.id), name: 'x', created_at: '', updated_at: '' }));
- }),
- );
- });
- afterEach(() => {
- setAuthToken(null);
- });
- it('lets an owner rename and delete their own non-empty folder', async () => {
- mockAuthUser(['library:read_own', 'library:update_own', 'library:delete_own']);
- render(<FileManagerPage />);
- await waitFor(() => expect(screen.getByText('MyModels')).toBeInTheDocument());
- const user = userEvent.setup();
- const row = await openFolderMenu(user, 'MyModels');
- expect(within(row).getByRole('button', { name: 'Rename' })).not.toBeDisabled();
- expect(within(row).getByRole('button', { name: 'Delete' })).not.toBeDisabled();
- // Sharing stays with library:update_all.
- expect(within(row).queryByRole('button', { name: 'Share with everyone' })).not.toBeInTheDocument();
- });
- it("disables rename on someone else's shared folder and marks it shared", async () => {
- mockAuthUser(['library:read_own', 'library:update_own', 'library:delete_own']);
- render(<FileManagerPage />);
- await waitFor(() => expect(screen.getByText('ClassProject')).toBeInTheDocument());
- const user = userEvent.setup();
- const row = await openFolderMenu(user, 'ClassProject');
- expect(within(row).getByTitle('Shared with everyone')).toBeInTheDocument();
- expect(within(row).getByRole('button', { name: 'Rename' })).toBeDisabled();
- });
- it('lets an admin share a folder', async () => {
- mockAuthUser(['library:read_all', 'library:update_all']);
- render(<FileManagerPage />);
- await waitFor(() => expect(screen.getByText('MyModels')).toBeInTheDocument());
- const user = userEvent.setup();
- const row = await openFolderMenu(user, 'MyModels');
- await user.click(within(row).getByRole('button', { name: 'Share with everyone' }));
- await waitFor(() => expect(sharedBody).toEqual({ shared: true }));
- });
- it('offers to stop sharing a shared folder', async () => {
- mockAuthUser(['library:read_all', 'library:update_all']);
- render(<FileManagerPage />);
- await waitFor(() => expect(screen.getByText('ClassProject')).toBeInTheDocument());
- const user = userEvent.setup();
- const row = await openFolderMenu(user, 'ClassProject');
- await user.click(within(row).getByRole('button', { name: 'Stop sharing' }));
- await waitFor(() => expect(sharedBody).toEqual({ shared: false }));
- });
- it('disables upload in a folder the user can only pass through', async () => {
- mockAuthUser(['library:read_own', 'library:upload']);
- render(<FileManagerPage />);
- await waitFor(() => expect(screen.getByText('PassThrough')).toBeInTheDocument());
- const user = userEvent.setup();
- const upload = screen.getByRole('button', { name: 'Upload' });
- expect(upload).not.toBeDisabled();
- await user.click(screen.getByText('PassThrough'));
- await waitFor(() => expect(screen.getByRole('button', { name: 'Upload' })).toBeDisabled());
- expect(screen.getByRole('button', { name: 'Upload' })).toHaveAttribute(
- 'title',
- 'You can only add to your own folders and folders shared with everyone',
- );
- expect(screen.getByRole('button', { name: 'New Folder' })).toBeDisabled();
- await user.click(screen.getByText('ClassProject'));
- await waitFor(() => expect(screen.getByRole('button', { name: 'Upload' })).not.toBeDisabled());
- });
- });
|