Skip to content

Commit

Permalink
Merge pull request #41712 from nextcloud/fix/make-cypress-work-again
Browse files Browse the repository at this point in the history
fix(cypress): Adjust cypress tests that relied on version to be 28
  • Loading branch information
susnux authored Nov 24, 2023
2 parents 704326f + d58b413 commit 957a4e8
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 1 deletion.
1 change: 1 addition & 0 deletions cypress/e2e/theming/navigation-bar-settings.cy.ts
Original file line number Diff line number Diff line change
Expand Up @@ -89,6 +89,7 @@ describe('Admin theming set default apps', () => {
})

it('See the default app is changed back to default', () => {
cy.reload()
cy.get('#nextcloud').click()
cy.url().should('match', /apps\/dashboard/)
})
Expand Down
2 changes: 1 addition & 1 deletion cypress/support/commonUtils.ts
Original file line number Diff line number Diff line change
Expand Up @@ -45,7 +45,7 @@ export function installTestApp() {
const version = output.stdout.match(/(\d\d+)\.\d+\.\d+/)?.[1]
cy.wrap(version).should('not.be.undefined')
cy.exec(`docker cp '${testAppPath}' nextcloud-cypress-tests-server:/var/www/html/apps`, { log: true })
cy.exec(`docker exec nextcloud-cypress-tests-server sed -i 's|version="[0-9]+|version="${version}|' apps/testapp/appinfo/info.xml`)
cy.exec(`docker exec nextcloud-cypress-tests-server sed -i -e 's|-version="[0-9]\\+|-version="${version}|g' apps/testapp/appinfo/info.xml`)
cy.runOccCommand('app:enable testapp')
})
}
Expand Down

0 comments on commit 957a4e8

Please sign in to comment.