Skip to content

cypress: test grids#5281

Draft
sreichel wants to merge 11 commits intoOpenMage:mainfrom
sreichel:cypress/update-2
Draft

cypress: test grids#5281
sreichel wants to merge 11 commits intoOpenMage:mainfrom
sreichel:cypress/update-2

Conversation

@sreichel
Copy link
Contributor

@sreichel sreichel commented Feb 27, 2026

Description (*)

Fixes/changes:

  • added missing default sort to some grids
  • added missing sort to session storage to some grids

Tests:

  • added tests for grid sorting
  • let some tests correctly fail ...

Issues:

When adding new entris from admin grids, most models have kind of input validation. Some have not. Some only have js validation for required input.

  • search terms
  • cms blocks
  • cms pages
  • customer groups
  • both promo rules
  • designs

Manual testing scenarios (*)

  1. remove html classes from input fields
  2. submit empty form

It should be an error shown, but form is saved.

Questions or comments

Local tests require ddev + cypress-addon.

PR welcome to fix.

@github-actions github-actions bot added Template : admin Relates to admin template Component: Adminhtml Relates to Mage_Adminhtml Component: Widget Relates to Mage_Widget Component: Index Relates to Mage_Index cypress labels Feb 27, 2026
@sonarqubecloud
Copy link

sonarqubecloud bot commented Mar 1, 2026

Quality Gate Failed Quality Gate failed

Failed conditions
53.0% Duplication on New Code (required ≤ 3%)

See analysis details on SonarQube Cloud

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

Labels

Component: Adminhtml Relates to Mage_Adminhtml Component: Index Relates to Mage_Index Component: Widget Relates to Mage_Widget cypress Template : admin Relates to admin template

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant