test: adjust cypress tests

Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>

Author: susnux

cypress.config.ts

@@ -25,9 +25,9 @@ export default defineConfig({
 	viewportWidth: 1280,
 	viewportHeight: 720,
 
-	// Tries again 2 more times on failure
+	// Tries again when in run mode (cypress run) e.g. on CI
 	retries: {
-		runMode: 2,
+		runMode: 3,
 		// do not retry in `cypress open`
 		openMode: 0,
 	},

cypress/e2e/files/FilesUtils.ts

@@ -33,12 +33,9 @@ export function getActionEntryForFileId(fileid: number, actionId: string) {
  * @param actionId
  */
 export function getActionEntryForFile(file: string, actionId: string) {
-	getActionButtonForFile(file)
-		.if('not.have.attr', 'aria-expanded', 'true')
-		.click({ force: true })
-
-	return cy.findByRole('menu')
-		.find(`[data-cy-files-list-row-action="${CSS.escape(actionId)}"]`)
+	return getActionButtonForFile(file)
+		.should('have.attr', 'aria-controls')
+		.then((menuId) => cy.get(`#${menuId}`).find(`[data-cy-files-list-row-action="${CSS.escape(actionId)}"]`))
 }
 
 /**
@@ -85,8 +82,12 @@ export function triggerActionForFileId(fileid: number, actionId: string) {
  */
 export function triggerActionForFile(filename: string, actionId: string) {
 	getActionButtonForFile(filename)
-		.should('have.attr', 'aria-expanded', 'false')
-		.click({ force: true, scrollBehavior: 'nearest' }) // force to avoid issues with overlaying file list header
+		.should('exist')
+		.scrollIntoView()
+	getActionButtonForFile(filename)
+		.should('be.visible')
+		.and('have.attr', 'aria-expanded', 'false')
+		.click()
 	getActionEntryForFile(filename, actionId)
 		.find('button')
 		.should('be.visible')

cypress/e2e/files_versions/version_naming.cy.ts

@@ -68,8 +68,14 @@ describe('Versions naming', () => {
 		setupTestSharedFileFromUser(user, randomFileName, { update: false })
 		openVersionsPanel(randomFileName)
 
-		cy.get('[data-files-versions-version]').eq(0).find('.action-item__menutoggle').should('not.exist')
-		cy.get('[data-files-versions-version]').eq(0).get('[data-cy-version-action="label"]').should('not.exist')
+		cy.get('[data-files-versions-version]')
+			.eq(0)
+			.find('.action-item__menutoggle')
+			.should('not.exist')
+		cy.get('[data-files-versions-version]')
+			.eq(0)
+			.get('[data-cy-version-action="label"]')
+			.should('not.exist')
 
 		doesNotHaveAction(1, 'label')
 		doesNotHaveAction(2, 'label')