From d535c377cfbf2f802f02e79ed7dad6e3a14cfcd0 Mon Sep 17 00:00:00 2001 From: Ferdinand Thiessen Date: Thu, 22 May 2025 13:37:56 +0200 Subject: [PATCH] fix: recommended apps button test Regression of https://github.com/nextcloud/server/pull/52703 where the `>` was added by accident and thus the button has a wrong label including the rest of the button HTML. Signed-off-by: Ferdinand Thiessen --- core/src/components/setup/RecommendedApps.vue | 9 ++++----- cypress/e2e/core/setup.ts | 13 ++++++++++++- 2 files changed, 16 insertions(+), 6 deletions(-) diff --git a/core/src/components/setup/RecommendedApps.vue b/core/src/components/setup/RecommendedApps.vue index b31e4b54ca4..f2120c28402 100644 --- a/core/src/components/setup/RecommendedApps.vue +++ b/core/src/components/setup/RecommendedApps.vue @@ -38,17 +38,16 @@
+ variant="tertiary"> {{ t('core', 'Skip') }} + variant="primary" @click.stop.prevent="installApps"> {{ installingApps ? t('core', 'Installing apps …') : t('core', 'Install recommended apps') }} diff --git a/cypress/e2e/core/setup.ts b/cypress/e2e/core/setup.ts index 498e908585c..a94fdde2b25 100644 --- a/cypress/e2e/core/setup.ts +++ b/cypress/e2e/core/setup.ts @@ -124,8 +124,19 @@ function sharedSetup() { cy.location('pathname', { timeout: 10000 }) .should('include', '/core/apps/recommended') + // See the apps setup + cy.get('[data-cy-setup-recommended-apps]') + .should('be.visible') + .within(() => { + cy.findByRole('heading', { name: 'Recommended apps' }) + .should('be.visible') + cy.findByRole('link', { name: 'Skip' }) + .should('be.visible') + cy.findByRole('button', { name: 'Install recommended apps' }) + .should('be.visible') + }) + // Skip the setup apps - cy.get('[data-cy-setup-recommended-apps]').should('be.visible') cy.get('[data-cy-setup-recommended-apps-skip]').click() // Go to files