Skip to content
Merged
Show file tree
Hide file tree
Changes from 1 commit
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
feat: add rename action with tests
Signed-off-by: John Molakvoæ <[email protected]>
  • Loading branch information
skjnldsv committed Jun 22, 2023
commit 135929c38ca9f61b41c23a207e7fdaa37b43c011
2 changes: 1 addition & 1 deletion apps/files/src/actions/deleteAction.spec.ts
Original file line number Diff line number Diff line change
Expand Up @@ -25,8 +25,8 @@ import { File, Folder, Permission } from '@nextcloud/files'
import { FileAction } from '../services/FileAction'
import * as eventBus from '@nextcloud/event-bus'
import axios from '@nextcloud/axios'
import type { Navigation } from '../services/Navigation'
import logger from '../logger'
import type { Navigation } from '../services/Navigation'

const view = {
id: 'files',
Expand Down
111 changes: 111 additions & 0 deletions apps/files/src/actions/renameAction.spec.ts
Original file line number Diff line number Diff line change
@@ -0,0 +1,111 @@
/**
* @copyright Copyright (c) 2023 John Molakvoæ <[email protected]>
*
* @author John Molakvoæ <[email protected]>
*
* @license AGPL-3.0-or-later
*
* This program is free software: you can redistribute it and/or modify
* it under the terms of the GNU Affero General Public License as
* published by the Free Software Foundation, either version 3 of the
* License, or (at your option) any later version.
*
* This program is distributed in the hope that it will be useful,
* but WITHOUT ANY WARRANTY; without even the implied warranty of
* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
* GNU Affero General Public License for more details.
*
* You should have received a copy of the GNU Affero General Public License
* along with this program. If not, see <http://www.gnu.org/licenses/>.
*
*/
import { action } from './renameAction'
import { expect } from '@jest/globals'
import { File, Permission } from '@nextcloud/files'
import { FileAction } from '../services/FileAction'
import * as eventBus from '@nextcloud/event-bus'
import type { Navigation } from '../services/Navigation'

const view = {
id: 'files',
name: 'Files',
} as Navigation

describe('Rename action conditions tests', () => {
test('Default values', () => {
expect(action).toBeInstanceOf(FileAction)
expect(action.id).toBe('rename')
expect(action.displayName([], view)).toBe('Rename')
expect(action.iconSvgInline([], view)).toBe('SvgMock')
expect(action.default).toBe(false)
expect(action.order).toBe(10)
})
})

describe('Rename action enabled tests', () => {
test('Enabled for node with UPDATE permission', () => {
const file = new File({
id: 1,
source: 'https://cloud.domain.com/remote.php/dav/files/admin/foobar.txt',
owner: 'admin',
mime: 'text/plain',
permissions: Permission.UPDATE,
})

expect(action.enabled).toBeDefined()
expect(action.enabled!([file], view)).toBe(true)
})

test('Disabled for node without UPDATE permission', () => {
const file = new File({
id: 1,
source: 'https://cloud.domain.com/remote.php/dav/files/admin/foobar.txt',
owner: 'admin',
mime: 'text/plain',
permissions: Permission.READ,
})

expect(action.enabled).toBeDefined()
expect(action.enabled!([file], view)).toBe(false)
})

test('Disabled if more than one node', () => {
window.OCA = { Files: { Sidebar: {} } }

const file1 = new File({
id: 1,
source: 'https://cloud.domain.com/remote.php/dav/files/admin/foo.txt',
owner: 'admin',
mime: 'text/plain',
})
const file2 = new File({
id: 1,
source: 'https://cloud.domain.com/remote.php/dav/files/admin/bar.txt',
owner: 'admin',
mime: 'text/plain',
})

expect(action.enabled).toBeDefined()
expect(action.enabled!([file1, file2], view)).toBe(false)
})
})

describe('Rename action exec tests', () => {
test('Rename', async () => {
jest.spyOn(eventBus, 'emit')

const file = new File({
id: 1,
source: 'https://cloud.domain.com/remote.php/dav/files/admin/foobar.txt',
owner: 'admin',
mime: 'text/plain',
})

const exec = await action.exec(file, view, '/')

// Silent action
expect(exec).toBe(null)
expect(eventBus.emit).toBeCalledTimes(1)
expect(eventBus.emit).toHaveBeenCalledWith('files:node:rename', file)
})
})
51 changes: 51 additions & 0 deletions apps/files/src/actions/renameAction.ts
Original file line number Diff line number Diff line change
@@ -0,0 +1,51 @@
/**
* @copyright Copyright (c) 2023 John Molakvoæ <[email protected]>
*
* @author John Molakvoæ <[email protected]>
*
* @license AGPL-3.0-or-later
*
* This program is free software: you can redistribute it and/or modify
* it under the terms of the GNU Affero General Public License as
* published by the Free Software Foundation, either version 3 of the
* License, or (at your option) any later version.
*
* This program is distributed in the hope that it will be useful,
* but WITHOUT ANY WARRANTY; without even the implied warranty of
* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
* GNU Affero General Public License for more details.
*
* You should have received a copy of the GNU Affero General Public License
* along with this program. If not, see <http://www.gnu.org/licenses/>.
*
*/
import { Permission, type Node } from '@nextcloud/files'
import { translate as t } from '@nextcloud/l10n'
import PencilSvg from '@mdi/svg/svg/pencil.svg?raw'

import { emit } from '@nextcloud/event-bus'
import { registerFileAction, FileAction } from '../services/FileAction'

export const ACTION_DETAILS = 'details'

export const action = new FileAction({
id: 'rename',
displayName: () => t('files', 'Rename'),
iconSvgInline: () => PencilSvg,

enabled: (nodes: Node[]) => {
return nodes.length > 0 && nodes
.map(node => node.permissions)
.every(permission => (permission & Permission.UPDATE) !== 0)
},

async exec(node: Node) {
// Renaming is a built-in feature of the files app
emit('files:node:rename', node)
return null
},

order: 10,
})

registerFileAction(action)
2 changes: 1 addition & 1 deletion apps/files/src/actions/sidebarAction.spec.ts
Original file line number Diff line number Diff line change
Expand Up @@ -42,7 +42,7 @@ describe('Open sidebar action conditions tests', () => {
})
})

describe('Open folder action enabled tests', () => {
describe('Open sidebar action enabled tests', () => {
test('Enabled for ressources within user root folder', () => {
window.OCA = { Files: { Sidebar: {} } }

Expand Down