Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[5.1] Cypress com_cache joomla dialog fix #43682

Merged

Conversation

LadySolveig
Copy link
Contributor

Pull Request for Issue # .

Summary of Changes

Fix for Cypress test for com_cache 5.1 for com_cache from 5.1 and upwards like suggested here by @muhme #43633 (comment)

Testing Instructions

Review and passing tests

Actual result BEFORE applying this Pull Request

Tests pass

Expected result AFTER applying this Pull Request

Tests pass

Link to documentations

Please select:

  • Documentation link for docs.joomla.org:

  • No documentation changes for docs.joomla.org needed

  • Pull Request link for manual.joomla.org:

  • No documentation changes for manual.joomla.org needed

@laoneo laoneo self-assigned this Jun 20, 2024
@laoneo laoneo enabled auto-merge (squash) June 20, 2024 13:17
@laoneo laoneo added this to the Joomla! 5.1.2 milestone Jun 20, 2024
@laoneo laoneo merged commit d8eb36e into joomla:5.1-dev Jun 20, 2024
3 checks passed
it('can clear expired cache', () => {
cy.get('#toolbar-delete2').click();
cy.get('body').then(($body) => {
if ($body.find('div.buttons-holder button[data-button-ok]').length > 0) {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

For joomla 5 there a command clickDialogConfirm

// Click Joomla Dialog Confirm, isOkay: true = push "ok" button, false = push "cancel" button
Cypress.Commands.add('clickDialogConfirm', (isOkay) => {

it used in many places already, like:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants