Skip to content

Commit

Permalink
fix(cypress): Adjust cypress tests for changed theming
Browse files Browse the repository at this point in the history
Signed-off-by: Ferdinand Thiessen <[email protected]>
  • Loading branch information
susnux committed Feb 7, 2024
1 parent 2d548e6 commit 31a72b4
Show file tree
Hide file tree
Showing 2 changed files with 11 additions and 6 deletions.
13 changes: 9 additions & 4 deletions cypress/e2e/theming/themingUtils.ts
Original file line number Diff line number Diff line change
Expand Up @@ -21,8 +21,8 @@
*/
import { colord } from 'colord'

export const defaultPrimary = '#0082c9'
export const defaultAccessiblePrimary = '#00679e'
const defaultNextcloudBlue = '#0082c9'
export const defaultPrimary = '#00679e'
export const defaultBackground = 'kamil-porembinski-clouds.jpg'

/**
Expand Down Expand Up @@ -60,14 +60,19 @@ export const validateUserThemingDefaultCss = function(expectedColor = defaultPri

const defaultOptionBackground = defaultSelectButton.css('background-image')
const colorPickerOptionColor = defaultSelectButton.css('background-color')
const isNextcloudBlue = colord(colorPickerOptionColor).isEqual('#0082c9')

const isValidBackgroundImage = !expectedBackground
? defaultOptionBackground === 'none'
: defaultOptionBackground.includes(expectedBackground)

console.debug({ colorPickerOptionColor: colord(colorPickerOptionColor).toHex(), expectedColor, isValidBackgroundImage })
console.debug({ colorPickerOptionColor: colord(colorPickerOptionColor).toHex(), expectedColor, isValidBackgroundImage, isNextcloudBlue })

return isValidBackgroundImage && colord(colorPickerOptionColor).isEqual(expectedColor)
return isValidBackgroundImage && (
colord(colorPickerOptionColor).isEqual(expectedColor)
// we replace nextcloud blue with the the default rpimary (apps/theming/lib/Themes/DefaultTheme.php line 76)
|| (isNextcloudBlue && colord(expectedColor).isEqual(defaultPrimary))
)
}

export const pickRandomColor = function(): Cypress.Chainable<string> {
Expand Down
4 changes: 2 additions & 2 deletions cypress/e2e/theming/user-background.cy.ts
Original file line number Diff line number Diff line change
Expand Up @@ -95,7 +95,7 @@ describe('User select shipped backgrounds and remove background', function() {

// Validate changed background and primary
cy.wait('@setBackground')
cy.waitUntil(() => validateBodyThemingCss('#56633d', background))
cy.waitUntil(() => validateBodyThemingCss('#869171', background))
})

it('Remove background', function() {
Expand All @@ -109,7 +109,7 @@ describe('User select shipped backgrounds and remove background', function() {

// Validate clear background
cy.wait('@clearBackground')
cy.waitUntil(() => validateBodyThemingCss('#56633d', null))
cy.waitUntil(() => validateBodyThemingCss('#869171', null))
})
})

Expand Down

0 comments on commit 31a72b4

Please sign in to comment.