chore(cypress): Try to fix flaky e2e tests on file version tests

Assisted-by: ClaudeCode:claude-fable-5
Signed-off-by: David Dreschner <david.dreschner@nextcloud.com>

Author: DerDreschner

cypress/e2e/files/FilesUtils.ts

@@ -62,6 +62,36 @@ export function getInlineActionEntryForFile(file: string, actionId: string) {
 	return cy.get(`[data-cy-files-list-row-name="${CSS.escape(file)}"] [data-cy-files-list-row-action="${CSS.escape(actionId)}"]`)
 }
 
+/**
+ * Open the actions menu of a file row and wait until it is really open.
+ *
+ * Async row re-renders (preview and share-status loading) re-mount the
+ * NcActions component, which silently closes a menu that was just opened.
+ * So click only while the toggle reports "closed" and poll aria-expanded,
+ * retrying until the open state sticks.
+ *
+ * @param getActionButton query for the actions menu toggle of the row
+ * @param attempts remaining retries before giving up
+ */
+function openActionsMenu(getActionButton: () => Cypress.Chainable<JQuery<HTMLElement>>, attempts = 5) {
+	getActionButton().then(($toggle) => {
+		if ($toggle.attr('aria-expanded') !== 'true') {
+			cy.wrap($toggle).click({ force: true }) // force to avoid issues with overlaying file list header
+		}
+	})
+	getActionButton().then(($toggle) => {
+		if ($toggle.attr('aria-expanded') === 'true') {
+			return
+		}
+		if (attempts <= 0) {
+			throw new Error('Actions menu did not open')
+		}
+		// eslint-disable-next-line cypress/no-unnecessary-waiting -- give the popover time to render before retrying
+		cy.wait(200)
+		openActionsMenu(getActionButton, attempts - 1)
+	})
+}
+
 /**
  *
  * @param fileid
@@ -70,8 +100,7 @@ export function getInlineActionEntryForFile(file: string, actionId: string) {
 export function triggerActionForFileId(fileid: number, actionId: string) {
 	getActionButtonForFileId(fileid)
 		.scrollIntoView()
-	getActionButtonForFileId(fileid)
-		.click({ force: true }) // force to avoid issues with overlaying file list header
+	openActionsMenu(() => getActionButtonForFileId(fileid))
 	getActionEntryForFileId(fileid, actionId)
 		.find('button')
 		.should('be.visible')
@@ -86,8 +115,7 @@ export function triggerActionForFileId(fileid: number, actionId: string) {
 export function triggerActionForFile(filename: string, actionId: string) {
 	getActionButtonForFile(filename)
 		.scrollIntoView()
-	getActionButtonForFile(filename)
-		.click({ force: true }) // force to avoid issues with overlaying file list header
+	openActionsMenu(() => getActionButtonForFile(filename))
 	getActionEntryForFile(filename, actionId)
 		.find('button')
 		.should('be.visible')