Skip to content

Stabilizing ColumnReordering tests#32974

Merged
Raushen merged 1 commit intoDevExpress:26_1from
Raushen:Fix-unstable-5
Mar 19, 2026
Merged

Stabilizing ColumnReordering tests#32974
Raushen merged 1 commit intoDevExpress:26_1from
Raushen:Fix-unstable-5

Conversation

@Raushen
Copy link
Copy Markdown
Contributor

@Raushen Raushen commented Mar 19, 2026

No description provided.

@Raushen Raushen self-assigned this Mar 19, 2026
@Raushen Raushen requested a review from a team as a code owner March 19, 2026 13:47
@Raushen Raushen added the 26_1 label Mar 19, 2026
Copilot AI review requested due to automatic review settings March 19, 2026 13:47
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Stabilizes DataGrid column reordering TestCafe tests by ensuring the grid widget has finished initializing before drag/screenshot actions run.

Changes:

  • Added await t.expect(dataGrid.isReady()).ok(); at the start of several column reordering visual tests.
  • Added the same readiness assertion to column reordering functional tests that perform early header drag operations.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
e2e/testcafe-devextreme/tests/dataGrid/common/columnReordering/visual.ts Waits for DataGrid readiness before drag operations and screenshot comparisons in visual tests.
e2e/testcafe-devextreme/tests/dataGrid/common/columnReordering/functional.ts Waits for DataGrid readiness before header move/drag operations in functional tests.

@Raushen Raushen merged commit 7a45ac3 into DevExpress:26_1 Mar 19, 2026
106 checks passed
Raushen added a commit to Raushen/DevExtreme that referenced this pull request Mar 19, 2026
Raushen added a commit to Raushen/DevExtreme that referenced this pull request Mar 19, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants