Skip to content

Commit

Permalink
test(cypress): Avoid failure when editor is updated on webdav requests
Browse files Browse the repository at this point in the history
Signed-off-by: Julius Härtl <[email protected]>
  • Loading branch information
juliusknorr authored and mejo- committed May 28, 2024
1 parent 73c8b23 commit 9b5aef8
Showing 1 changed file with 9 additions and 4 deletions.
13 changes: 9 additions & 4 deletions cypress/e2e/propfind.spec.js
Original file line number Diff line number Diff line change
Expand Up @@ -33,7 +33,6 @@ describe('Text PROPFIND extension ', function() {

beforeEach(function() {
cy.login(user)
cy.visit('/apps/files')
})

describe('with workspaces enabled', function() {
Expand All @@ -45,7 +44,9 @@ describe('Text PROPFIND extension ', function() {
// Android app relies on this to detect rich workspace availability
it('always adds rich workspace property', function() {
cy.uploadFile('empty.md', 'text/markdown', '/Readme.md')
cy.visit('/apps/files')
// FIXME: Ideally we do not need a page context for those tests at all
// For now the dashboard avoids that we have failing requests due to conflicts when updating the file
cy.visit('/apps/dashboard')
cy.propfindFolder('/')
.should('have.property', richWorkspace, '')
cy.uploadFile('test.md', 'text/markdown', '/Readme.md')
Expand All @@ -59,7 +60,9 @@ describe('Text PROPFIND extension ', function() {
// Android app relies on this when navigating nested folders
it('adds rich workspace property to nested folders', function() {
cy.createFolder('/workspace')
cy.visit('/apps/files')
// FIXME: Ideally we do not need a page context for those tests at all
// For now the dashboard avoids that we have failing requests due to conflicts when updating the file
cy.visit('/apps/dashboard')
cy.propfindFolder('/', 1)
.then(results => results.pop().propStat[0].properties)
.should('have.property', richWorkspace, '')
Expand All @@ -78,7 +81,9 @@ describe('Text PROPFIND extension ', function() {
})

it('does not return a rich workspace property', function() {
cy.visit('/apps/files')
// FIXME: Ideally we do not need a page context for those tests at all
// For now the dashboard avoids that we have failing requests due to conflicts when updating the file
cy.visit('/apps/dashboard')
cy.propfindFolder('/')
.should('not.have.property', richWorkspace)
cy.uploadFile('test.md', 'text/markdown', '/Readme.md')
Expand Down

0 comments on commit 9b5aef8

Please sign in to comment.