fix(cypress): close attachments file before proceeding

Deleting the file later on failed because the file still was locked.

Signed-off-by: Max <[email protected]>

Author: max-nextcloud

cypress/e2e/attachments.spec.js

@@ -104,6 +104,8 @@ const checkAttachment = (documentId, fileName, fileId, index, isImage = true) =>
 			const srcFileNameParam = isImage ? 'imageFileName' : 'mediaFileName'
 
 			cy.wrap($el)
+				// ensure the image is not hidden behind the menu bar.
+				.scrollIntoView({ offset: { top: -50, left: 0 } })
 				.should('be.visible')
 				.find('img')
 				.should('have.attr', 'src')
@@ -220,6 +222,7 @@ describe('Test all attachment insertion methods', () => {
 
 				return waitForRequestAndCheckAttachment(requestAlias)
 			})
+		cy.closeFile()
 	})
 
 	it('Upload a local image file (table.png)', () => {
@@ -236,6 +239,7 @@ describe('Test all attachment insertion methods', () => {
 
 				return waitForRequestAndCheckAttachment(requestAlias)
 			})
+		cy.closeFile()
 	})
 
 	it('Upload a local media file (file.txt.gz)', () => {
@@ -252,6 +256,7 @@ describe('Test all attachment insertion methods', () => {
 
 				return waitForRequestAndCheckAttachment(requestAlias, undefined, false)
 			})
+		cy.closeFile()
 	})
 
 	it('Upload image files with the same name', () => {
@@ -280,8 +285,9 @@ describe('Test all attachment insertion methods', () => {
 					assertImage(index).then(resolve, reject)
 				})
 			})
-		return cy.getEditor().find('[data-component="image-view"]')
+		cy.getEditor().find('[data-component="image-view"]')
 			.should('have.length', 3)
+		cy.closeFile()
 	})
 
 	it('test if attachment files are in the attachment folder', () => {

cypress/support/commands.js

@@ -306,7 +306,7 @@ Cypress.Commands.add('openFileInShare', fileName => {
 	cy.wait(250)
 })
 
-Cypress.Commands.add('closeFile', (fileName, params = {}) => {
+Cypress.Commands.add('closeFile', (params = {}) => {
 	cy.intercept({ method: 'POST', url: '**/apps/text/session/close' })
 		.as('close')
 	cy.get('#viewer .modal-header button.header-close').click(params)