diff options
author | Ferdinand Thiessen <opensource@fthiessen.de> | 2024-04-17 17:38:41 +0200 |
---|---|---|
committer | Ferdinand Thiessen <opensource@fthiessen.de> | 2024-06-10 18:11:45 +0200 |
commit | 892f17a7a8b32450fe392840875a18cea9dd6078 (patch) | |
tree | 7f36131cac7db7846aae4dcf54879438834a530f | |
parent | 3808a5b0565847284f8a8398d8f10ac7da51e306 (diff) | |
download | nextcloud-server-892f17a7a8b32450fe392840875a18cea9dd6078.tar.gz nextcloud-server-892f17a7a8b32450fe392840875a18cea9dd6078.zip |
feat(cypress): Add tests for files sidebar
Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
-rw-r--r-- | apps/files/src/views/Sidebar.vue | 1 | ||||
-rw-r--r-- | cypress/e2e/files/FilesUtils.ts | 4 | ||||
-rw-r--r-- | cypress/e2e/files/files-sidebar.cy.ts | 98 | ||||
-rw-r--r-- | cypress/support/commands.ts | 9 |
4 files changed, 109 insertions, 3 deletions
diff --git a/apps/files/src/views/Sidebar.vue b/apps/files/src/views/Sidebar.vue index 3b79cda260c..16c7fed24f5 100644 --- a/apps/files/src/views/Sidebar.vue +++ b/apps/files/src/views/Sidebar.vue @@ -23,6 +23,7 @@ <template> <NcAppSidebar v-if="file" ref="sidebar" + data-cy-sidebar v-bind="appSidebar" :force-menu="true" @close="close" diff --git a/cypress/e2e/files/FilesUtils.ts b/cypress/e2e/files/FilesUtils.ts index f7c4c33ccb4..3ff459d4c43 100644 --- a/cypress/e2e/files/FilesUtils.ts +++ b/cypress/e2e/files/FilesUtils.ts @@ -28,7 +28,9 @@ export const getActionButtonForFile = (filename: string) => getActionsForFile(fi export const triggerActionForFile = (filename: string, actionId: string) => { getActionButtonForFile(filename).click({ force: true }) - cy.get(`[data-cy-files-list-row-action="${CSS.escape(actionId)}"] > button`).should('be.visible').click({ force: true }) + cy.get(`[data-cy-files-list-row-action="${CSS.escape(actionId)}"] > button`).should('exist') + cy.get(`[data-cy-files-list-row-action="${CSS.escape(actionId)}"] > button`).scrollIntoView() + cy.get(`[data-cy-files-list-row-action="${CSS.escape(actionId)}"] > button`).click({ force: true }) } export const moveFile = (fileName: string, dirPath: string) => { diff --git a/cypress/e2e/files/files-sidebar.cy.ts b/cypress/e2e/files/files-sidebar.cy.ts new file mode 100644 index 00000000000..8e7dd699913 --- /dev/null +++ b/cypress/e2e/files/files-sidebar.cy.ts @@ -0,0 +1,98 @@ +/** + * @copyright Copyright (c) 2024 Ferdinand Thiessen <opensource@fthiessen.de> + * + * @author Ferdinand Thiessen <opensource@fthiessen.de> + * + * @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 type { User } from '@nextcloud/cypress' +import { getRowForFile, navigateToFolder, triggerActionForFile } from './FilesUtils' + +describe('Files: Sidebar', { testIsolation: true }, () => { + let user: User + let fileId: number = 0 + + beforeEach(() => cy.createRandomUser().then(($user) => { + user = $user + + cy.mkdir(user, '/folder') + cy.uploadContent(user, new Blob([]), 'text/plain', '/file').then((response) => { + fileId = Number.parseInt(response.headers['oc-fileid'] ?? '0') + }) + cy.login(user) + })) + + it('opens the sidebar', () => { + cy.visit('/apps/files') + getRowForFile('file').should('be.visible') + + triggerActionForFile('file', 'details') + + cy.get('[data-cy-sidebar]').should('be.visible') + }) + + it('changes the current fileid', () => { + cy.visit('/apps/files') + getRowForFile('file').should('be.visible') + + triggerActionForFile('file', 'details') + + cy.get('[data-cy-sidebar]').should('be.visible') + cy.url().should('contain', `apps/files/files/${fileId}`) + }) + + it('closes the sidebar on delete', () => { + cy.visit('/apps/files') + getRowForFile('file').should('be.visible') + + // open the sidebar + triggerActionForFile('file', 'details') + // validate it is open + cy.get('[data-cy-sidebar]').should('be.visible') + // wait for the sidebar to be settled + cy.wait(500) + + triggerActionForFile('file', 'delete') + cy.get('[data-cy-sidebar]').should('not.exist') + }) + + it('changes the fileid on delete', () => { + cy.uploadContent(user, new Blob([]), 'text/plain', '/folder/other').then((response) => { + const otherFileId = Number.parseInt(response.headers['oc-fileid'] ?? '0') + cy.login(user) + cy.visit('/apps/files') + + getRowForFile('folder').should('be.visible') + navigateToFolder('folder') + getRowForFile('other').should('be.visible') + + // open the sidebar + triggerActionForFile('other', 'details') + // validate it is open + cy.get('[data-cy-sidebar]').should('be.visible') + cy.url().should('contain', `apps/files/files/${otherFileId}`) + // wait for the sidebar to be settled + cy.wait(500) + + triggerActionForFile('other', 'delete') + cy.get('[data-cy-sidebar]').should('not.exist') + // Ensure the URL is changed + cy.url().should('not.contain', `apps/files/files/${otherFileId}`) + }) + }) +}) diff --git a/cypress/support/commands.ts b/cypress/support/commands.ts index 477a366a421..205835eca8d 100644 --- a/cypress/support/commands.ts +++ b/cypress/support/commands.ts @@ -19,7 +19,9 @@ * along with this program. If not, see <http://www.gnu.org/licenses/>. * */ -/* eslint-disable n/no-unpublished-import */ +// eslint-disable-next-line n/no-extraneous-import +import type { AxiosResponse } from 'axios' + import axios from '@nextcloud/axios' import { addCommands, User } from '@nextcloud/cypress' import { basename } from 'path' @@ -33,10 +35,12 @@ addCommands() declare global { // eslint-disable-next-line @typescript-eslint/no-namespace namespace Cypress { + // eslint-disable-next-line @typescript-eslint/no-explicit-any, @typescript-eslint/no-unused-vars interface Chainable<Subject = any> { /** * Enable or disable a given user */ + // eslint-disable-next-line @typescript-eslint/no-explicit-any enableUser(user: User, enable?: boolean): Cypress.Chainable<Cypress.Response<any>>, /** @@ -49,7 +53,7 @@ declare global { * Upload a raw content to a given user storage. * **Warning**: Using this function will reset the previous session */ - uploadContent(user: User, content: Blob, mimeType: string, target: string, mtime?: number): Cypress.Chainable<void>, + uploadContent(user: User, content: Blob, mimeType: string, target: string, mtime?: number): Cypress.Chainable<AxiosResponse>, /** * Create a new directory @@ -223,6 +227,7 @@ Cypress.Commands.add('uploadContent', (user, blob, mimeType, target, mtime = und }, }) cy.log(`Uploaded content as ${fileName}`, response) + return response } catch (error) { cy.log('error', error) throw new Error('Unable to process fixture') |