Skip to content

Stabilization Dragging tests#32961

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

Stabilization Dragging tests#32961
Raushen merged 1 commit intoDevExpress:26_1from
Raushen:Fix-unstable-2

Conversation

@Raushen
Copy link
Copy Markdown
Contributor

@Raushen Raushen commented Mar 18, 2026

No description provided.

@Raushen Raushen self-assigned this Mar 18, 2026
@Raushen Raushen requested a review from a team as a code owner March 18, 2026 17:43
@Raushen Raushen added the 26_1 label Mar 18, 2026
Copilot AI review requested due to automatic review settings March 18, 2026 17:43
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

This PR stabilizes DataGrid row-dragging TestCafe tests by explicitly waiting for the DataGrid (and, where applicable, the second grid) to report a ready state before performing drag/scroll actions.

Changes:

  • Added t.expect(<dataGrid>.isReady()).ok() guards before drag/scroll interactions in row dragging functional tests.
  • Added the same ready-state guard to the row dragging visual screenshot test.

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/rowDragging/visual.ts Waits for the grid to be ready before starting the drag sequence used for the visual regression screenshot.
e2e/testcafe-devextreme/tests/dataGrid/common/rowDragging/functional.ts Adds readiness assertions before multiple drag/scroll scenarios (including cross-component cases) to reduce flakiness.

@Raushen Raushen merged commit 107f12e into DevExpress:26_1 Mar 19, 2026
146 of 148 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