test: Use utils function from server and wait for requests

Signed-off-by: Louis Chemineau <[email protected]>

cypress/e2e/filesUtils.ts

@@ -20,37 +20,111 @@
  *
  */
 
+export const getRowForFile = (filename: string) => cy.get(`[data-cy-files-list-row-name="${CSS.escape(filename)}"]`)
+export const getActionsForFile = (filename: string) => getRowForFile(filename).find('[data-cy-files-list-row-actions]')
+export const getActionButtonForFile = (filename: string) => getActionsForFile(filename).findByRole('button', { name: 'Actions' })
+
+const searchForActionInRow = (row: JQuery<HTMLElement>, actionId: string): Cypress.Chainable<JQuery<HTMLElement>> => {
+	const action = row.find(`[data-cy-files-list-row-action="${CSS.escape(actionId)}"]`)
+	if (action.length > 0) {
+		cy.log('Found action in row')
+		return cy.wrap(action)
+	}
+
+	// Else look in the action menu
+	const menuButtonId = row.find('button[aria-controls]').attr('aria-controls')
+	if (menuButtonId === undefined) {
+		return cy.wrap(Cypress.$())
+	}
+
+	// eslint-disable-next-line no-unused-expressions
+	expect(menuButtonId).not.to.be.undefined
+	return cy.get(`#${menuButtonId} [data-cy-files-list-row-action="${CSS.escape(actionId)}"]`)
+}
+
+export const getActionEntryForFile = (filename: string, actionId: string): Cypress.Chainable<JQuery<HTMLElement>> => {
+	// If we cannot find the action in the row, it might be in the action menu
+	return getRowForFile(filename).should('be.visible')
+		.then(row => searchForActionInRow(row, actionId))
+}
+
+export const triggerActionForFile = (filename: string, actionId: string) => {
+	// Even if it's inline, we open the action menu to get all actions visible
+	getActionButtonForFile(filename).click({ force: true })
+	getActionEntryForFile(filename, actionId)
+		.find('button').last()
+		.should('exist').click({ force: true })
+}
+
 export function renameFile(fileName: string, newName: string) {
-	toggleMenuAction(fileName)
-	cy.get(`[data-cy-files-list] [data-cy-files-list-row-action="rename"]`).click()
-	cy.get(`[data-cy-files-list] [data-cy-files-list-row-name="${fileName}"] .files-list__row-rename input`).clear()
-	cy.get(`[data-cy-files-list] [data-cy-files-list-row-name="${fileName}"] .files-list__row-rename input`).type(`${newName}.txt`)
-	cy.get(`[data-cy-files-list] [data-cy-files-list-row-name="${fileName}"] .files-list__row-rename`).submit()
+	getRowForFile(fileName)
+	triggerActionForFile(fileName, 'rename')
+
+	// intercept the move so we can wait for it
+	cy.intercept('MOVE', /\/(remote|public)\.php\/dav\/files\//).as('moveFile')
+
+	getRowForFile(fileName).find('[data-cy-files-list-row-name] input').clear()
+	getRowForFile(fileName).find('[data-cy-files-list-row-name] input').type(`${newName}{enter}`)
 	cy.get('.toast-close').click()
-	cy.wait(500)
+	cy.wait('@moveFile')
 }
 
-export function goToDir(dirName: string) {
-	cy.get(`[data-cy-files-list] [data-cy-files-list-row-name="${dirName}"]`).click()
-	cy.wait(500)
+export function navigateToFolder(dirPath: string) {
+	cy.intercept('PROPFIND', /\/remote.php\/dav\/files\//).as('navigateFolder')
+
+	const directories = dirPath.split('/')
+	for (const directory of directories) {
+		if (directory === '') {
+			continue
+		}
+
+		getRowForFile(directory).should('be.visible').find('[data-cy-files-list-row-name-link]').click()
+		cy.wait('@navigateFolder')
+	}
 }
 
 export function createFolder (dirName: string) {
-	cy.get('.files-list__header .breadcrumb__actions button.action-item__menutoggle').click()
-	cy.get('.v-popper__popper').contains('New folder').click()
-	cy.contains('Folder name').siblings('input').clear()
-	cy.contains('Folder name').siblings('input').type(`${dirName}{enter}`)
-	cy.log('Created folder', dirName)
-	cy.wait(500)
+	cy.intercept('MKCOL', /\/remote.php\/dav\/files\//).as('createFolder')
+
+	// TODO: replace by proper data-cy selectors
+	cy.get('[data-cy-upload-picker] .action-item__menutoggle').first().click()
+	cy.get('[data-cy-upload-picker-menu-entry="newFolder"] button').click()
+	cy.get('[data-cy-files-new-node-dialog]').should('be.visible')
+	cy.get('[data-cy-files-new-node-dialog-input]').type(`{selectall}${dirName}`)
+	cy.get('[data-cy-files-new-node-dialog-submit]').click()
+
+	cy.wait('@createFolder')
+
+	getRowForFile(dirName).should('be.visible')
 }
 
 export function moveFile (fileName: string, dirName: string) {
 	toggleMenuAction(fileName)
 	cy.get(`[data-cy-files-list] [data-cy-files-list-row-action="move-copy"]`).click()
 	cy.get('.file-picker').within(() => {
-		cy.get(`[data-filename="${dirName}"]`).click()
-		cy.contains(`Move to ${dirName}`).click()
-		cy.wait(500)
+		// intercept the copy so we can wait for it
+		cy.intercept('MOVE', /\/(remote|public)\.php\/dav\/files\//).as('moveFile')
+
+		if (dirName === '/') {
+			// select home folder
+			cy.get('button[title="Home"]').should('be.visible').click()
+			// click move
+			cy.contains('button', 'Move').should('be.visible').click()
+		} else if (dirName === '.') {
+			// click move
+			cy.contains('button', 'Copy').should('be.visible').click()
+		} else {
+			const directories = dirName.split('/')
+			directories.forEach((directory) => {
+				// select the folder
+				cy.get(`[data-filename="${directory}"]`).should('be.visible').click()
+			})
+
+			// click move
+			cy.contains('button', `Move to ${directories.at(-1)}`).should('be.visible').click()
+		}
+
+		cy.wait('@moveFile')
 	})
 }
 

cypress/e2e/sidebar.cy.ts

@@ -20,7 +20,7 @@
  *
  */
 
-import { createFolder, goToDir, moveFile, renameFile } from './filesUtils'
+import { createFolder, moveFile, navigateToFolder, renameFile } from './filesUtils'
 import { addComment, addTag, addToFavorites, createPublicShare, removeFromFavorites, showActivityTab } from './sidebarUtils'
 
 describe('Check activity listing in the sidebar', { testIsolation: true }, () => {
@@ -56,8 +56,8 @@ describe('Check activity listing in the sidebar', { testIsolation: true }, () =>
 	})
 
 	it('Has rename activity', () => {
-		renameFile('welcome.txt', 'new name')
-		renameFile('new name.txt', 'welcome')
+		renameFile('welcome.txt', 'new name.txt')
+		renameFile('new name.txt', 'welcome.txt')
 
 		showActivityTab('welcome.txt')
 		cy.get('.activity-entry').first().should('contains.text', 'You renamed')
@@ -66,8 +66,7 @@ describe('Check activity listing in the sidebar', { testIsolation: true }, () =>
 	it('Has move activity', () => {
 		createFolder('Test folder')
 		moveFile('welcome.txt', 'Test folder')
-		cy.get('.toast-close').click({ multiple: true })
-		goToDir('Test folder')
+		navigateToFolder('Test folder')
 
 		showActivityTab('welcome.txt')
 		cy.get('.activity-entry').first().should('contains.text', 'You moved')

cypress/e2e/sidebarUtils.ts

@@ -33,15 +33,20 @@ function closeSidebar() {
 	cy.get('body')
 		.then(($body) => {
 			if ($body.find('.app-sidebar__close').length !== 0) {
-				cy.get('.app-sidebar__close').click({ force: true })
+				// {force: true} as it might be hidden behind toasts
+				cy.get('[data-cy-sidebar] .app-sidebar__close').click({ force: true })
 			}
 		})
 	cy.get('#app-sidebar-vue').should('not.exist')
 }
 
 export function showActivityTab(fileName: string) {
+	cy.intercept('GET', '/ocs/v2.php/apps/activity/api/v2/activity/filter**').as('getActivities')
+
 	showSidebarForFile(fileName)
 	cy.get('#app-sidebar-vue').contains('Activity').click()
+
+	cy.wait('@getActivities')
 }
 
 export function addToFavorites(fileName: string) {
@@ -62,13 +67,17 @@ export function removeFromFavorites(fileName: string) {
  * @param fileName Name of the file to share
  */
 export function createPublicShare(fileName: string) {
+	cy.intercept('POST', '/ocs/v2.php/apps/files_sharing/api/v1/shares').as('createPublicShare')
+
 	showSidebarForFile(fileName)
 	cy.get('#app-sidebar-vue').contains('Sharing').click()
 
 	cy.get('#app-sidebar-vue #tab-sharing').should('be.visible')
 	cy.get('#app-sidebar-vue button.new-share-link').click({ force: true })
 	cy.get('#app-sidebar-vue .sharing-entry__copy').should('be.visible')
 	closeSidebar()
+
+	cy.wait('@createPublicShare')
 }
 
 export function addTag(fileName: string, tag: string) {

cypress/tsconfig.json

@@ -5,7 +5,8 @@
 		"types": [
 			"cypress",
 			"dockerode",
-			"node"
+			"node",
+			"@testing-library/cypress"
 		]
 	}
 }